)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"f175261273e3cfe00012d4f35f0bdb43b3f6e5ed","unresolved":false,"context_lines":[{"line_number":13,"context_line":"for choosing the ovs branch (latest-release or master). Now, it possible"},{"line_number":14,"context_line":"to pass as many options as needed and it will be much easier to extend."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This patch will be used at: 6791b94429f4a2f494e3f45426e074e8608ce917"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Idee833a90c2c2dacea8c92ff58f67f6c3822bb9e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7f287b81_268f632f","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":68},"updated":"2017-09-05 03:50:44.000000000","message":"is this still the case?","commit_id":"f06a77c31275f08986bc74e4b29214eaa18a6db6"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"2cdf8e51a749cdc05011e00b971c7f2086915d54","unresolved":false,"context_lines":[{"line_number":13,"context_line":"for choosing the ovs branch (latest-release or master). Now, it possible"},{"line_number":14,"context_line":"to pass as many options as needed and it will be much easier to extend."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This patch will be used at: 6791b94429f4a2f494e3f45426e074e8608ce917"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Idee833a90c2c2dacea8c92ff58f67f6c3822bb9e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7f287b81_92e9b181","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":68},"in_reply_to":"7f287b81_268f632f","updated":"2017-09-05 08:23:36.000000000","message":"Yeah it is, once/if this patch gets merged we can update the job definition in project-config to start using the devstackgaterc from this repository.\n\nThe goal here is to bring some control over the tests to netowrking-ovn, so change/tweak that regex without having to submit yet another patch to project-config.","commit_id":"f06a77c31275f08986bc74e4b29214eaa18a6db6"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"3698e823b5ef5a6778b50f3dcf619d0c30ce0776","unresolved":false,"context_lines":[{"line_number":13,"context_line":"for choosing the ovs branch (latest-release or master). Now, it possible"},{"line_number":14,"context_line":"to pass as many options as needed and it will be much easier to extend."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This patch will be used at: 6791b94429f4a2f494e3f45426e074e8608ce917"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Idee833a90c2c2dacea8c92ff58f67f6c3822bb9e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7f287b81_5407b6ca","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":68},"in_reply_to":"7f287b81_92e9b181","updated":"2017-09-05 08:46:38.000000000","message":"ok, it makes sense.","commit_id":"f06a77c31275f08986bc74e4b29214eaa18a6db6"}],"devstack/devstackgaterc":[{"author":{"_account_id":10267,"name":"venkata anil","email":"anilvenkata@redhat.com","username":"anil"},"change_message_id":"a55a802d049464c9b41275a913c2eee3d4f03cbc","unresolved":false,"context_lines":[{"line_number":50,"context_line":"fi"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"if [[ \"${OVN_OPTS}\" \u003d\u003d *\"neutron-api-tests\"* ]] ; then"},{"line_number":53,"context_line":"    export DEVSTACK_GATE_TEMPEST\u003d1"},{"line_number":54,"context_line":"    export DEVSTACK_GATE_TEMPEST_ALL_PLUGINS\u003d1"},{"line_number":55,"context_line":"    export DEVSTACK_GATE_TEMPEST_REGEX\u003d\"^neutron.tests.tempest.api\\.\""},{"line_number":56,"context_line":"fi"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"7f287b81_8dda4848","line":53,"updated":"2017-08-29 13:01:49.000000000","message":"As we are already exporting these macros in https://review.openstack.org/#/c/498466/1/jenkins/jobs/networking-ovn.yaml why do we need here again?","commit_id":"49137ef4cb7f40e3651054f1abc483df930cb02d"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"73bb381f2b009b370339f0ba558338ffe52d358f","unresolved":false,"context_lines":[{"line_number":50,"context_line":"fi"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"if [[ \"${OVN_OPTS}\" \u003d\u003d *\"neutron-api-tests\"* ]] ; then"},{"line_number":53,"context_line":"    export DEVSTACK_GATE_TEMPEST\u003d1"},{"line_number":54,"context_line":"    export DEVSTACK_GATE_TEMPEST_ALL_PLUGINS\u003d1"},{"line_number":55,"context_line":"    export DEVSTACK_GATE_TEMPEST_REGEX\u003d\"^neutron.tests.tempest.api\\.\""},{"line_number":56,"context_line":"fi"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"7f287b81_4da460a6","line":53,"in_reply_to":"7f287b81_8dda4848","updated":"2017-08-29 13:17:54.000000000","message":"Hi there,\n\nYesterday on IRC people suggested to have these macros here instead of having in the project-config so we can tweak things around without depending on submitting patches to project-config (which can take time to get merged).","commit_id":"49137ef4cb7f40e3651054f1abc483df930cb02d"},{"author":{"_account_id":10237,"name":"Numan Siddique","email":"nusiddiq@redhat.com","username":"numansiddique"},"change_message_id":"149240f71bb8768536ccc1fd044686248d9e746b","unresolved":false,"context_lines":[{"line_number":19,"context_line":"#"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"# A list of options separated by comma(,), e.g: latest-release,neuton-api-tests"},{"line_number":22,"context_line":"OVN_OPTS\u003d$1"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"OVERRIDE_ENABLED_SERVICES\u003dkey,n-api,n-cpu,n-cond,n-sch,n-crt,n-cauth,n-obj,n-api-meta,placement-api,g-api,g-reg,c-sch,c-api,c-vol,rabbit,mysql,dstat,ovn-northd,ovn-controller,q-svc"},{"line_number":25,"context_line":"# FIXME(dalvarez): Remove this once OVS 2.8 is released. Metadata support depends on it."}],"source_content_type":"application/octet-stream","patch_set":2,"id":"7f287b81_6fd8f451","line":22,"updated":"2017-08-31 11:09:24.000000000","message":"Is it possible to pass the options individually ? (what i mean is instead of \u0027,\u0027 separated, space separated  so that its easier to check)\ni.e latest-release neutron-api-tests ... by project-config job ?\n\nWe can have something like\n\nOVS_MASTER\u003dTrue\nfor i in $@\ndo\n    if [ \"$i\" \u003d\u003d \"latest-release\" ]; then\n        OVS_MASTER\u003dFalse\n    fi\n\n    if [ \"$i\" \u003d\u003d \"neutron-api-tests\"; then\n        export DEVSTACK_GATE_TEMPEST\u003d1\n        export DEVSTACK_GATE_TEMPEST_ALL_PLUGINS\u003d1\n        export DEVSTACK_GATE_TEMPEST_REGEX\u003d\"^neutron.tests.tempest.api\\.\"\n    fi\n    ...\ndone\n\n\nJust checking if this makes sense.","commit_id":"9ce023e18ea179fa498f8fff358238a7f93e6c9a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"5924c38d16f5e9e2c70ed9e79a35f4b716598fb7","unresolved":false,"context_lines":[{"line_number":19,"context_line":"#"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"# A list of options separated by comma(,), e.g: latest-release,neuton-api-tests"},{"line_number":22,"context_line":"OVN_OPTS\u003d$1"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"OVERRIDE_ENABLED_SERVICES\u003dkey,n-api,n-cpu,n-cond,n-sch,n-crt,n-cauth,n-obj,n-api-meta,placement-api,g-api,g-reg,c-sch,c-api,c-vol,rabbit,mysql,dstat,ovn-northd,ovn-controller,q-svc"},{"line_number":25,"context_line":"# FIXME(dalvarez): Remove this once OVS 2.8 is released. Metadata support depends on it."}],"source_content_type":"application/octet-stream","patch_set":2,"id":"7f287b81_a2a498ba","line":22,"in_reply_to":"7f287b81_22466823","updated":"2017-09-01 09:33:19.000000000","message":"I think we can achieve what you want very easily, we just need to replace the OVN_OPTS\u003d$1 with OVN_OPTS\u003d$@. For example:\n\n OVN_OPTS\u003d$@                                                                     \n if [[ \"${OVN_OPTS}\" \u003d\u003d *\"latest-release\"* ]] ; then                             \n     echo \"Latest release\"                                                       \n elif [[ \"${OVN_OPTS}\" \u003d\u003d *\"master\"* ]] ; then                                   \n     echo \"Master\"                                                               \n else                                                                            \n     echo \"No ovs branch specified, using the default from the devstack plugin\"  \n fi                                                                              \n                                                                               \n if [[ \"${OVN_OPTS}\" \u003d\u003d *\"neutron-api-tests\"* ]] ; then                          \n     echo \"Neutron API tests\"                                                    \n fi              \n\n\nIf you save that in a file and run:\n\n $ source testrc latest-release\n Latest release\n $ source testrc latest-release neutron-api-tests\n Latest release\n Neutron API tests\n $ source testrc master\n Master\n $ source testrc master neutron-api-tests\n Master\n Neutron API tests\n $ source testrc\n No ovs branch specified, using the default from the devstack plugin\n\n\nWhat do you think ?","commit_id":"9ce023e18ea179fa498f8fff358238a7f93e6c9a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"f49c492832e1c17c282c419d455dae8d961f10b1","unresolved":false,"context_lines":[{"line_number":19,"context_line":"#"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"# A list of options separated by comma(,), e.g: latest-release,neuton-api-tests"},{"line_number":22,"context_line":"OVN_OPTS\u003d$1"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"OVERRIDE_ENABLED_SERVICES\u003dkey,n-api,n-cpu,n-cond,n-sch,n-crt,n-cauth,n-obj,n-api-meta,placement-api,g-api,g-reg,c-sch,c-api,c-vol,rabbit,mysql,dstat,ovn-northd,ovn-controller,q-svc"},{"line_number":25,"context_line":"# FIXME(dalvarez): Remove this once OVS 2.8 is released. Metadata support depends on it."}],"source_content_type":"application/octet-stream","patch_set":2,"id":"7f287b81_22466823","line":22,"in_reply_to":"7f287b81_6fd8f451","updated":"2017-09-01 09:20:08.000000000","message":"Thanks for the review @numan, yeah it should be possible!\n\nI will work on it","commit_id":"9ce023e18ea179fa498f8fff358238a7f93e6c9a"}]}