)]}'
{"id":"openstack%2Fopenstack-ansible-tests~571760","triplet_id":"openstack%2Fopenstack-ansible-tests~master~I1a6e887142e2144f483be79bb203ecc4423c2338","project":"openstack/openstack-ansible-tests","branch":"master","topic":"move-db-setup-into-roles","hashtags":[],"change_id":"I1a6e887142e2144f483be79bb203ecc4423c2338","subject":"Move database creation into role (ironic)","status":"MERGED","created":"2018-06-01 15:07:49.000000000","updated":"2018-06-05 05:09:36.000000000","submitted":"2018-06-05 05:09:36.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":"571760-1528175376947-5c9d8c9f","meta_rev_id":"67c609d5249a9a28876896234571951b1f938241","_number":571760,"virtual_id_number":571760,"owner":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},{"value":2,"date":"2018-06-05 05:09:36.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"}],"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":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"all":[{"value":2,"date":"2018-06-04 14:07:31.000000000","_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2018-06-01 21:13:21.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"}],"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":[{"value":0,"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2018-06-01 21:13:21.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"}],"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":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_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":"2018-06-01 21:13:21.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":"2018-06-04 14:07:31.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":"2018-06-05 05:09:36.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":"5e6ade10ced78e0ba5d79220ce522cbeea1b06f4","author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"date":"2018-06-01 15:07:49.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"03a0bc91a66ca9c618fd8de4c2ff1fb59c26af71","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-06-01 15:55:42.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/60/571760/1/check/build-openstack-sphinx-docs/56ca0a4/html/ : SUCCESS in 3m 14s\n- openstack-ansible-linters http://logs.openstack.org/60/571760/1/check/openstack-ansible-linters/fabb726/ : SUCCESS in 7m 52s\n- openstack-ansible-functional-centos-7 http://logs.openstack.org/60/571760/1/check/openstack-ansible-functional-centos-7/368cec0/ : SUCCESS in 46m 19s\n- openstack-ansible-functional-opensuse-423 http://logs.openstack.org/60/571760/1/check/openstack-ansible-functional-opensuse-423/da8449b/ : SUCCESS in 31m 21s\n- openstack-ansible-functional-ubuntu-bionic-nv http://logs.openstack.org/60/571760/1/check/openstack-ansible-functional-ubuntu-bionic-nv/09e0c8b/ : FAILURE in 18m 00s (non-voting)\n- openstack-ansible-functional-ubuntu-xenial http://logs.openstack.org/60/571760/1/check/openstack-ansible-functional-ubuntu-xenial/fc793e1/ : SUCCESS in 31m 05s","accounts_in_message":[],"_revision_number":1},{"id":"caf67cba69aedd5d4060300910a5cc692451b57d","author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"date":"2018-06-01 21:13:21.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"9bd145def0ff53321d9797d2da1ab3094a4e8176","author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"date":"2018-06-04 14:07:31.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"6e78e87ac10d81b8d6fbe97d1471f21ee2ee400a","author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"date":"2018-06-05 04:00:23.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"6f005b16e9f64ccde83f382bd7780535d2a3e6f0","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-06-05 04:00:36.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"3f352ebc29bd5620327acf9c595db47c75d03fa6","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-06-05 04:51:56.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/60/571760/1/check/build-openstack-sphinx-docs/4dfb030/html/ : SUCCESS in 2m 20s\n- openstack-ansible-linters http://logs.openstack.org/60/571760/1/check/openstack-ansible-linters/8f6ee05/ : SUCCESS in 9m 31s\n- openstack-ansible-functional-centos-7 http://logs.openstack.org/60/571760/1/check/openstack-ansible-functional-centos-7/e92aa7b/ : SUCCESS in 48m 00s\n- openstack-ansible-functional-opensuse-423 http://logs.openstack.org/60/571760/1/check/openstack-ansible-functional-opensuse-423/f0a9bb4/ : SUCCESS in 32m 36s\n- openstack-ansible-functional-ubuntu-bionic-nv http://logs.openstack.org/60/571760/1/check/openstack-ansible-functional-ubuntu-bionic-nv/0506441/ : FAILURE in 19m 39s (non-voting)\n- openstack-ansible-functional-ubuntu-xenial http://logs.openstack.org/60/571760/1/check/openstack-ansible-functional-ubuntu-xenial/edcb695/ : SUCCESS in 31m 24s","accounts_in_message":[],"_revision_number":1},{"id":"6dc2432513103bb63fb08bf7b68aeb861c68e9ab","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-06-05 05:09:36.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"99ceb55531ac8b16947d0abd5056d3853c4cc846","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-06-05 05:09:36.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/60/571760/1/gate/build-openstack-sphinx-docs/720474c/html/ : SUCCESS in 3m 20s\n- openstack-ansible-linters http://logs.openstack.org/60/571760/1/gate/openstack-ansible-linters/cb830ac/ : SUCCESS in 9m 56s\n- openstack-ansible-functional-centos-7 http://logs.openstack.org/60/571760/1/gate/openstack-ansible-functional-centos-7/fa6a2ec/ : SUCCESS in 47m 56s\n- openstack-ansible-functional-opensuse-423 http://logs.openstack.org/60/571760/1/gate/openstack-ansible-functional-opensuse-423/802b77f/ : SUCCESS in 34m 55s\n- openstack-ansible-functional-ubuntu-xenial http://logs.openstack.org/60/571760/1/gate/openstack-ansible-functional-ubuntu-xenial/673a35c/ : SUCCESS in 31m 24s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"9cbab7594da3f6f1aedba1de88a880708fa5d663","revisions":{"9cbab7594da3f6f1aedba1de88a880708fa5d663":{"kind":"REWORK","_number":1,"created":"2018-06-01 15:07:49.000000000","uploader":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"ref":"refs/changes/60/571760/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-tests","ref":"refs/changes/60/571760/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/60/571760/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/60/571760/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/60/571760/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/60/571760/1"}}},"commit":{"parents":[{"commit":"287c89058488776ee8130d8380716b3cf10f07a3","subject":"Merge \"test-ansible-deps: Bump SHA for Ansible 2.4\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/287c89058488776ee8130d8380716b3cf10f07a3"}]}],"author":{"name":"Jesse Pretorius","email":"jesse.pretorius@rackspace.co.uk","date":"2018-06-01 15:06:39.000000000","tz":60},"committer":{"name":"Jesse Pretorius","email":"jesse.pretorius@rackspace.co.uk","date":"2018-06-01 15:06:39.000000000","tz":60},"subject":"Move database creation into role (ironic)","message":"Move database creation into role (ironic)\n\nThere is no record for why we implement the database creation outside\nof the role in the playbook, when we could do it inside the role.\n\nImplementing it inside the role allows us to reduce the quantity of\ngroup_vars duplicated from the role, and allows us to better document\nthe required variables in the role. The delegation can still be done\nas it is done in the playbook too.\n\nIn this patch we remove the test-vars which were duplicated from the\nrole, and remove the DB setup tasks as they are no longer required.\n\nChange-Id: I1a6e887142e2144f483be79bb203ecc4423c2338\nDepends-On: https://review.openstack.org/571759\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/9cbab7594da3f6f1aedba1de88a880708fa5d663"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/9cbab7594da3f6f1aedba1de88a880708fa5d663"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}