)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"6a1f77119f0081e7d852d9bbc997d7c7f9741877","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Adds an experimental jenkins job to check proper installation"},{"line_number":10,"context_line":"of the BGPVPN service plugin with the dummy driver."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"The job is duplicated, one for master branch, one for"},{"line_number":13,"context_line":"liberty."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I049a521b4e948a042c9b49eaecd9399cf84353d1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"7a2fa921_9064a222","line":12,"updated":"2015-10-07 16:36:07.000000000","message":"These two lines are wrong now, let\u0027s remove them","commit_id":"e6620e64a5f35fc38743fbf869eadcc23153ac2a"}],"jenkins/jobs/networking-bgpvpn.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"ee41ae366a7b0a09084bab4fd9780f9c63032d7c","unresolved":false,"context_lines":[{"line_number":20,"context_line":"          export DEVSTACK_GATE_NEUTRON\u003d1"},{"line_number":21,"context_line":"          # For now, simply enable bringing stack.sh up with the BGPVPN service plugin."},{"line_number":22,"context_line":"          # Another job that isn\u0027t voting yet runs tempest."},{"line_number":23,"context_line":"          export DEVSTACK_GATE_TEMPEST\u003d0"},{"line_number":24,"context_line":"          # Enable only a very minimal set of services"},{"line_number":25,"context_line":"          export OVERRIDE_ENABLED_SERVICES\u003dq-svc,q-agt,q-dhcp,q-l3,key,mysql,rabbit"},{"line_number":26,"context_line":"          export BRANCH_OVERRIDE\u003d{branch-override}"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7a2fa921_1a11e4ac","line":23,"updated":"2015-10-06 04:41:32.000000000","message":"is this really true, or just copy-paste stuff?","commit_id":"5a1258d21539569ba906b6de59d7203e5ed0b996"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"0de6c08dd69c609341f90e03a61c013f11810c53","unresolved":false,"context_lines":[{"line_number":20,"context_line":"          export DEVSTACK_GATE_NEUTRON\u003d1"},{"line_number":21,"context_line":"          # For now, simply enable bringing stack.sh up with the BGPVPN service plugin."},{"line_number":22,"context_line":"          # Another job that isn\u0027t voting yet runs tempest."},{"line_number":23,"context_line":"          export DEVSTACK_GATE_TEMPEST\u003d0"},{"line_number":24,"context_line":"          # Enable only a very minimal set of services"},{"line_number":25,"context_line":"          export OVERRIDE_ENABLED_SERVICES\u003dq-svc,q-agt,q-dhcp,q-l3,key,mysql,rabbit"},{"line_number":26,"context_line":"          export BRANCH_OVERRIDE\u003d{branch-override}"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7a2fa921_dd74c3fe","line":23,"in_reply_to":"7a2fa921_1a11e4ac","updated":"2015-10-06 07:46:49.000000000","message":"It was copy-pasted from networking-ovn.yaml because like the job it was stolen from, this job simply aims at checking that devstack can install our service plugin and that neutron will start.\n\nThe comment isn\u0027t correct though and I will fix it (we don\u0027t have yet a tempest job, this will be the next step).","commit_id":"5a1258d21539569ba906b6de59d7203e5ed0b996"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"aeb0e0fc3829e12efdf0aafbd82013a8577ab0f5","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    wrappers:"},{"line_number":8,"context_line":"      - build-timeout:"},{"line_number":9,"context_line":"          timeout: 125"},{"line_number":10,"context_line":"      - timestamps"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    builders:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"7a2fa921_d33544da","line":9,"updated":"2015-10-07 16:37:29.000000000","message":"Do you really need a timeout of 125 mins? Since you do not run tempest, this looks very long.\n\nPlease adjust this - latest once you move this out of experimental.","commit_id":"dd2dc79ca9e00dab7ec4b56adec267934100ff33"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"1856e46302d82f67d42c8a05773db3f1da909330","unresolved":false,"context_lines":[{"line_number":22,"context_line":"          # Future jobs will run tempest."},{"line_number":23,"context_line":"          export DEVSTACK_GATE_TEMPEST\u003d0"},{"line_number":24,"context_line":"          # Enable only a very minimal set of services"},{"line_number":25,"context_line":"          export OVERRIDE_ENABLED_SERVICES\u003dq-svc,q-agt,q-dhcp,q-l3,key,mysql,rabbit"},{"line_number":26,"context_line":"          export BRANCH_OVERRIDE\u003d{branch-override}"},{"line_number":27,"context_line":"          if [ \"$BRANCH_OVERRIDE\" !\u003d \"default\" ] ; then"},{"line_number":28,"context_line":"              export OVERRIDE_ZUUL_BRANCH\u003d$BRANCH_OVERRIDE"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"7a2fa921_f8557fc1","line":25,"updated":"2015-10-08 19:01:53.000000000","message":"I\u0027d personally suggest you to keep the project-config at a minimum and drive the rest from gate hooks provided. For an example look at the ODL project setup. However this is good too.","commit_id":"dd2dc79ca9e00dab7ec4b56adec267934100ff33"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"b77185d320c873d42dc5456e1590006f32aa5edf","unresolved":false,"context_lines":[{"line_number":22,"context_line":"          # Future jobs will run tempest."},{"line_number":23,"context_line":"          export DEVSTACK_GATE_TEMPEST\u003d0"},{"line_number":24,"context_line":"          # Enable only a very minimal set of services"},{"line_number":25,"context_line":"          export OVERRIDE_ENABLED_SERVICES\u003dq-svc,q-agt,q-dhcp,q-l3,key,mysql,rabbit"},{"line_number":26,"context_line":"          export BRANCH_OVERRIDE\u003d{branch-override}"},{"line_number":27,"context_line":"          if [ \"$BRANCH_OVERRIDE\" !\u003d \"default\" ] ; then"},{"line_number":28,"context_line":"              export OVERRIDE_ZUUL_BRANCH\u003d$BRANCH_OVERRIDE"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"7a2fa921_3edf5f0e","line":25,"in_reply_to":"7a2fa921_f8557fc1","updated":"2015-10-08 20:01:58.000000000","message":"Ok, I\u0027ll keep this as it is, but have it in mind when adding other jobs to cover tempest tests.  Thanks.","commit_id":"dd2dc79ca9e00dab7ec4b56adec267934100ff33"}],"jenkins/jobs/projects.yaml":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"d8afaf3fa10a74bd3d3d159030e2671ad86babd0","unresolved":false,"context_lines":[{"line_number":857,"context_line":"          pipeline: gate"},{"line_number":858,"context_line":"          node: \u0027devstack-precise || devstack-trusty\u0027"},{"line_number":859,"context_line":"          branch-designator: \u0027\u0027"},{"line_number":860,"context_line":"          branch-override: default"},{"line_number":861,"context_line":""},{"line_number":862,"context_line":"- project:"},{"line_number":863,"context_line":"    name: networking-calico"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9a1a9d01_058b8c1f","line":860,"updated":"2015-10-03 10:11:14.000000000","message":"You should duplicate the entry as you mentioned. I\u0027m surprised that our testsuite passes","commit_id":"ed3f13b35595dc7bdf25d6eeb98be7c9fa1171a5"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"6edab983161ce6e880c9c5ab093fad1e629d629d","unresolved":false,"context_lines":[{"line_number":857,"context_line":"          pipeline: gate"},{"line_number":858,"context_line":"          node: \u0027devstack-precise || devstack-trusty\u0027"},{"line_number":859,"context_line":"          branch-designator: \u0027\u0027"},{"line_number":860,"context_line":"          branch-override: default"},{"line_number":861,"context_line":""},{"line_number":862,"context_line":"- project:"},{"line_number":863,"context_line":"    name: networking-calico"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9a1a9d01_10cbac54","line":860,"in_reply_to":"9a1a9d01_058b8c1f","updated":"2015-10-05 13:28:49.000000000","message":"Done.\n\n(I had wrongly assumed that the job name here \u0027{pipeline}-install-dsvm-networking-bgpvpn{branch-designator}\u0027 would match \u0027gate-install-dsvm-networking-bgpvpn-liberty\u0027 in zuul/layout.xml, resulting in \u0027branch-designator\u0027 to take \u0027-liberty\u0027 as a value.)","commit_id":"ed3f13b35595dc7bdf25d6eeb98be7c9fa1171a5"}],"zuul/layout.yaml":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"1856e46302d82f67d42c8a05773db3f1da909330","unresolved":false,"context_lines":[{"line_number":927,"context_line":"    voting: false"},{"line_number":928,"context_line":""},{"line_number":929,"context_line":"  - name: gate-install-dsvm-networking-bgpvpn"},{"line_number":930,"context_line":"    branch: ^(?!stable/(icehouse|juno|kilo)).*$"},{"line_number":931,"context_line":""},{"line_number":932,"context_line":"  - name: gate-networking-odl-python34"},{"line_number":933,"context_line":"    branch: ^(?!stable/(icehouse|juno|kilo)).*$"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"7a2fa921_18f63bf9","line":930,"updated":"2015-10-08 19:01:53.000000000","message":"what about Liberty? Master is mitaka now, so I believe this needs revision now.","commit_id":"dd2dc79ca9e00dab7ec4b56adec267934100ff33"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"2a0d4ba4fcc99763b9e2fd83112fbb4970e225fa","unresolved":false,"context_lines":[{"line_number":927,"context_line":"    voting: false"},{"line_number":928,"context_line":""},{"line_number":929,"context_line":"  - name: gate-install-dsvm-networking-bgpvpn"},{"line_number":930,"context_line":"    branch: ^(?!stable/(icehouse|juno|kilo)).*$"},{"line_number":931,"context_line":""},{"line_number":932,"context_line":"  - name: gate-networking-odl-python34"},{"line_number":933,"context_line":"    branch: ^(?!stable/(icehouse|juno|kilo)).*$"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"7a2fa921_d81f6306","line":930,"in_reply_to":"7a2fa921_18f63bf9","updated":"2015-10-08 19:03:21.000000000","message":"true you don\u0027t have a liberty branch, when you do I am assuming this would work fine?","commit_id":"dd2dc79ca9e00dab7ec4b56adec267934100ff33"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"2b73296b8898646da7c4ee2302db7b05671331bc","unresolved":false,"context_lines":[{"line_number":927,"context_line":"    voting: false"},{"line_number":928,"context_line":""},{"line_number":929,"context_line":"  - name: gate-install-dsvm-networking-bgpvpn"},{"line_number":930,"context_line":"    branch: ^(?!stable/(icehouse|juno|kilo)).*$"},{"line_number":931,"context_line":""},{"line_number":932,"context_line":"  - name: gate-networking-odl-python34"},{"line_number":933,"context_line":"    branch: ^(?!stable/(icehouse|juno|kilo)).*$"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"7a2fa921_6773ad16","line":930,"in_reply_to":"7a2fa921_7e74f734","updated":"2015-10-12 10:53:08.000000000","message":"no, what this does it limit which branches changes this runs against.\n\nDevstack gate picks the base branch based on the branch that the incoming commit is in (falling back to master if that branch doesn\u0027t exist). https://github.com/openstack-infra/devstack-gate/blob/ad6c5a28cd84b9cd315b4c95464496bc775f999f/functions.sh#L337-L361. If you want to do something more clever than that you need to make changes to devstack-gate, as we did with grenade support.","commit_id":"dd2dc79ca9e00dab7ec4b56adec267934100ff33"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"b77185d320c873d42dc5456e1590006f32aa5edf","unresolved":false,"context_lines":[{"line_number":927,"context_line":"    voting: false"},{"line_number":928,"context_line":""},{"line_number":929,"context_line":"  - name: gate-install-dsvm-networking-bgpvpn"},{"line_number":930,"context_line":"    branch: ^(?!stable/(icehouse|juno|kilo)).*$"},{"line_number":931,"context_line":""},{"line_number":932,"context_line":"  - name: gate-networking-odl-python34"},{"line_number":933,"context_line":"    branch: ^(?!stable/(icehouse|juno|kilo)).*$"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"7a2fa921_7e74f734","line":930,"in_reply_to":"7a2fa921_d81f6306","updated":"2015-10-08 20:01:58.000000000","message":"The intent is to have our changes in our one and only master branch to be tested against the master and stable/liberty branches of Neutron (and other Openstack) projects).\n\nIf I\u0027m correct, this is what this statement does.\n\nWhen we fork a stable/liberty branch, I think we will have to change zuul config and possible the jenkins job so that our changes in our stable/liberty branch are tested against stable/liberty for all other projects, and our master against the master of all other projects.  I have only started to discuss this with infra people (Andreas).","commit_id":"dd2dc79ca9e00dab7ec4b56adec267934100ff33"}]}