Page MenuHomeSoftware Heritage

document the firewalls upgrade procedure
ClosedPublic

Authored by vsellier on May 6 2021, 11:12 AM.

Details

Summary

initiate an infrastructure section and a network sub-section

Remark: the png of the plantuml diagram is committed because
the svg in not correctly rendered

Related to T3203

Diff Detail

Repository
rDDOC Development documentation
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

  • split procedure per firewall
  • clarify the initial status before each section
  • fix minor typos
ardumont requested changes to this revision.May 7 2021, 9:16 AM
ardumont added a subscriber: ardumont.

lgtm

couples of questions and suggestions inline. Request changes for the sake of those though.

docs/images/network.png
1

there are 2 gateway inria ^. I gather it's for readability.
In any case, we should probably use the same case sensitivity for both...

or is it a typo on the vlan names? i don't recall what's the vlan 1330 (vs vlan 1300).

docs/images/network.uml
10
48
49

i don't recall which vlan is it?

(probably inria come to think of it since i don't see it declared in the firewall).

What are the differences between the 2 gateways again?

docs/infrastructure/network.rst
92
126

?

147
This revision now requires changes to proceed.May 7 2021, 9:16 AM
docs/images/network.uml
49

yes it's a typo, its 1300, our dedicated public vlan.

It's just cosmetic to have a better organization of the graph. if the same gw is used for the 2 vlans, the rendering is broken.

docs/images/network.uml
49

well it's 210 actually

  • rebase
  • network schema:
    • change the vlan order to be able to use only one gateway
    • Adapt several labels
  • rst: adatp according to the feebacks

network schema: reactivate the description of the firewalls' group

This revision is now accepted and ready to land.May 7 2021, 2:32 PM
This revision was automatically updated to reflect the committed changes.