)]}'
{"id":"openstack%2Fopenstack-ansible-os_cinder~290947","triplet_id":"openstack%2Fopenstack-ansible-os_cinder~master~I5810e12806ca5ab5ca8ba7167e87f371aeb7f951","project":"openstack/openstack-ansible-os_cinder","branch":"master","topic":"test_vars_cleanup","hashtags":[],"change_id":"I5810e12806ca5ab5ca8ba7167e87f371aeb7f951","subject":"Remove default vars from the test playbook","status":"ABANDONED","created":"2016-03-10 02:09:47.000000000","updated":"2016-03-15 18:07:14.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"0932d1c8e5cd6d6ab5977d6e16b3692f868ef923","_number":290947,"virtual_id_number":290947,"owner":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"actions":{},"labels":{"Verified":{"rejected":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"date":"2016-03-15 17:35:53.000000000","_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":-2,"date":"2016-03-15 15:50:00.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2016-03-15 14:07:38.000000000","_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},{"value":2,"date":"2016-03-15 15:39:15.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"value":2,"date":"2016-03-15 14:24:13.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"all":[{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},{"value":1,"date":"2016-03-15 15:39:15.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"}],"values":{"-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-03-15 14:24:13.000000000","updated_by":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"reviewer":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"state":"REVIEWER"},{"updated":"2016-03-15 15:39:15.000000000","updated_by":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"reviewer":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"state":"REVIEWER"},{"updated":"2016-03-15 15:50:00.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2016-03-15 17:35:53.000000000","updated_by":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"reviewer":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"state":"REVIEWER"}],"messages":[{"id":"2293486b65fa16d92c6ae984d08043cf65167892","author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"date":"2016-03-10 02:09:47.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"15bd9db8449a31445e258fc0faa68a4b97a91013","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-03-10 02:34:54.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-openstack-ansible-os_cinder-docs http://docs-draft.openstack.org/47/290947/1/check/gate-openstack-ansible-os_cinder-docs/0cef2d7//doc/build/html/ : SUCCESS in 0s\n- gate-openstack-ansible-os_cinder-linters http://logs.openstack.org/47/290947/1/check/gate-openstack-ansible-os_cinder-linters/30624ee/ : SUCCESS in 22s\n- gate-openstack-ansible-os_cinder-ansible-func-ubuntu-trusty http://logs.openstack.org/47/290947/1/check/gate-openstack-ansible-os_cinder-ansible-func-ubuntu-trusty/7e084d3/ : SUCCESS in 16m 20s\n- gate-openstack-ansible-os_cinder-dsvm-ansible-functional-centos7-nv http://logs.openstack.org/47/290947/1/check/gate-openstack-ansible-os_cinder-dsvm-ansible-functional-centos7-nv/e4428fc/ : FAILURE in 1m 08s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"631c79716705808bfe6a312b737d66f2e4138b04","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-03-11 19:43:13.000000000","message":"Patch Set 1: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"d64672dc02b5fd40d82ff27d785906b50d5bf058","author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"date":"2016-03-11 19:48:49.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"6ecf91b37b09ba574d7928cc1527f6751f81d2e8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-03-11 20:06:20.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-openstack-ansible-os_cinder-docs http://docs-draft.openstack.org/47/290947/2/check/gate-openstack-ansible-os_cinder-docs/aa696da//doc/build/html/ : SUCCESS in 37s\n- gate-openstack-ansible-os_cinder-linters http://logs.openstack.org/47/290947/2/check/gate-openstack-ansible-os_cinder-linters/151f1fa/ : SUCCESS in 14s\n- gate-openstack-ansible-os_cinder-ansible-func-ubuntu-trusty http://logs.openstack.org/47/290947/2/check/gate-openstack-ansible-os_cinder-ansible-func-ubuntu-trusty/7992a50/ : SUCCESS in 15m 16s\n- gate-openstack-ansible-os_cinder-dsvm-ansible-functional-centos7-nv http://logs.openstack.org/47/290947/2/check/gate-openstack-ansible-os_cinder-dsvm-ansible-functional-centos7-nv/5efbd5f/ : FAILURE in 1m 09s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"f3ee5bd3724f56dbf43d7caa61573fe80b3c7d7d","author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"date":"2016-03-15 09:34:05.000000000","message":"Patch Set 2: Code-Review-1\n\nHi Travis, aren\u0027t these vars required to properly wire cinder up to galera/rabbitmq?  They\u0027re not needed for provisioning the galera/rabbit bits but if we ever want to do any further functional testing then this will need to be present so that cinder is fully functional.","accounts_in_message":[],"_revision_number":2},{"id":"11e61721e01397e909031b76e16276ce168b4b3d","author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"date":"2016-03-15 14:07:38.000000000","message":"Patch Set 2:\n\nThey are already defined by the role included in the playbook, therefore, defining them again in the playbook\u0027s vars with the same values as those in defaults/main.yml has no effect.","accounts_in_message":[],"_revision_number":2},{"id":"1f1ae74cd0a20a4c32eb8486f0574e670dcfba3b","author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"date":"2016-03-15 14:24:13.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"768ecf07943b32d05183eaf3869e7d689cc5a08d","author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"date":"2016-03-15 15:39:15.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"f227d753366001658d34f915c84c4ee394b449d5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-03-15 15:39:24.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"b0fecdbea93fa32f72eabf16130a6960e4f3ba4a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-03-15 15:50:00.000000000","message":"Patch Set 2: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-openstack-ansible-os_cinder-docs http://docs-draft.openstack.org/47/290947/2/gate/gate-openstack-ansible-os_cinder-docs/7e58da6//doc/build/html/ : SUCCESS in 26s\n- gate-openstack-ansible-os_cinder-linters http://logs.openstack.org/47/290947/2/gate/gate-openstack-ansible-os_cinder-linters/2f83cd3/ : FAILURE in 37s\n- gate-openstack-ansible-os_cinder-ansible-func-ubuntu-trusty http://logs.openstack.org/47/290947/2/gate/gate-openstack-ansible-os_cinder-ansible-func-ubuntu-trusty/4dc79fa/ : SUCCESS in 10m 09s","accounts_in_message":[],"_revision_number":2},{"id":"6e6e58e0ec04f5b632229b62af88f15a6f0d54fe","author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"date":"2016-03-15 17:35:53.000000000","message":"Patch Set 2:\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"7caf4650bf4a01bfa425f9bc57ed366105f366d0","author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"date":"2016-03-15 18:07:14.000000000","message":"Abandoned\n\nin favor of https://review.openstack.org/#/c/288103/","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"97768c9d6d79d982f9c0afbff9f768dac9fa3a1f","revisions":{"aafadd31c852a844aae2a7ca6a4d217e42d6ca13":{"kind":"REWORK","_number":1,"created":"2016-03-10 02:09:47.000000000","uploader":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"ref":"refs/changes/47/290947/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-os_cinder","ref":"refs/changes/47/290947/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_cinder refs/changes/47/290947/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_cinder refs/changes/47/290947/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_cinder refs/changes/47/290947/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-os_cinder refs/changes/47/290947/1"}}},"commit":{"parents":[{"commit":"e9c5b96a5ef3be6848c3ed63197a585f5539f176","subject":"Account for oslo_config deprecation in cinder.conf template","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_cinder/commit/e9c5b96a5ef3be6848c3ed63197a585f5539f176"}]}],"author":{"name":"Travis Truman","email":"travis_truman@cable.comcast.com","date":"2016-03-10 02:09:08.000000000","tz":-300},"committer":{"name":"Travis Truman","email":"travis_truman@cable.comcast.com","date":"2016-03-10 02:09:08.000000000","tz":-300},"subject":"Remove default vars from the test playbook","message":"Remove default vars from the test playbook\n\nNo need to redefine them in the test\n\nChange-Id: I5810e12806ca5ab5ca8ba7167e87f371aeb7f951\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_cinder/commit/aafadd31c852a844aae2a7ca6a4d217e42d6ca13"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_cinder/commit/aafadd31c852a844aae2a7ca6a4d217e42d6ca13"}]},"branch":"refs/heads/master"},"97768c9d6d79d982f9c0afbff9f768dac9fa3a1f":{"kind":"REWORK","_number":2,"created":"2016-03-11 19:48:49.000000000","uploader":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"ref":"refs/changes/47/290947/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-os_cinder","ref":"refs/changes/47/290947/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_cinder refs/changes/47/290947/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_cinder refs/changes/47/290947/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_cinder refs/changes/47/290947/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-os_cinder refs/changes/47/290947/2"}}},"commit":{"parents":[{"commit":"883a359079669c3c0a50ed2c9f96f25dd752ac3f","subject":"Prevent IP conflicts on lxcbr0","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_cinder/commit/883a359079669c3c0a50ed2c9f96f25dd752ac3f"}]}],"author":{"name":"Travis Truman","email":"travis_truman@cable.comcast.com","date":"2016-03-10 02:09:08.000000000","tz":-300},"committer":{"name":"Travis Truman","email":"travis_truman@cable.comcast.com","date":"2016-03-11 19:48:43.000000000","tz":-300},"subject":"Remove default vars from the test playbook","message":"Remove default vars from the test playbook\n\nNo need to redefine them in the test\n\nChange-Id: I5810e12806ca5ab5ca8ba7167e87f371aeb7f951\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_cinder/commit/97768c9d6d79d982f9c0afbff9f768dac9fa3a1f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_cinder/commit/97768c9d6d79d982f9c0afbff9f768dac9fa3a1f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}