)]}'
{"jenkins/jobs/networking-bgpvpn.yaml":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f6af44a4d51a3721dad020bd52fa56062ff74831","unresolved":false,"context_lines":[{"line_number":83,"context_line":"          # documentation purposes."},{"line_number":84,"context_line":"          export DEVSTACK_LOCAL_CONFIG\u003d\"enable_plugin networking-bgpvpn git://git.openstack.org/openstack/networking-bgpvpn\""},{"line_number":85,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"NETWORKING_BGPVPN_DRIVER\u003dBGPVPN:BaGPipe:networking_bgpvpn.neutron.services.service_drivers.bagpipe.bagpipe.BaGPipeBGPVPNDriver:default\""},{"line_number":86,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"enable_plugin networking-bagpipe https://git.openstack.org/openstack/networking-bagpipe\""},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"          # Keep localrc to be able to set some vars in pre_test_hook"},{"line_number":89,"context_line":"          export KEEP_LOCALRC\u003d1"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5aef4532_6bd424eb","side":"PARENT","line":86,"updated":"2016-03-10 18:24:32.000000000","message":"Why do you remove the bagpipe lines here?","commit_id":"d15384cb9e309d4ff4b4c7d6921e8d9e79f50030"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"b8181dfd6f464e9dd83ff46ad0372d1d37a9d329","unresolved":false,"context_lines":[{"line_number":83,"context_line":"          # documentation purposes."},{"line_number":84,"context_line":"          export DEVSTACK_LOCAL_CONFIG\u003d\"enable_plugin networking-bgpvpn git://git.openstack.org/openstack/networking-bgpvpn\""},{"line_number":85,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"NETWORKING_BGPVPN_DRIVER\u003dBGPVPN:BaGPipe:networking_bgpvpn.neutron.services.service_drivers.bagpipe.bagpipe.BaGPipeBGPVPNDriver:default\""},{"line_number":86,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"enable_plugin networking-bagpipe https://git.openstack.org/openstack/networking-bagpipe\""},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"          # Keep localrc to be able to set some vars in pre_test_hook"},{"line_number":89,"context_line":"          export KEEP_LOCALRC\u003d1"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5aef4532_cb657845","side":"PARENT","line":86,"in_reply_to":"5aef4532_6bd424eb","updated":"2016-03-10 18:29:22.000000000","message":"Because this will now be done by the rcfile in the repo.\nSee the line below: \n\n          export DEVSTACK_GATE_SETTINGS\u003d$BASE/new/networking-bgpvpn/devstack/devstack-gate-bagpipe-rc","commit_id":"d15384cb9e309d4ff4b4c7d6921e8d9e79f50030"},{"author":{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},"change_message_id":"afbf6ccb10a43327b64f57da082d0b1ac10cc4af","unresolved":false,"context_lines":[{"line_number":36,"context_line":"          export KEEP_LOCALRC\u003d1"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"          function pre_test_hook {{"},{"line_number":39,"context_line":"              if [ -f $BASE/new/networking-bgpvpn/devstack/pre_test_hook.sh ] ; then"},{"line_number":40,"context_line":"                  . $BASE/new/networking-bgpvpn/devstack/pre_test_hook.sh"},{"line_number":41,"context_line":"              fi"},{"line_number":42,"context_line":"          }}"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5aef4532_46c95fab","line":39,"range":{"start_line":39,"start_character":31,"end_line":39,"end_character":32},"updated":"2016-03-10 18:35:28.000000000","message":"these if lines are discouraged, especially if your dependent chaneg is merging first, which it is.","commit_id":"63368126b7fe4292e3a9b1eade15f797ea4eb980"}]}