)]}'
{"id":"openstack%2Fopenstack-ansible~200721","triplet_id":"openstack%2Fopenstack-ansible~master~Icc592e4ed1d2419cd305142d41e1c35b6dad9d82","project":"openstack/openstack-ansible","branch":"master","topic":"bug/1473548","hashtags":[],"change_id":"Icc592e4ed1d2419cd305142d41e1c35b6dad9d82","subject":"Adding galera_root_password token during upgrade Closes-Bug: #1473548","status":"ABANDONED","created":"2015-07-10 20:58:35.000000000","updated":"2015-07-13 22:02:48.000000000","total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"56b5802e0990ae2151972c1fc2f9cb1d4b9834d9","_number":200721,"virtual_id_number":200721,"owner":{"_account_id":14552,"name":"Bjoern Teipel","email":"bjoern.teipel@rackspace.com","username":"bjoern"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":-1,"date":"2015-07-13 20:35:23.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":12952,"name":"ice yao","email":"yao3690093@gmail.com","username":"xiabing.yao"},{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},{"_account_id":7614,"name":"Jacob Wagner","email":"swagner1104@gmail.com","username":"jwagner"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"all":[{"value":-1,"date":"2015-07-13 16:00:46.000000000","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":1,"date":"2015-07-13 16:08:58.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":12952,"name":"ice yao","email":"yao3690093@gmail.com","username":"xiabing.yao"},{"value":2,"date":"2015-07-13 16:00:46.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"value":-1,"date":"2015-07-13 16:05:37.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},{"value":-1,"date":"2015-07-13 16:00:46.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7614,"name":"Jacob Wagner","email":"swagner1104@gmail.com","username":"jwagner"}],"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":{"all":[{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":12952,"name":"ice yao","email":"yao3690093@gmail.com","username":"xiabing.yao"},{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},{"_account_id":7614,"name":"Jacob Wagner","email":"swagner1104@gmail.com","username":"jwagner"}],"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":12952,"name":"ice yao","email":"yao3690093@gmail.com","username":"xiabing.yao"},{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},{"_account_id":7614,"name":"Jacob Wagner","email":"swagner1104@gmail.com","username":"jwagner"}],"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":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":7614,"name":"Jacob Wagner","email":"swagner1104@gmail.com","username":"jwagner"},{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},{"_account_id":12952,"name":"ice yao","email":"yao3690093@gmail.com","username":"xiabing.yao"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-07-11 02:51:41.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":"2015-07-13 15:43:56.000000000","updated_by":{"_account_id":7614,"name":"Jacob Wagner","email":"swagner1104@gmail.com","username":"jwagner"},"reviewer":{"_account_id":7614,"name":"Jacob Wagner","email":"swagner1104@gmail.com","username":"jwagner"},"state":"REVIEWER"},{"updated":"2015-07-13 16:00:36.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":"2015-07-13 16:05:37.000000000","updated_by":{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},"reviewer":{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},"state":"REVIEWER"},{"updated":"2015-07-13 16:08:58.000000000","updated_by":{"_account_id":12952,"name":"ice yao","email":"yao3690093@gmail.com","username":"xiabing.yao"},"reviewer":{"_account_id":12952,"name":"ice yao","email":"yao3690093@gmail.com","username":"xiabing.yao"},"state":"REVIEWER"},{"updated":"2015-07-13 20:35:23.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"393e61ab37e601c18b7ba7af7e1b5630306f3d30","author":{"_account_id":14552,"name":"Bjoern Teipel","email":"bjoern.teipel@rackspace.com","username":"bjoern"},"date":"2015-07-10 20:58:35.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"9b20b3ecf90476519067a1f2576875e09e7d04ba","author":{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},"date":"2015-07-10 22:58:59.000000000","message":"Patch Set 1: Code-Review+1\n\n(1 comment)\n\nChange looks good, but since we don\u0027t have any gating actually running against run-upgrades.sh I want to try it before +2ing, especially since that file is seeing a bunch of churn over the last few days.","accounts_in_message":[],"_revision_number":1},{"id":"bed93d751da050d59db6404ad952db0158c15241","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-10 23:53:45.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- os-ansible-deployment-check-lint http://logs.openstack.org/21/200721/1/check/os-ansible-deployment-check-lint/0b63afa/ : SUCCESS in 26s\n- os-ansible-deployment-dsvm-check-commit http://logs.openstack.org/21/200721/1/check/os-ansible-deployment-dsvm-check-commit/d72b750/ : SUCCESS in 52m 21s","accounts_in_message":[],"_revision_number":1},{"id":"c3f48b0ef3d25d263556162efb804458140d0101","author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"date":"2015-07-11 02:51:41.000000000","message":"Patch Set 1: Code-Review+2\n\n(1 comment)\n\nLgtm - upgrade seems to work as expected.","accounts_in_message":[],"_revision_number":1},{"id":"004d4ab23eec6fb9a5d3a87a7dae55211c72f147","author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"date":"2015-07-13 12:45:04.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"2a672a7f73df56630e2c65668505f0646af98328","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-13 12:45:09.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"7699db031828e5a12bdc8c23c0cdc27640921b83","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-13 13:37:19.000000000","message":"Patch Set 1: 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- os-ansible-deployment-dsvm-check-commit http://logs.openstack.org/21/200721/1/gate/os-ansible-deployment-dsvm-check-commit/2300599/ : FAILURE in 1m 44s","accounts_in_message":[],"_revision_number":1},{"id":"4b29c7ff349fa04c2909d840f0b979ab1572d8fc","author":{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},"date":"2015-07-13 13:55:41.000000000","message":"Patch Set 1:\n\nrecheck - mongo server issue. Don\u0027t think this was submitted/rebased in time for the gating fix.","accounts_in_message":[],"_revision_number":1},{"id":"384b630cd46871ac2dc93547bfcf0712929d06d5","author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"date":"2015-07-13 13:59:19.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"e3ae7793d7783fee57c83c7161aef369e6117c99","author":{"_account_id":7614,"name":"Jacob Wagner","email":"swagner1104@gmail.com","username":"jwagner"},"date":"2015-07-13 15:43:56.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)\n\nI believe this variable should exists inside the user_secrets.yml file not the user_variables.yml\n\nhttps://github.com/stackforge/os-ansible-deployment/blob/kilo/etc/openstack_deploy/user_secrets.yml#L28","accounts_in_message":[],"_revision_number":1},{"id":"9c6daf9c9a4ec7281f1fbe32fdd56228dd598a6e","author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"date":"2015-07-13 16:00:36.000000000","message":"Patch Set 1: Code-Review-1 -Workflow\n\nJacob is right - pwgen should be run against user_secrets.","accounts_in_message":[],"_revision_number":1},{"id":"bf631d1ab725aace31f65f70d1a12dab810f23f1","author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"date":"2015-07-13 16:00:46.000000000","message":"Patch Set 2: Patch Set 1 was rebased","accounts_in_message":[],"_revision_number":1},{"id":"4abdd2d71205ce3268102da56bf8cd6567fc3d25","author":{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},"date":"2015-07-13 16:05:37.000000000","message":"Patch Set 2: Code-Review-1\n\nI just double checked - it looks like the value should be moved already by this line:\n\nhttps://github.com/stackforge/os-ansible-deployment/blob/kilo/scripts/run-upgrade.sh#L211\n\nAs Jacob said, this line should be in user_secrets.yml (thanks for the catch!), so this patch isn\u0027t necessary, unless we\u0027ve seen actual failures.","accounts_in_message":[],"_revision_number":2},{"id":"a4de3dda7f6d6686ba5a39ac0647aa23b30185e3","author":{"_account_id":12952,"name":"ice yao","email":"yao3690093@gmail.com","username":"xiabing.yao"},"date":"2015-07-13 16:08:58.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"9f9d9dd9bbef14822167f39eb40de5c489314709","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-13 20:35:23.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- os-ansible-deployment-check-lint http://logs.openstack.org/21/200721/2/check/os-ansible-deployment-check-lint/c82fc51/ : SUCCESS in 42s\n- os-ansible-deployment-dsvm-check-commit http://logs.openstack.org/21/200721/2/check/os-ansible-deployment-dsvm-check-commit/93bacc9/ : FAILURE in 22m 56s","accounts_in_message":[],"_revision_number":2},{"id":"19a00af6090e7a024be1ecef9f3ab455a8095990","author":{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},"date":"2015-07-13 22:02:48.000000000","message":"Abandoned\n\nAbandoning since reporter closed bug as invalid, and the galera_root_password value is actually written to user_secrets.yml, not user_variables.yml.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"9728eb600e2770d92b1efe8ddfcd1ef5a5dff8a8","revisions":{"1b9149f107cbdfba30f81f8f89f0807e1f9a3da4":{"kind":"REWORK","_number":1,"created":"2015-07-10 20:58:35.000000000","uploader":{"_account_id":14552,"name":"Bjoern Teipel","email":"bjoern.teipel@rackspace.com","username":"bjoern"},"ref":"refs/changes/21/200721/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible","ref":"refs/changes/21/200721/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/21/200721/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/21/200721/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/21/200721/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible refs/changes/21/200721/1"}}},"commit":{"parents":[{"commit":"f5388b61e4c41c071f3935066b1150f1a6cffe01","subject":"Merge \"Enable all services to use Keystone \u0027insecurely\u0027\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/f5388b61e4c41c071f3935066b1150f1a6cffe01"}]}],"author":{"name":"Bjoern Teipel","email":"bjoern.teipel@rackspace.com","date":"2015-07-10 20:57:46.000000000","tz":-300},"committer":{"name":"Bjoern Teipel","email":"bjoern.teipel@rackspace.com","date":"2015-07-10 20:57:46.000000000","tz":-300},"subject":"Adding galera_root_password token during upgrade Closes-Bug: #1473548","message":"Adding galera_root_password token during upgrade\nCloses-Bug: #1473548\n\nChange-Id: Icc592e4ed1d2419cd305142d41e1c35b6dad9d82\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/1b9149f107cbdfba30f81f8f89f0807e1f9a3da4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/1b9149f107cbdfba30f81f8f89f0807e1f9a3da4"}]},"branch":"refs/heads/master"},"9728eb600e2770d92b1efe8ddfcd1ef5a5dff8a8":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2015-07-13 16:00:46.000000000","uploader":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"ref":"refs/changes/21/200721/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible","ref":"refs/changes/21/200721/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/21/200721/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/21/200721/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/21/200721/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible refs/changes/21/200721/2"}}},"commit":{"parents":[{"commit":"8cb06d4412f3320f7129d18a141cae6f0bfd1959","subject":"Merge \"Wait until mongo responds after restart\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/8cb06d4412f3320f7129d18a141cae6f0bfd1959"}]}],"author":{"name":"Bjoern Teipel","email":"bjoern.teipel@rackspace.com","date":"2015-07-10 20:57:46.000000000","tz":-300},"committer":{"name":"Jesse Pretorius","email":"jesse.pretorius@rackspace.co.uk","date":"2015-07-13 16:00:46.000000000","tz":0},"subject":"Adding galera_root_password token during upgrade Closes-Bug: #1473548","message":"Adding galera_root_password token during upgrade\nCloses-Bug: #1473548\n\nChange-Id: Icc592e4ed1d2419cd305142d41e1c35b6dad9d82\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/9728eb600e2770d92b1efe8ddfcd1ef5a5dff8a8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/9728eb600e2770d92b1efe8ddfcd1ef5a5dff8a8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}