Attention is currently required from: laforge, lynxis lazus, pespin.
13 comments:
File src/ipa.c:
Patch Set #8, Line 51: #include "ss7_as_loadshare_tcap.h"
#ifdef WITH_TCAP_LOADSHARING
Done
Patch Set #8, Line 315: /* update TCAP Transaction Tracking (Rx) */
no, not needed.
Done
Patch Set #8, Line 368: if (hh_ext->proto == IPAC_PROTO_EXT_TCAP_ROUTING) {
Should be ok with the ifdef encompassing the whole case.
Done
File src/ss7_as.h:
Patch Set #8, Line 6: #include <osmocom/core/hashtable.h>
Does the include hurt? I would try to prevent cluttering #ifdefs everywhere.
Here I would agree.
File src/ss7_as.c:
Patch Set #8, Line 246: tcap_disable(as);
#ifdef WITH_TCAP_LOADSHARING
Done
File src/ss7_as_loadshare_tcap.h:
Patch Set #8, Line 14: #ifdef WITH_TCAP_LOADSHARING
No, I want to keep it here because of the #else branch.
Done
File src/ss7_as_loadshare_tcap.c:
Patch Set #8, Line 379: /* FIXME: use UTDS */
This needs to be fixed?
It's used in asp_loadshare_tcap_sccp. @lynxis@fe80.eu this comment can be removed?
Patch Set #8, Line 752: LOGPASP(asp, DLSS7, LOGL_ERROR, "TCAP routing message is too small\n");
I think we really want to add a new log category "TCAP" here.
ACK
File src/ss7_as_vty.c:
Patch Set #8, Line 39: #include "ss7_as_loadshare_tcap.h"
No
Done
Patch Set #8, Line 158: tcap_disable(as);
No, not needed.
Done
Patch Set #8, Line 165: "traffic-mode (loadshare|loadshare-tcap) [bindings] [sls] [opc-sls] [opc-shift] [<0-2>]",
I'm not 100% familiar with the tcap_enable/disable hooks, but wouldn't you need to call tcap_disable […]
Done
Patch Set #8, Line 186: vty_out(vty, "tcap loadsharing is not supported in this build.\n");
This is wrong, will fix it in v2
Done
File src/ss7_asp.c:
Patch Set #8, Line 53: #include "ss7_as_loadshare_tcap.h"
#ifdef WITH_TCAP_LOADSHARING
Done
To view, visit change 41309. To unsubscribe, or for help writing mail filters, visit settings.