> On June 20, 2018, 2:22 p.m., Ganesh Murthy wrote: > > 1. Should displayLinkRoutes() function in qpid-dispatch/tools/qdstat be > > modified to show "addExternalPrefix" and "delExternalPrefix" attributes? > > 2. I think we should have an additional test added to > > system_tests_link_routes_add_external_prefix which will run qdmanage query > > on link routes and make sure that "addExternalPrefix" and > > "delExternalPrefix" are showing correctly > > Gordon Sim wrote: > Done. (The extra columns do add a little noise to the qdstat output > considering they will often not be set) > > Ganesh Murthy wrote: > The qdstat test looks good. Can you also please add a qdmanage test > (qdmanage query --type=linkroute) to make sure that the "addExternalPrefix" > and "delExternalPrefix" columns are present.
Done! - Gordon ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66839/#review205080 ----------------------------------------------------------- On June 21, 2018, 3:19 p.m., Gordon Sim wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66839/ > ----------------------------------------------------------- > > (Updated June 21, 2018, 3:19 p.m.) > > > Review request for qpid and Ted Ross. > > > Bugs: DISPATCH-980 > https://issues.apache.org/jira/browse/DISPATCH-980 > > > Repository: qpid-dispatch > > > Description > ------- > > This adds two fields to the link route config, addExternalPrefix and > delExternalPrefix, that behave similar to the externalAddr in auto links. > I.e. they will cause a prefix to be inserted and/or deleted in the final hop > for a link routed link attach. That prefix will also be stripped off or > inserted back into the reply, making the renaming completely transparent to > clients. > > > Diffs > ----- > > include/qpid/dispatch/router_core.h 8f144b0 > python/qpid_dispatch/management/qdrouter.json 42f501f > src/router_config.c 94758df > src/router_core/agent_config_link_route.h 57f6f0d > src/router_core/agent_config_link_route.c 981d142 > src/router_core/connections.c 5fdc3bf > src/router_core/forwarder.c fca19c3 > src/router_core/route_control.h 3c715bc > src/router_core/route_control.c 61f426d > src/router_core/router_core.c 9d7f7a5 > src/router_core/router_core_private.h cd2ffa4 > src/router_core/terminus.c 4c0e0a3 > tests/CMakeLists.txt 159521d > tests/system_tests_link_routes_add_external_prefix.py PRE-CREATION > tools/qdstat 1e3a3a3 > > > Diff: https://reviews.apache.org/r/66839/diff/6/ > > > Testing > ------- > > Added system tests plus some ad hoc testing. > > > Thanks, > > Gordon Sim > >