)]}'
{"id":"openstack%2Fpuppet-nova~691676","triplet_id":"openstack%2Fpuppet-nova~stable%2Frocky~If4d6a4a8f4cc1e7066cacb00da67d5433acd6ac5","project":"openstack/puppet-nova","branch":"stable/rocky","topic":"bug/1529065-stable/train-stable/stein-stable/rocky","hashtags":[],"change_id":"If4d6a4a8f4cc1e7066cacb00da67d5433acd6ac5","subject":"Include nova::availability_zone in nova::api and nova::conductor","status":"MERGED","created":"2019-10-28 13:53:58.000000000","updated":"2019-10-29 08:52:39.000000000","submitted":"2019-10-29 08:52:39.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":"691676-1572339159368-b61b701d","meta_rev_id":"a481ef7192ba5017d659b7e57926dc75442a5268","_number":691676,"virtual_id_number":691676,"owner":{"_account_id":17216,"name":"Martin Schuppert","email":"mschuppert@redhat.com","username":"mcschupp"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},{"value":0,"_account_id":9414,"name":"zhongshengping","email":"chdzsp@163.com","username":"ZhongShengping"},{"value":0,"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2019-10-29 08:52:39.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":9414,"name":"zhongshengping","email":"chdzsp@163.com","username":"ZhongShengping"},"all":[{"value":0,"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},{"value":2,"date":"2019-10-29 07:25:28.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":9414,"name":"zhongshengping","email":"chdzsp@163.com","username":"ZhongShengping"},{"value":2,"date":"2019-10-28 18:44:43.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":9414,"name":"zhongshengping","email":"chdzsp@163.com","username":"ZhongShengping"},"all":[{"value":0,"date":"2019-10-28 13:53:58.000000000","_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},{"value":1,"date":"2019-10-29 07:25:28.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":9414,"name":"zhongshengping","email":"chdzsp@163.com","username":"ZhongShengping"},{"value":0,"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":9414,"name":"zhongshengping","email":"chdzsp@163.com","username":"ZhongShengping"},{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-10-28 13:53:58.000000000","updated_by":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"reviewer":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"state":"REVIEWER"},{"updated":"2019-10-28 18:44:43.000000000","updated_by":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"reviewer":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"state":"REVIEWER"},{"updated":"2019-10-29 07:25:28.000000000","updated_by":{"_account_id":9414,"name":"zhongshengping","email":"chdzsp@163.com","username":"ZhongShengping"},"reviewer":{"_account_id":9414,"name":"zhongshengping","email":"chdzsp@163.com","username":"ZhongShengping"},"state":"REVIEWER"},{"updated":"2019-10-29 08:52:39.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":"973ead1551acc4c3887fbc55a5d6299bf92b6cc6","author":{"_account_id":17216,"name":"Martin Schuppert","email":"mschuppert@redhat.com","username":"mcschupp"},"date":"2019-10-28 13:53:58.000000000","message":"Patch Set 1: Cherry Picked from branch stable/stein.","accounts_in_message":[],"_revision_number":1},{"id":"862770979a4e259f3aea1dc3f70f2992cd07aea5","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-28 16:02:16.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- puppet-openstack-lint https://zuul.opendev.org/t/openstack/build/12fef8509ede4ad9859d20fd8a7e8938 : SUCCESS in 4m 15s\n- puppet-openstack-syntax-4 https://zuul.opendev.org/t/openstack/build/ebc854cc335d4a7cbe16e0c2dea61876 : SUCCESS in 4m 21s\n- puppet-openstack-syntax-5 https://zuul.opendev.org/t/openstack/build/2bd1fb89d9ae412a8db4e750f98afde0 : SUCCESS in 4m 15s\n- puppet-openstack-unit-4.8-centos-7 https://zuul.opendev.org/t/openstack/build/582c6ca953984bbd9a4c82b851d057af : SUCCESS in 35m 10s\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/b45089b061634982a95e0ac3393c7aa0 : SUCCESS in 51m 27s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/f91551b6c010460fa03db40faa656b4b : SUCCESS in 1h 14m 07s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/7ff46eb52ce84d1bb51e9630d935b824 : SUCCESS in 1h 08m 04s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-mimic https://zuul.opendev.org/t/openstack/build/d7b751c9bfaa4e4b8167d1bad4e01158 : SUCCESS in 42m 34s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/7f1e61cf6c8340da8aa98ba134025b9d : TIMED_OUT in 1h 31m 12s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/bd20772e19854799bfe2588ff68f980a : TIMED_OUT in 1h 31m 41s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/cf1eafacae954340acec3914356daf2a : TIMED_OUT in 1h 31m 22s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/d617333344184e7584cf95eee2d85ac6 : TIMED_OUT in 1h 31m 19s (non-voting)\n- puppet-openstack-integration-4-scenario001-tempest-debian-stable-luminous https://zuul.opendev.org/t/openstack/build/614fc9d6d48145ca8c73dc1cfad71df9 : TIMED_OUT in 1h 30m 49s (non-voting)\n- puppet-openstack-integration-4-scenario002-tempest-debian-stable https://zuul.opendev.org/t/openstack/build/8064df654f214473a1baef6ffa090a93 : FAILURE in 49m 51s (non-voting)\n- puppet-openstack-integration-4-scenario003-tempest-debian-stable https://zuul.opendev.org/t/openstack/build/9821b5afaa36481180eeddf8828c81af : TIMED_OUT in 1h 31m 18s (non-voting)\n- puppet-openstack-integration-4-scenario004-tempest-debian-stable-luminous https://zuul.opendev.org/t/openstack/build/5e9474e958954071ac40578f422119da : TIMED_OUT in 1h 31m 26s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/cc91c9470a194e4782e7e8566e8a686b : SUCCESS in 25m 20s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/23ceac53c0b04ab0bfca1236434977c2 : SUCCESS in 18m 06s\n- tripleo-puppet-ci-centos-7-undercloud-oooq https://zuul.opendev.org/t/openstack/build/81dd825be0eb43648f911f34218213c9 : SUCCESS in 1h 13m 00s\n- tripleo-puppet-ci-centos-7-undercloud-containers https://zuul.opendev.org/t/openstack/build/c5b836571ecc44bb8d6deaea855abe69 : SUCCESS in 2h 03m 21s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"52d1c2c59e76d19b03fd584f1b0b4d3c6f36f0f1","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2019-10-28 18:44:43.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"0ae6fe40410627a5c96ccc1c70cd346f413d17cf","author":{"_account_id":9414,"name":"zhongshengping","email":"chdzsp@163.com","username":"ZhongShengping"},"date":"2019-10-29 07:25:28.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"075b5f50e14209fc109a9d72838bb50ba99dd3ea","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-29 07:25:39.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"f0bc43d4ea7afee672a51962d501e5d14636ac47","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-29 08:52:39.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- puppet-openstack-lint https://zuul.opendev.org/t/openstack/build/672e0d6928844446ba89deafd39eda67 : SUCCESS in 5m 41s\n- puppet-openstack-syntax-4 https://zuul.opendev.org/t/openstack/build/bcf6d1fca2a54be0a206bc80213f5213 : SUCCESS in 5m 35s\n- puppet-openstack-syntax-5 https://zuul.opendev.org/t/openstack/build/70d0a9f074cc40d29cad8cffbd5840db : SUCCESS in 5m 47s\n- puppet-openstack-unit-4.8-centos-7 https://zuul.opendev.org/t/openstack/build/63011bbbf12b4082a23626907ccfa79b : SUCCESS in 28m 55s\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/48ab941497d74a6eacad2b43710052b0 : SUCCESS in 59m 02s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/57b2c2c3449a46d4ad7db818f567f6d2 : SUCCESS in 1h 03m 54s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/22661e4b5d60479e85c4715c4f50a89a : SUCCESS in 1h 02m 07s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-mimic https://zuul.opendev.org/t/openstack/build/66f9eeb3b824413d920b71cd9094615e : SUCCESS in 46m 49s\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/5aa151bd7e954797845fdd46df51d5bf : SUCCESS in 20m 37s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/cb31fe1ad88b4b1ab0a022dbd863d5a2 : SUCCESS in 18m 40s\n- tripleo-puppet-ci-centos-7-undercloud-oooq https://zuul.opendev.org/t/openstack/build/8d7069e002604d66880bbf6286324643 : SUCCESS in 1h 11m 06s\n- tripleo-puppet-ci-centos-7-undercloud-containers https://zuul.opendev.org/t/openstack/build/f4281b6c752643f697164c79abbce22b : SUCCESS in 1h 26m 00s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"b95c3183f29f351dfd44542feb86b9e254496598","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-29 08:52:39.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"effda25202c9270b843e113c12edef5b80130559","revisions":{"effda25202c9270b843e113c12edef5b80130559":{"kind":"REWORK","_number":1,"created":"2019-10-28 13:53:58.000000000","uploader":{"_account_id":17216,"name":"Martin Schuppert","email":"mschuppert@redhat.com","username":"mcschupp"},"ref":"refs/changes/76/691676/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-nova","ref":"refs/changes/76/691676/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-nova refs/changes/76/691676/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-nova refs/changes/76/691676/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-nova refs/changes/76/691676/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-nova refs/changes/76/691676/1"}}},"commit":{"parents":[{"commit":"9079de85fc9fccf1fdb35682ca080b1a5e7e92f4","subject":"Don\u0027t restart libvirt-guests on config change","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-nova/commit/9079de85fc9fccf1fdb35682ca080b1a5e7e92f4"}]}],"author":{"name":"Oliver Walsh","email":"owalsh@redhat.com","date":"2019-10-17 20:43:43.000000000","tz":60},"committer":{"name":"Martin Schuppert","email":"mschuppert@redhat.com","date":"2019-10-28 13:53:58.000000000","tz":0},"subject":"Include nova::availability_zone in nova::api and nova::conductor","message":"Include nova::availability_zone in nova::api and nova::conductor\n\nFor some reason this was included in nova::compute and nova::scheduler\nbut never included in nova::api eventhough bug #1529065 specifically\ncalls out the fact that it is missing.\n\nLooking at the code - nova-api definitely requires it.\nE.g the server create request handler sets the target AZ to\nCONF.default_schedule_zone if none\nhttps://github.com/openstack/nova/blob/stable/queens/nova/api/openstack/compute/servers.py#L486\n\nand the az request handler skips the internal_service_availabily_zone\nwhen listing AZs\nhttps://github.com/openstack/nova/blob/stable/queens/nova/api/openstack/compute/availability_zone.py#L38\n\nnova-conductor also requires it as it calls\navailability_zones.get_host_availability_zone which defaults to\nCONF.default_availability_zone\n\nChange-Id: If4d6a4a8f4cc1e7066cacb00da67d5433acd6ac5\nCloses-bug: #1529065\n(cherry picked from commit d7eb253e4de03c05eb6597db0959e4137831ad91)\n(cherry picked from commit 57ff2a154c0e50da4a833ffabfef27a8d0db4fb6)\n(cherry picked from commit 75d67bd6c8ff0fd147e1ac7516ede2bc60d960b2)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-nova/commit/effda25202c9270b843e113c12edef5b80130559"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-nova/commit/effda25202c9270b843e113c12edef5b80130559"}]},"branch":"refs/heads/stable/rocky"}},"requirements":[],"submit_records":[],"submit_requirements":[]}