)]}'
{"id":"openstack%2Fopenstack-ansible-plugins~527415","triplet_id":"openstack%2Fopenstack-ansible-plugins~master~If91214dd2d2a36da82e5433a3a4a36c3946cbf4e","project":"openstack/openstack-ansible-plugins","branch":"master","hashtags":[],"change_id":"If91214dd2d2a36da82e5433a3a4a36c3946cbf4e","subject":"Remove the vars plugin as it\u0027s no longer used","status":"MERGED","created":"2017-12-12 14:36:18.000000000","updated":"2017-12-13 14:13:10.000000000","submitted":"2017-12-13 14:13:10.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"527415-1513174390458-78624289","meta_rev_id":"4a1da633ecae1c0caa640f0fc84ed6acf18626e8","_number":527415,"virtual_id_number":527415,"owner":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":0,"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},{"value":0,"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"value":0,"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},{"value":2,"date":"2017-12-13 14:13:10.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"}],"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":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"all":[{"value":0,"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":2,"date":"2017-12-13 04:15:53.000000000","_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},{"value":0,"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"value":2,"date":"2017-12-13 13:47:40.000000000","_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"}],"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":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"all":[{"value":0,"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":0,"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},{"value":0,"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"value":1,"date":"2017-12-13 13:47:40.000000000","_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":0,"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},{"value":0,"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"value":0,"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"}],"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":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-12-12 14:37:19.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"},{"updated":"2017-12-12 15:24:09.000000000","updated_by":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"reviewer":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"state":"REVIEWER"},{"updated":"2017-12-13 04:15:53.000000000","updated_by":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"reviewer":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"state":"REVIEWER"},{"updated":"2017-12-13 13:47:40.000000000","updated_by":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"reviewer":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"state":"REVIEWER"},{"updated":"2017-12-13 14:13:10.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"313a5f805fa44073ff0c9f5535ac209361077617","author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"date":"2017-12-12 14:36:18.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b7713245a1aa6c037bd193df3b896b57d4077bd8","author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"date":"2017-12-12 14:37:19.000000000","message":"Patch Set 1: Code-Review-1\n\nIs that true, even before https://review.openstack.org/527200 merges? Also, there should be a reno for this.","accounts_in_message":[],"_revision_number":1},{"id":"a0680c9f161f74347675ebd95aaf1befff55ecdf","author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"date":"2017-12-12 14:41:16.000000000","message":"Patch Set 1:\n\n\u003e Is that true, even before https://review.openstack.org/527200\n \u003e merges? Also, there should be a reno for this.\n\nI believe this plugin is only used in the integrated repo and for that it is on ansible 2.4 since this merged -\n https://review.openstack.org/#/c/522778/\n\nI will make a reno.","accounts_in_message":[],"_revision_number":1},{"id":"dcc6c710623372994256a84ff77f2a2fd170c13d","author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"date":"2017-12-12 15:24:09.000000000","message":"Patch Set 1: Code-Review-1\n\n\u003e \u003e Is that true, even before https://review.openstack.org/527200\n \u003e \u003e merges? Also, there should be a reno for this.\n \u003e \n \u003e I believe this plugin is only used in the integrated repo and for\n \u003e that it is on ansible 2.4 since this merged -\n \u003e https://review.openstack.org/#/c/522778/\n \u003e \n \u003e I will make a reno.\n\nThe functionality doesn\u0027t really change. But I like the idea of having a reno for telling we now configure differently, using ANSIBLE_INVENTORY env var.","accounts_in_message":[],"_revision_number":1},{"id":"08934370d46132fa985a4723586da638d28e14e0","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-12-13 01:19:35.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- build-openstack-sphinx-docs http://logs.openstack.org/15/527415/1/check/build-openstack-sphinx-docs/94ce2c6/html/ : SUCCESS in 2m 53s\n- openstack-ansible-linters http://logs.openstack.org/15/527415/1/check/openstack-ansible-linters/0f43bf2/ : SUCCESS in 4m 19s\n- openstack-ansible-functional-centos-7 http://logs.openstack.org/15/527415/1/check/openstack-ansible-functional-centos-7/f60bd75/ : SUCCESS in 23m 39s\n- openstack-ansible-functional-opensuse-423 http://logs.openstack.org/15/527415/1/check/openstack-ansible-functional-opensuse-423/f43b76b/ : SUCCESS in 10m 35s\n- openstack-ansible-functional-ubuntu-xenial http://logs.openstack.org/15/527415/1/check/openstack-ansible-functional-ubuntu-xenial/4d98ad7/ : TIMED_OUT in 4h 33m 56s\n- openstack-ansible-python3-ubuntu-xenial-nv http://logs.openstack.org/15/527415/1/check/openstack-ansible-python3-ubuntu-xenial-nv/01c79e2/ : FAILURE in 10m 13s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"14c310063764bff2f0c88a690245bda6991f5b96","author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"date":"2017-12-13 04:09:51.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"f77856aa92937bfb6dd2766df8351c3445cc8a01","author":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"date":"2017-12-13 04:15:53.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"cfac79c243c3e3b570c855620c2b45981e50b744","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-12-13 04:43:50.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/15/527415/2/check/build-openstack-sphinx-docs/6235273/html/ : SUCCESS in 2m 06s\n- build-openstack-releasenotes http://logs.openstack.org/15/527415/2/check/build-openstack-releasenotes/cbc4eba/html/ : SUCCESS in 3m 18s\n- openstack-ansible-linters http://logs.openstack.org/15/527415/2/check/openstack-ansible-linters/c1c0de2/ : SUCCESS in 4m 50s\n- openstack-ansible-functional-centos-7 http://logs.openstack.org/15/527415/2/check/openstack-ansible-functional-centos-7/4d71496/ : SUCCESS in 23m 13s\n- openstack-ansible-functional-opensuse-423 http://logs.openstack.org/15/527415/2/check/openstack-ansible-functional-opensuse-423/4006ff3/ : SUCCESS in 15m 18s\n- openstack-ansible-functional-ubuntu-xenial http://logs.openstack.org/15/527415/2/check/openstack-ansible-functional-ubuntu-xenial/0f9c4ad/ : SUCCESS in 10m 52s\n- openstack-ansible-python3-ubuntu-xenial-nv http://logs.openstack.org/15/527415/2/check/openstack-ansible-python3-ubuntu-xenial-nv/ebb01db/ : FAILURE in 11m 27s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"a5410a86a1bf916ffb5887dd52881003a252229c","author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"date":"2017-12-13 13:47:40.000000000","message":"Patch Set 2: Workflow+1 Code-Review+2\n\nDeleting code feels good.","accounts_in_message":[],"_revision_number":2},{"id":"14f25ba49ee6df7a0260836b766bd1c173704fc2","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-12-13 13:47:51.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"515038b52887d9bfca6ab500d6d3c25b3fb5c053","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-12-13 14:13:10.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2},{"id":"4a1da633ecae1c0caa640f0fc84ed6acf18626e8","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-12-13 14:13:10.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/15/527415/2/gate/build-openstack-sphinx-docs/aa3c6c8/html/ : SUCCESS in 3m 26s\n- build-openstack-releasenotes http://logs.openstack.org/15/527415/2/gate/build-openstack-releasenotes/6d5862a/html/ : SUCCESS in 2m 06s\n- openstack-ansible-linters http://logs.openstack.org/15/527415/2/gate/openstack-ansible-linters/64a1bfd/ : SUCCESS in 4m 14s\n- openstack-ansible-functional-centos-7 http://logs.openstack.org/15/527415/2/gate/openstack-ansible-functional-centos-7/821d83e/ : SUCCESS in 23m 34s\n- openstack-ansible-functional-opensuse-423 http://logs.openstack.org/15/527415/2/gate/openstack-ansible-functional-opensuse-423/ee5bdb8/ : SUCCESS in 13m 38s\n- openstack-ansible-functional-ubuntu-xenial http://logs.openstack.org/15/527415/2/gate/openstack-ansible-functional-ubuntu-xenial/73f7339/ : SUCCESS in 10m 37s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"c665f9f2a2dea2ac532402d41b68b6132fd774bc","revisions":{"e33eb357c73fcc5ba045e3ae1857ace43a984d22":{"kind":"REWORK","_number":1,"created":"2017-12-12 14:36:18.000000000","uploader":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"ref":"refs/changes/15/527415/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-plugins","ref":"refs/changes/15/527415/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/15/527415/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/15/527415/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/15/527415/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/15/527415/1"}}},"commit":{"parents":[{"commit":"d2f60237761646968a4b39b15185fb5c84e7386f","subject":"Merge \"Allow the experimental trigger of the integrated repo\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/d2f60237761646968a4b39b15185fb5c84e7386f"}]}],"author":{"name":"Kevin Carter","email":"kevin.carter@rackspace.com","date":"2017-12-12 14:31:00.000000000","tz":-360},"committer":{"name":"Kevin Carter","email":"kevin.carter@rackspace.com","date":"2017-12-12 14:34:28.000000000","tz":-360},"subject":"Remove the vars plugin as it\u0027s no longer used","message":"Remove the vars plugin as it\u0027s no longer used\n\nThis plugin is no longer used and conflicts with Ansible 2.4. Master\nhas recently moved to Ansible 2.4 and placed all of the group_vars back\ninto the inventory path. Being that the plugin is no longer required\nand results in a runtime warning[0] it\u0027s being removed.\n\n[0] - http://paste.openstack.org/show/628674/\n\nChange-Id: If91214dd2d2a36da82e5433a3a4a36c3946cbf4e\nSigned-off-by: Kevin Carter \u003ckevin.carter@rackspace.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/e33eb357c73fcc5ba045e3ae1857ace43a984d22"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/e33eb357c73fcc5ba045e3ae1857ace43a984d22"}]},"branch":"refs/heads/master"},"c665f9f2a2dea2ac532402d41b68b6132fd774bc":{"kind":"REWORK","_number":2,"created":"2017-12-13 04:09:51.000000000","uploader":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"ref":"refs/changes/15/527415/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-plugins","ref":"refs/changes/15/527415/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/15/527415/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/15/527415/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/15/527415/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/15/527415/2"}}},"commit":{"parents":[{"commit":"d2f60237761646968a4b39b15185fb5c84e7386f","subject":"Merge \"Allow the experimental trigger of the integrated repo\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/d2f60237761646968a4b39b15185fb5c84e7386f"}]}],"author":{"name":"Kevin Carter","email":"kevin.carter@rackspace.com","date":"2017-12-12 14:31:00.000000000","tz":-360},"committer":{"name":"Kevin Carter","email":"kevin.carter@rackspace.com","date":"2017-12-13 04:09:29.000000000","tz":-360},"subject":"Remove the vars plugin as it\u0027s no longer used","message":"Remove the vars plugin as it\u0027s no longer used\n\nThis plugin is no longer used and conflicts with Ansible 2.4. Master\nhas recently moved to Ansible 2.4 and placed all of the group_vars back\ninto the inventory path. Being that the plugin is no longer required\nand results in a runtime warning[0] it\u0027s being removed.\n\n[0] - http://paste.openstack.org/show/628674/\n\nChange-Id: If91214dd2d2a36da82e5433a3a4a36c3946cbf4e\nSigned-off-by: Kevin Carter \u003ckevin.carter@rackspace.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/c665f9f2a2dea2ac532402d41b68b6132fd774bc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/c665f9f2a2dea2ac532402d41b68b6132fd774bc"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}