)]}'
{"id":"x%2Fpackstack~760644","triplet_id":"x%2Fpackstack~master~I81464c3267190b3d21b99796ac42f30b6bc86cb7","project":"x/packstack","branch":"master","hashtags":[],"change_id":"I81464c3267190b3d21b99796ac42f30b6bc86cb7","subject":"Use magnum::keystone::keystone_auth for magnum config","status":"MERGED","created":"2020-10-30 17:45:58.000000000","updated":"2020-11-03 12:31:25.000000000","submitted":"2020-11-03 12:31:25.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":"760644-1604406685895-294064b8","meta_rev_id":"a5fe3675b5f102d55dd6f476c9d5794b1a1d6f3d","_number":760644,"virtual_id_number":760644,"owner":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":16339,"name":"Cyril Lopez","email":"cylopez@redhat.com","username":"cylopez"},{"value":0,"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-11-03 12:31:25.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"date":"2020-11-03 09:29:41.000000000","_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"}],"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":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"recommended":{"_account_id":16339,"name":"Cyril Lopez","email":"cylopez@redhat.com","username":"cylopez"},"all":[{"value":1,"date":"2020-10-30 19:30:09.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":16339,"name":"Cyril Lopez","email":"cylopez@redhat.com","username":"cylopez"},{"value":2,"date":"2020-11-03 11:28:21.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":2,"date":"2020-11-03 11:17:30.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"}],"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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"all":[{"value":0,"_account_id":16339,"name":"Cyril Lopez","email":"cylopez@redhat.com","username":"cylopez"},{"value":1,"date":"2020-11-03 11:28:21.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},{"_account_id":16339,"name":"Cyril Lopez","email":"cylopez@redhat.com","username":"cylopez"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-10-30 19:30:09.000000000","updated_by":{"_account_id":16339,"name":"Cyril Lopez","email":"cylopez@redhat.com","username":"cylopez"},"reviewer":{"_account_id":16339,"name":"Cyril Lopez","email":"cylopez@redhat.com","username":"cylopez"},"state":"REVIEWER"},{"updated":"2020-11-03 11:17:30.000000000","updated_by":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"reviewer":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"state":"REVIEWER"},{"updated":"2020-11-03 11:28:21.000000000","updated_by":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"reviewer":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"state":"REVIEWER"},{"updated":"2020-11-03 12:31:25.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":"f8884b2ce0e56460a4259bc7f54fae3baa17e41a","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2020-10-30 17:45:58.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"64f115d4841f1ee6caaa9a7b121603ac1a0e3afd","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-10-30 18:45:43.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2d0cbbda7a5440f5b2cd06f36011027a : SUCCESS in 3m 09s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/202f586a23ef494380f208c74024dcdf : SUCCESS in 3m 19s\n- packstack-centos8-integration-scenario001 https://zuul.opendev.org/t/openstack/build/5676ebe161ca450cb4b7b9bffd68c9dc : SUCCESS in 49m 48s\n- packstack-centos8-integration-scenario002 https://zuul.opendev.org/t/openstack/build/1ea0778e3a774ab6a7d8009a3bfacdbd : SUCCESS in 51m 16s\n- packstack-centos8-integration-scenario003 https://zuul.opendev.org/t/openstack/build/8c7e6f0981c14d8f8725d28dbb435f40 : FAILURE in 55m 09s\n- packstack-centos8-multinode-scenario002-tempest https://zuul.opendev.org/t/openstack/build/457419966b014c6d955fe4f1232ba3a0 : SUCCESS in 44m 23s","accounts_in_message":[],"_revision_number":1},{"id":"aab4de1115fbbdfb499e91092f7dfc866723f706","author":{"_account_id":16339,"name":"Cyril Lopez","email":"cylopez@redhat.com","username":"cylopez"},"date":"2020-10-30 19:30:09.000000000","message":"Patch Set 1: Code-Review+1\n\nwork for me","accounts_in_message":[],"_revision_number":1},{"id":"5d2cff5e8fab1721baf3c4818b0cc528a1a97bc1","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2020-11-03 09:29:41.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"70a2ff79359455daaf6778b03eaa15a6e7529c07","author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"date":"2020-11-03 10:12:38.000000000","message":"Patch Set 1:\n\nI see keystone_auth section correct but there are some errors from magnum-conductor:\n\nOct 30 18:20:53 centos-8-vexxhost-ca-ymq-1-0021287870 magnum-conductor[76556]: 2020-10-30 18:20:53.141 76630 ERROR oslo.messaging._drivers.impl_rabbit [-] Connection failed: [Errno 111] ECONNREFUSED (retrying in 18.0 seconds): ConnectionRefusedError: [Errno 111] ECONNREFUSED\n\nhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_8c7/760644/1/check/packstack-centos8-integration-scenario003/8c7e6f0/logs/diag/journalctl_--no-pager.txt\n\nAre we missing config in oslo_messaging_rabbit section?","accounts_in_message":[],"_revision_number":1},{"id":"2851378651950a0dac362889fd19d97dffe894b1","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-03 10:39:25.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/575bb58624e246a4bf69332f163950ed : SUCCESS in 3m 40s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/4eaa2e3fef204bc0ab2e0355c67c1715 : SUCCESS in 4m 15s\n- packstack-centos8-integration-scenario001 https://zuul.opendev.org/t/openstack/build/da8e69523a6144d68ae5c67903f8754e : SUCCESS in 50m 36s\n- packstack-centos8-integration-scenario002 https://zuul.opendev.org/t/openstack/build/d205da4f7d9c4a2f9d21b639aba7265f : SUCCESS in 38m 53s\n- packstack-centos8-integration-scenario003 https://zuul.opendev.org/t/openstack/build/14eed8508a784521bbadd44f5bdd1e83 : SUCCESS in 51m 03s\n- packstack-centos8-multinode-scenario002-tempest https://zuul.opendev.org/t/openstack/build/507f6dbccd574d32828b9c2f70bc42b1 : SUCCESS in 51m 23s","accounts_in_message":[],"_revision_number":1},{"id":"78390f9403500be2d9d3c680bd825456f382591d","author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"date":"2020-11-03 11:17:30.000000000","message":"Patch Set 1: Code-Review+2\n\n\u003e I see keystone_auth section correct but there are some errors from\n \u003e magnum-conductor:\n \u003e \n \u003e Oct 30 18:20:53 centos-8-vexxhost-ca-ymq-1-0021287870\n \u003e magnum-conductor[76556]: 2020-10-30 18:20:53.141 76630 ERROR\n \u003e oslo.messaging._drivers.impl_rabbit [-] Connection failed: [Errno\n \u003e 111] ECONNREFUSED (retrying in 18.0 seconds): ConnectionRefusedError:\n \u003e [Errno 111] ECONNREFUSED\n \u003e \n \u003e https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_8c7/760644/1/check/packstack-centos8-integration-scenario003/8c7e6f0/logs/diag/journalctl_--no-pager.txt\n \u003e \n \u003e Are we missing config in oslo_messaging_rabbit section?\n\nForget my comment about the rabbitmq messages, seems previous to actual rabbitmq server startup.","accounts_in_message":[],"_revision_number":1},{"id":"0c925351e29714a53878eab0a1d7ce245fdc15ce","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2020-11-03 11:28:21.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"4d95ca846a12f1fb1aaa99219eb8be1bb6d139f4","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-03 11:28:32.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"564468cf5cb3f2d53f76e8244cb8f20741284aee","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-03 12:31:25.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/a38b0d711219447995a94f333505ee2b : SUCCESS in 4m 36s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/bae7cb23547742b69b26d18e91feec34 : SUCCESS in 3m 50s\n- packstack-centos8-integration-scenario001 https://zuul.opendev.org/t/openstack/build/5392145b976744ecacd8759545ddd9ec : SUCCESS in 54m 22s\n- packstack-centos8-integration-scenario002 https://zuul.opendev.org/t/openstack/build/905b4378552449a5b00083d07580de13 : SUCCESS in 45m 36s\n- packstack-centos8-integration-scenario003 https://zuul.opendev.org/t/openstack/build/77237a545dcb43f8bb427be5692f37c9 : SUCCESS in 44m 10s\n- packstack-centos8-multinode-scenario002-tempest https://zuul.opendev.org/t/openstack/build/6e9735084b354a10b63a3ef6da010574 : SUCCESS in 55m 43s","accounts_in_message":[],"_revision_number":1},{"id":"7177aaf2d32df81d40d9a878d8863a1aa7e99158","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-03 12:31:25.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"745214d0a7941038ff04775e63c339305464246b","revisions":{"745214d0a7941038ff04775e63c339305464246b":{"kind":"REWORK","_number":1,"created":"2020-10-30 17:45:58.000000000","uploader":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"ref":"refs/changes/44/760644/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/x/packstack","ref":"refs/changes/44/760644/1","commands":{"Checkout":"git fetch https://review.opendev.org/x/packstack refs/changes/44/760644/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/x/packstack refs/changes/44/760644/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/x/packstack refs/changes/44/760644/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/x/packstack refs/changes/44/760644/1"}}},"commit":{"parents":[{"commit":"4149f4dfcdab177671b17488059d5c445e4a6be2","subject":"Merge \"Move to cirros version 0.5.1 by default\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/4149f4dfcdab177671b17488059d5c445e4a6be2"}]}],"author":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2020-10-30 17:42:30.000000000","tz":60},"committer":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2020-10-30 17:42:30.000000000","tz":60},"subject":"Use magnum::keystone::keystone_auth for magnum config","message":"Use magnum::keystone::keystone_auth for magnum config\n\nPuppet-magnum removed the old fallback method of configuring\nthe service user with [1], and added a new one using keystone_auth\nwith [2]. This commit uses the new magnum::keystone::keystone_auth\nclass to fix the configuration.\n\nNote that the change started in stable/ussuri, so this patch should\nbe backported all the way back to that release.\n\nCloses-Bug: #1902252\n\n[1] - https://review.opendev.org/659774\n[2] - https://review.opendev.org/758981\n\nChange-Id: I81464c3267190b3d21b99796ac42f30b6bc86cb7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/745214d0a7941038ff04775e63c339305464246b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/745214d0a7941038ff04775e63c339305464246b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}