Attention is currently required from: laforge, dexter.

View Change

2 comments:

• File include/osmocom/mgcp_client/mgcp_client_internal.h:

  • Patch Set #4, Line 22: conn_up

    one mihgt have differentiated betwween UP, DOWN and UNKNOWN. […]

    well, we really never know if they are UP, we just guess they are UP since they answered some time ago. The only thing that really changes is how far ago we ended up knowing that it was alive.
    So I'm not really sure we benefit from adding the UNKNOWN state, imho it makes things more confusing, but I can add it if you think it makes more sense.

• File src/libosmo-mgcp-client/mgcp_client_vty.c:

  • Patch Set #4, Line 676: pool_member->clien

    one could also keep the bool conn_up and add a check here if keepalives are actually enabled. […]

    not sure if it really benefits from operational point of view, since the MGW in UNKNOWN state should be picked at some point (so same as UP), and then at that point it may already become UP.

To view, visit change 33308. To unsubscribe, or for help writing mail filters, visit settings.