Attention is currently required from: osmith, laforge, dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/33310 )
Change subject: mgwpool: Document keepalive feature ......................................................................
Patch Set 1:
(1 comment)
File common/chapters/mgwpool.adoc:
https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/33310/comment/069d22e2_9c279... PS1, Line 211: NOTE: The `keepalive` feature is disabled by default, and must be explicitly configured in order to enable it.
(just curious, not related to review) why?
Because older osmo-mgw versions don't support the "null" endpoint, and they should be configured explicitly using any specific endpoint in order to avoid errors showing up in the log file (the keepalive mechansim would work fine still though).
I'd enable it by default after next osmocom CNI release. This way also we leave users willing it to test it further meanwhile, while not forcing it on everybody for now.