)]}'
{"jenkins/jobs/networking-ovn.yaml":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"af50db9bf7818dc7922dd3321968e4a43a872856","unresolved":false,"context_lines":[{"line_number":58,"context_line":"      - devstack-logs"},{"line_number":59,"context_line":"      - console-log"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"- job-template:"},{"line_number":62,"context_line":"    name: \u0027{pipeline}-networking-ovn-dsvm-rally{job-suffix}\u0027"},{"line_number":63,"context_line":"    node: \u0027{node}\u0027"},{"line_number":64,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7a740942_afaab1d0","line":61,"updated":"2015-12-10 11:45:52.000000000","message":"I noticed that most rally jobs template are however here http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/rally.yaml","commit_id":"652a53986d21ef34aa401af789fae81fe42fb485"},{"author":{"_account_id":11343,"name":"Gal Sagie","email":"gal.sagie@gmail.com","username":"GalSagie"},"change_message_id":"e9c1cf5bb1c103f9d6cafc4608fc0ab59d3d04c4","unresolved":false,"context_lines":[{"line_number":58,"context_line":"      - devstack-logs"},{"line_number":59,"context_line":"      - console-log"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"- job-template:"},{"line_number":62,"context_line":"    name: \u0027{pipeline}-networking-ovn-dsvm-rally{job-suffix}\u0027"},{"line_number":63,"context_line":"    node: \u0027{node}\u0027"},{"line_number":64,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7a740942_cd72a2e3","line":61,"in_reply_to":"7a740942_afaab1d0","updated":"2015-12-10 13:23:14.000000000","message":"Yes, i have asked that on the rally channel.\n\nBasically it can go either way and there are 2 examples of projects doing it like this.\n\nThe reason why i picked this way, is because i don\u0027t see networking-ovn actually contributing code to Rally, but more using the Neutron scenarios in rally.\n(Unlike all the other projects which has their job configured in the rally project but also their plugins in the rally repo)\n\nIf this logic doesnt make sense, we can move it.","commit_id":"652a53986d21ef34aa401af789fae81fe42fb485"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"af50db9bf7818dc7922dd3321968e4a43a872856","unresolved":false,"context_lines":[{"line_number":87,"context_line":"          export DEVSTACK_LOCAL_CONFIG\u003d\"enable_plugin networking-ovn git://git.openstack.org/openstack/networking-ovn\""},{"line_number":88,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"enable_plugin rally git://git.openstack.org/openstack/rally\""},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"          export OVERRIDE_ENABLED_SERVICES\u003dkey,rally,n-api,n-cpu,n-cond,n-sch,n-crt,n-cauth,n-obj,g-api,g-reg,c-sch,c-api,c-vol,horizon,rabbit,tempest,mysql,dstat,ovn-northd,ovn-controller,q-svc,q-dhcp,q-l3"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"          function post_test_hook {{"},{"line_number":93,"context_line":"              $BASE/new/rally/tests/ci/rally-gate.sh"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7a740942_cc114b5f","line":90,"updated":"2015-12-10 11:45:52.000000000","message":"I see you are already enabling a \"minimal\" set of services only.\nThis is the right decision imho.\n\nWould it be possibly to remove also:\n- cinder\n- horizon\n- tempest (not a big deal anyway)","commit_id":"652a53986d21ef34aa401af789fae81fe42fb485"},{"author":{"_account_id":11343,"name":"Gal Sagie","email":"gal.sagie@gmail.com","username":"GalSagie"},"change_message_id":"e9c1cf5bb1c103f9d6cafc4608fc0ab59d3d04c4","unresolved":false,"context_lines":[{"line_number":87,"context_line":"          export DEVSTACK_LOCAL_CONFIG\u003d\"enable_plugin networking-ovn git://git.openstack.org/openstack/networking-ovn\""},{"line_number":88,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"enable_plugin rally git://git.openstack.org/openstack/rally\""},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"          export OVERRIDE_ENABLED_SERVICES\u003dkey,rally,n-api,n-cpu,n-cond,n-sch,n-crt,n-cauth,n-obj,g-api,g-reg,c-sch,c-api,c-vol,horizon,rabbit,tempest,mysql,dstat,ovn-northd,ovn-controller,q-svc,q-dhcp,q-l3"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"          function post_test_hook {{"},{"line_number":93,"context_line":"              $BASE/new/rally/tests/ci/rally-gate.sh"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7a740942_8d7c9ab6","line":90,"in_reply_to":"7a740942_cc114b5f","updated":"2015-12-10 13:23:14.000000000","message":"Done","commit_id":"652a53986d21ef34aa401af789fae81fe42fb485"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"d5f7ac2c140cb31b26dd20968e413f3b7caa6278","unresolved":false,"context_lines":[{"line_number":87,"context_line":"          export DEVSTACK_LOCAL_CONFIG\u003d\"enable_plugin networking-ovn git://git.openstack.org/openstack/networking-ovn\""},{"line_number":88,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"enable_plugin rally git://git.openstack.org/openstack/rally\""},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"          export OVERRIDE_ENABLED_SERVICES\u003dkey,rally,n-api,n-cpu,n-cond,n-sch,n-crt,n-cauth,n-obj,g-api,g-reg,rabbit,tempest,mysql,dstat,ovn-northd,ovn-controller,q-svc,q-dhcp,q-l3"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"          function post_test_hook {{"},{"line_number":93,"context_line":"              $BASE/new/rally/tests/ci/rally-gate.sh"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7a740942_d4e8e618","line":90,"updated":"2015-12-10 15:10:22.000000000","message":"Another option is you could move this config into a file in the networking-ovn tree to make it easier to change.  That\u0027s what we do for the tempest job (networking-ovn/devstack/devstackgaterc).  We could have a similar file in the rally-jobs directory you created that configures the list of services and anything else we might want later.\n\nWe could do that later too, though.  No big deal.","commit_id":"c1095d813371699e59a97343dcd52347f90718a2"}],"zuul/layout.yaml":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"075f7a670411754bdafc351e7bf6d798be552363","unresolved":false,"context_lines":[{"line_number":1117,"context_line":"    voting: false"},{"line_number":1118,"context_line":""},{"line_number":1119,"context_line":"  - name: gate-networking-ovn-dsvm-rally-nv"},{"line_number":1120,"context_line":"    branch: ^(?!stable/(juno|kilo|liberty)).*$"},{"line_number":1121,"context_line":""},{"line_number":1122,"context_line":"  - name: ^gate-tempest-dsvm-dragonflow"},{"line_number":1123,"context_line":"    branch: ^(?!stable)"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7a740942_6da77c50","line":1120,"updated":"2015-12-13 09:08:30.000000000","message":"networking-ovn has no branches, so this not needed.","commit_id":"c1095d813371699e59a97343dcd52347f90718a2"},{"author":{"_account_id":11343,"name":"Gal Sagie","email":"gal.sagie@gmail.com","username":"GalSagie"},"change_message_id":"5a3f6cc96080d9fa3ec1451fe245b19b65d327d1","unresolved":false,"context_lines":[{"line_number":1117,"context_line":"    voting: false"},{"line_number":1118,"context_line":""},{"line_number":1119,"context_line":"  - name: gate-networking-ovn-dsvm-rally-nv"},{"line_number":1120,"context_line":"    branch: ^(?!stable/(juno|kilo|liberty)).*$"},{"line_number":1121,"context_line":""},{"line_number":1122,"context_line":"  - name: ^gate-tempest-dsvm-dragonflow"},{"line_number":1123,"context_line":"    branch: ^(?!stable)"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7a740942_d0f727d1","line":1120,"in_reply_to":"7a740942_6da77c50","updated":"2015-12-13 09:54:15.000000000","message":"Done","commit_id":"c1095d813371699e59a97343dcd52347f90718a2"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"ca625f249909066b5c16029fdf57cc19a6de343d","unresolved":false,"context_lines":[{"line_number":1116,"context_line":"    branch: ^(?!stable)"},{"line_number":1117,"context_line":"    voting: false"},{"line_number":1118,"context_line":""},{"line_number":1119,"context_line":"  - name: gate-networking-ovn-dsvm-rally-nv"},{"line_number":1120,"context_line":""},{"line_number":1121,"context_line":"  - name: ^gate-tempest-dsvm-dragonflow"},{"line_number":1122,"context_line":"    branch: ^(?!stable)"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7a740942_70c5135a","line":1119,"updated":"2015-12-13 10:01:54.000000000","message":"Remove the line completely, you have no conditions/actions, so it\u0027s not needed at all.","commit_id":"0bf3a6ba9f83c32badbe3883378eaecfd860fe55"}]}
