Attention is currently required from: laforge, pespin, msuraev.

neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/32273 )

Change subject: add static SS7 routing example to cs7-config.adoc ......................................................................

Patch Set 2:

(8 comments)

File common/chapters/cs7-config.adoc:

https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/32273/comment/2324a0d4_01e49... PS1, Line 349: Osmocom SS7 supports dynamic routing, allowing minimal SS7 configuration. If all of your components support dynamic

I'd do a s/dynamic routing/dynamic routing key management/ all over the text. […]

Done

https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/32273/comment/12b6141b_84091... PS1, Line 373: - its own point-code -- has to match the PC in osmo-stp.cfg

Perhaps better: "has to match its own PC entry in osmo-stp. […]

Done

https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/32273/comment/b5352d4a_92beb... PS1, Line 376: - local+remote IP:port for M3UA -- have to match the IP:ports in osmo-stp.cfg

IP:ports -> IP:port?

it's remote and local, two ports...

https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/32273/comment/50688bf4_e4510... PS1, Line 382: - local+remote IP:port for M3UA -- have to match the IP:ports in osmo-stp.cfg

IP:ports -> IP:port?

Done

https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/32273/comment/baa9bbe6_16059... PS1, Line 387: - all point-codes -- they have to match the PC in osmo-bsc.cfg and osmo-msc.cfg

You list first point codes and afterwards routing keys in the MSC and BSC section. […]

Done

https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/32273/comment/f2754bb7_974f1... PS1, Line 388: - local+remote IP:port for M3UA -- have to match the IP:ports in osmo-bsc.cfg and osmo-msc.cfg

IP:ports -> IP:port?

Done

https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/32273/comment/5f6c9192_31115... PS1, Line 454: ty synchronize point-codes and routing keys between STP and {BSC,MSC}:

typo: to

Done

https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/32273/comment/88575ba9_99725... PS1, Line 454: synchronize

I would replace synchronize with "manually configure", as "synchronize" might refer to some kind of […]

Done