)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"42bbc557fa7446b5ed396d90ba93c78f765412fc","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Tom Barron \u003ctpb@dyncloud.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-03-24 07:22:19 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[tests] run all dsvm jobs with python3"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Also override the services enabled in devstack so that"},{"line_number":10,"context_line":"only those run that are actually needed."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5fc1f717_0ac6739b","line":7,"range":{"start_line":7,"start_character":31,"end_line":7,"end_character":38},"updated":"2019-03-25 18:54:51.000000000","message":"You\u0027re also adding tls-proxy to more jobs... When we move these jobs to a more zuulv3 fashion, by the virtue of the base job, all the jobs will enable tls-proxy; so perhaps we want to run with tls-proxy right now..","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"cc894eaba2737a3891995e9d7af648b7acbe044a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Tom Barron \u003ctpb@dyncloud.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-03-24 07:22:19 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[tests] run all dsvm jobs with python3"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Also override the services enabled in devstack so that"},{"line_number":10,"context_line":"only those run that are actually needed."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5fc1f717_750a7c20","line":7,"range":{"start_line":7,"start_character":31,"end_line":7,"end_character":38},"in_reply_to":"5fc1f717_0ac6739b","updated":"2019-03-25 19:14:07.000000000","message":"Yeah - is there any reason to run some jobs without tls-proxy?","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"da527f2df01120bea80401b5dc50bf9085e93a8f","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Tom Barron \u003ctpb@dyncloud.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-03-24 07:22:19 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[tests] run all dsvm jobs with python3"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Also override the services enabled in devstack so that"},{"line_number":10,"context_line":"only those run that are actually needed."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5fc1f717_d65016d8","line":7,"range":{"start_line":7,"start_character":31,"end_line":7,"end_character":38},"in_reply_to":"5fc1f717_750a7c20","updated":"2019-03-25 21:44:59.000000000","message":"Nope...","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"}],"devstack/plugin.sh":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"170d3f88b4e1e2ceff5fb7b6974161535c1955bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"3fce034c_9ef790f6","updated":"2019-04-12 01:30:11.000000000","message":"These changes affect installation of the tempest plugin as part of manila\u0027s Devstack plugin. Developers and CI systems who don\u0027t enable the manila-tempest-plugin in their devstack local.conf files will be affected by this change as we discussed over IRC. Can we break this up into its own patch?","commit_id":"6d95c6e286306004e3142de91702c26f79c37eda"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"d7b15d6eaeb80bf3c3ec53775ad005b3de07e56c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"3fce034c_8a974391","in_reply_to":"3fce034c_9ef790f6","updated":"2019-04-12 14:06:30.000000000","message":"Yes.","commit_id":"6d95c6e286306004e3142de91702c26f79c37eda"}],"playbooks/legacy/manila-tempest-dsvm-container-scenario-custom-image/run.yaml":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"42bbc557fa7446b5ed396d90ba93c78f765412fc","unresolved":false,"context_lines":[{"line_number":46,"context_line":"          export DEVSTACK_GATE_POSTGRES\u003d0"},{"line_number":47,"context_line":"          export DEVSTACK_PROJECT_FROM_GIT\u003d\"python-manilaclient\""},{"line_number":48,"context_line":"          # Basic services needed for minimal job"},{"line_number":49,"context_line":"          export OVERRIDE_ENABLED_SERVICES\u003dkey,mysql,rabbit,tempest,tls-proxy"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5fc1f717_66dd146f","line":49,"range":{"start_line":49,"start_character":17,"end_line":49,"end_character":42},"updated":"2019-03-25 18:54:51.000000000","message":"This scenario test job needs nova, glance and neutron..","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"cc894eaba2737a3891995e9d7af648b7acbe044a","unresolved":false,"context_lines":[{"line_number":46,"context_line":"          export DEVSTACK_GATE_POSTGRES\u003d0"},{"line_number":47,"context_line":"          export DEVSTACK_PROJECT_FROM_GIT\u003d\"python-manilaclient\""},{"line_number":48,"context_line":"          # Basic services needed for minimal job"},{"line_number":49,"context_line":"          export OVERRIDE_ENABLED_SERVICES\u003dkey,mysql,rabbit,tempest,tls-proxy"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5fc1f717_2a224fba","line":49,"range":{"start_line":49,"start_character":17,"end_line":49,"end_character":42},"in_reply_to":"5fc1f717_66dd146f","updated":"2019-03-25 19:14:07.000000000","message":"ack","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"}],"playbooks/legacy/manila-tempest-dsvm-generic-no-share-servers/run.yaml":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"42bbc557fa7446b5ed396d90ba93c78f765412fc","unresolved":false,"context_lines":[{"line_number":58,"context_line":"          # Enable tls-proxy"},{"line_number":59,"context_line":"          OVERRIDE_ENABLED_SERVICES+\u003d,tls-proxy"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"          # Enable mandatory placement services for nova starting with ocata"},{"line_number":62,"context_line":"          if [[ \"stable/newton\" !\u003d $ZUUL_BRANCH ]]; then"},{"line_number":63,"context_line":"               OVERRIDE_ENABLED_SERVICES+\u003d,placement-api,placement-client"},{"line_number":64,"context_line":"          fi"},{"line_number":65,"context_line":"          export OVERRIDE_ENABLED_SERVICES"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5fc1f717_06d8d078","line":64,"range":{"start_line":61,"start_character":10,"end_line":64,"end_character":12},"updated":"2019-03-25 18:54:51.000000000","message":"since newton\u0027s gone we can get rid of this here and in all the jobs..","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"cc894eaba2737a3891995e9d7af648b7acbe044a","unresolved":false,"context_lines":[{"line_number":58,"context_line":"          # Enable tls-proxy"},{"line_number":59,"context_line":"          OVERRIDE_ENABLED_SERVICES+\u003d,tls-proxy"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"          # Enable mandatory placement services for nova starting with ocata"},{"line_number":62,"context_line":"          if [[ \"stable/newton\" !\u003d $ZUUL_BRANCH ]]; then"},{"line_number":63,"context_line":"               OVERRIDE_ENABLED_SERVICES+\u003d,placement-api,placement-client"},{"line_number":64,"context_line":"          fi"},{"line_number":65,"context_line":"          export OVERRIDE_ENABLED_SERVICES"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5fc1f717_f5150c3e","line":64,"range":{"start_line":61,"start_character":10,"end_line":64,"end_character":12},"in_reply_to":"5fc1f717_06d8d078","updated":"2019-03-25 19:14:07.000000000","message":"agree","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"}],"playbooks/legacy/manila-tempest-dsvm-glusterfs-native-heketi/run.yaml":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"42bbc557fa7446b5ed396d90ba93c78f765412fc","unresolved":false,"context_lines":[{"line_number":51,"context_line":"          export PYTHONUNBUFFERED\u003dtrue"},{"line_number":52,"context_line":"          export DEVSTACK_GATE_NEUTRON\u003d1"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"          OVERRIDE_ENABLED_SERVICES\u003dkey,mysql,rabbit,tempest"},{"line_number":55,"context_line":"          export OVERRIDE_ENABLED_SERVICES"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5fc1f717_0a419339","line":54,"range":{"start_line":54,"start_character":53,"end_line":54,"end_character":60},"updated":"2019-03-25 18:54:51.000000000","message":",tls-proxy\n\njust like the others?\n\nOr do you not want to run with tls-proxy in some jobs?","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"cc894eaba2737a3891995e9d7af648b7acbe044a","unresolved":false,"context_lines":[{"line_number":51,"context_line":"          export PYTHONUNBUFFERED\u003dtrue"},{"line_number":52,"context_line":"          export DEVSTACK_GATE_NEUTRON\u003d1"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"          OVERRIDE_ENABLED_SERVICES\u003dkey,mysql,rabbit,tempest"},{"line_number":55,"context_line":"          export OVERRIDE_ENABLED_SERVICES"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5fc1f717_15af78e3","line":54,"range":{"start_line":54,"start_character":53,"end_line":54,"end_character":60},"in_reply_to":"5fc1f717_0a419339","updated":"2019-03-25 19:14:07.000000000","message":"it was an oversite, I want to run with tls everywhere :)","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9be6b82842acfd9ccb54493f056c9c26817a9f16","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5fc1f717_f11d09be","updated":"2019-03-26 17:58:00.000000000","message":"For this and other jobs where you\u0027re limiting the number of services deployed... \n\nPer naming we\u0027re using for minimal jobs:\n\n manila-tempest-minimal-dsvm-glusterfs-native-heketi\n\nI think we ought to take a look at the job names and redo them when we introduce zuulv3 native names.. The presence of \"tempest\" in the name could signify it is a dsvm job... Added this to PTG topics.. [1]\n\n[1] https://etherpad.openstack.org/p/manila-denver-train-ptg-planning","commit_id":"76df2ae6bbd8f845653758e6cf7b542ecd64cbc6"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"8575d970cad5ba03409ac5dfd17e35e52fe665ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5fc1f717_691fe5b6","in_reply_to":"5fc1f717_f11d09be","updated":"2019-03-27 01:19:28.000000000","message":"sure, let\u0027s get a sane set of conventions and rename away.","commit_id":"76df2ae6bbd8f845653758e6cf7b542ecd64cbc6"}],"playbooks/legacy/manila-tempest-minimal-dsvm-lvm/run.yaml":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"42bbc557fa7446b5ed396d90ba93c78f765412fc","unresolved":false,"context_lines":[{"line_number":60,"context_line":"          # Enable tls-proxy"},{"line_number":61,"context_line":"          OVERRIDE_ENABLED_SERVICES+\u003d,tls-proxy"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"          # Enable mandatory placement services for nova starting with ocata"},{"line_number":64,"context_line":"          if [[ \"stable/newton\" !\u003d $ZUUL_BRANCH ]]; then"},{"line_number":65,"context_line":"               OVERRIDE_ENABLED_SERVICES+\u003d,placement-api,placement-client"},{"line_number":66,"context_line":"          fi"},{"line_number":67,"context_line":"          export OVERRIDE_ENABLED_SERVICES"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5fc1f717_aaee3f1f","line":66,"range":{"start_line":63,"start_character":10,"end_line":66,"end_character":12},"updated":"2019-03-25 18:54:51.000000000","message":"would be nice to remove","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"da527f2df01120bea80401b5dc50bf9085e93a8f","unresolved":false,"context_lines":[{"line_number":60,"context_line":"          # Enable tls-proxy"},{"line_number":61,"context_line":"          OVERRIDE_ENABLED_SERVICES+\u003d,tls-proxy"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"          # Enable mandatory placement services for nova starting with ocata"},{"line_number":64,"context_line":"          if [[ \"stable/newton\" !\u003d $ZUUL_BRANCH ]]; then"},{"line_number":65,"context_line":"               OVERRIDE_ENABLED_SERVICES+\u003d,placement-api,placement-client"},{"line_number":66,"context_line":"          fi"},{"line_number":67,"context_line":"          export OVERRIDE_ENABLED_SERVICES"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5fc1f717_563d068c","line":66,"range":{"start_line":63,"start_character":10,"end_line":66,"end_character":12},"in_reply_to":"5fc1f717_9584c85f","updated":"2019-03-25 21:44:59.000000000","message":"True.. wanted to just get rid of the condition..","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"cc894eaba2737a3891995e9d7af648b7acbe044a","unresolved":false,"context_lines":[{"line_number":60,"context_line":"          # Enable tls-proxy"},{"line_number":61,"context_line":"          OVERRIDE_ENABLED_SERVICES+\u003d,tls-proxy"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"          # Enable mandatory placement services for nova starting with ocata"},{"line_number":64,"context_line":"          if [[ \"stable/newton\" !\u003d $ZUUL_BRANCH ]]; then"},{"line_number":65,"context_line":"               OVERRIDE_ENABLED_SERVICES+\u003d,placement-api,placement-client"},{"line_number":66,"context_line":"          fi"},{"line_number":67,"context_line":"          export OVERRIDE_ENABLED_SERVICES"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5fc1f717_9584c85f","line":66,"range":{"start_line":63,"start_character":10,"end_line":66,"end_character":12},"in_reply_to":"5fc1f717_aaee3f1f","updated":"2019-03-25 19:14:07.000000000","message":"I\u0027ll remove the condition.  We have to keep the placement services though if we have the n-services, right?","commit_id":"a42ce07dd34bd1f2195c14b0cfee73acdf4a5678"}]}