)]}'
{"id":"openstack%2Fpuppet-nova~691121","triplet_id":"openstack%2Fpuppet-nova~stable%2Ftrain~If4d6a4a8f4cc1e7066cacb00da67d5433acd6ac5","project":"openstack/puppet-nova","branch":"stable/train","topic":"bug/1529065-stable/train","hashtags":[],"change_id":"If4d6a4a8f4cc1e7066cacb00da67d5433acd6ac5","subject":"Include nova::availability_zone in nova::api and nova::conductor","status":"MERGED","created":"2019-10-24 21:00:51.000000000","updated":"2019-10-26 08:49:34.000000000","submitted":"2019-10-25 19:20:04.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":"691121-1572031204234-5cb63f8a","meta_rev_id":"56d190c354e3239f5eb2243d925c9635a0d600ae","_number":691121,"virtual_id_number":691121,"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":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},{"value":0,"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2019-10-25 19:20:04.000000000","post_submit":true,"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":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"all":[{"value":2,"date":"2019-10-25 12:23:10.000000000","_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},{"value":2,"date":"2019-10-25 13:56:57.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":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"all":[{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"date":"2019-10-24 21:00:51.000000000","_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},{"value":1,"date":"2019-10-25 13:56:57.000000000","permitted_voting_range":{"min":1,"max":1},"_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":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"_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-24 21:00:51.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-25 12:23:10.000000000","updated_by":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"reviewer":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"state":"REVIEWER"},{"updated":"2019-10-25 13:56:57.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-25 19:20:04.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":"2bcfc76cddd8e44c62d992b4fbbd5fcc0344227c","author":{"_account_id":17216,"name":"Martin Schuppert","email":"mschuppert@redhat.com","username":"mcschupp"},"date":"2019-10-24 21:00:51.000000000","message":"Patch Set 1: Cherry Picked from branch master.","accounts_in_message":[],"_revision_number":1},{"id":"ac9d9dc84e119f6e769815cdf01e4cad66dc2af6","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-24 22:35:18.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/d8fc65f39acf471099cec06c66ef3cd8 : SUCCESS in 5m 29s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/14d00af4e5da45e4b561817dfe8f212c : SUCCESS in 4m 48s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/22d7fc26571948fab824743196667e96 : SUCCESS in 19m 52s\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/70fe8e764d68448da0f0d54d13c5f04a : SUCCESS in 56m 35s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/9d99fae8e5b640d2a0649afc13dbf12d : SUCCESS in 1h 00m 21s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/695b4cf5ce384f0080292bf4b90b848d : SUCCESS in 1h 16m 23s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/5975ee8013544e0e8fbb0ae3c687ec84 : SUCCESS in 46m 25s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/183287ae04d9426fafe2533c17e46ed3 : SUCCESS in 43m 13s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/5f27740dc6954285a2b714d388df3d10 : SUCCESS in 44m 55s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/10203cf530a645c3823fbb6547b81fb2 : SUCCESS in 47m 37s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/bd12463e18db4937b652c468608b70b8 : SUCCESS in 35m 18s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/8d0766497c0c44e8824d46a6f3a0f1d7 : SUCCESS in 27m 55s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/e8922af6b9ee40239942939ba7ba49e3 : FAILURE in 28m 01s (non-voting)\n- puppet-nova-tripleo-standalone https://zuul.opendev.org/t/openstack/build/e9dda3a5abf042379e034267c6c7f9ce : SUCCESS in 1h 30m 52s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"322c72cc48d1e1b38f5e145bcf5dfe7726484f5c","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2019-10-25 12:23:10.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"e4cf6546c013d71b60477be8b0b626f66e2ddb31","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2019-10-25 13:56:57.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"1abb74e456498769e3cc584435b04afc4bdc4c7c","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-25 13:57:09.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"a4c3a52a00a8a8ac37313be4d030d9a9d3fb8ead","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-25 15:34:43.000000000","message":"Patch Set 1: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/da266d9be4ab43848f119e56c393c177 : SUCCESS in 5m 15s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/249a7cced2b54072bedd09c2088f3ba7 : SUCCESS in 6m 00s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/d8e852c5ccc64a608e8aec41510572c2 : SUCCESS in 22m 39s\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/60d852aeae35428b887a0e4585d249a3 : SUCCESS in 55m 51s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/74e09111e23f4565a9d580116ffab7eb : FAILURE in 1h 05m 48s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/7b601104bc7348a08f185845514c579d : SUCCESS in 1h 06m 22s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/0a9669dd5d054e4b8c0d8e67c447539f : SUCCESS in 41m 11s\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/bf2bf25631104f7b8fefcdfb5f34d26a : SUCCESS in 35m 08s\n- puppet-nova-tripleo-standalone https://zuul.opendev.org/t/openstack/build/2248ea4d2e774ebabfc96cf1a2238e94 : SUCCESS in 1h 35m 22s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"2e85063945e21d0b128e482a7143185276ce81d8","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2019-10-25 15:44:51.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"551be78f142636dc941c6bf2ba31f195d4af02fe","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-25 17:50:11.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/5f25404cbf4649809637f6c4dfd955af : SUCCESS in 4m 14s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/fb792b4ec89d401eb85a0425476eeb76 : SUCCESS in 4m 32s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/5d1c8e2671814b61a31f20fe19539ab6 : SUCCESS in 18m 25s\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/2567e0c1b510418f9ce9387ff6a11966 : SUCCESS in 51m 58s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/b9e5df17b09a489cb5960cceeb242922 : SUCCESS in 1h 05m 31s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/0196fded590f46b6b412399961644f9c : SUCCESS in 59m 17s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/8accf1d2ca9b4b1185a0c4b14a8b19f4 : SUCCESS in 44m 00s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/bb4fd746075e4213837e36f34290abd9 : SUCCESS in 51m 25s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/c1b9d3f2bae9498598bbcf5f5a138c91 : SUCCESS in 46m 00s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/c4959692fa01478baedd7d8dde2e3e13 : SUCCESS in 37m 23s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/b58382631beb491eac9fcd39b5e45f38 : SUCCESS in 32m 06s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/d2057eafa3e74a448feec23684dac2f9 : SUCCESS in 23m 01s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/4a1ba21c109d486786174d2f72168eec : FAILURE in 20m 39s (non-voting)\n- puppet-nova-tripleo-standalone https://zuul.opendev.org/t/openstack/build/a31d4bccaeb14d2f8ee2dee89984c54e : SUCCESS in 1h 57m 56s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"6fdff898ffa45f173dcbe324d41ec2d184935cf7","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-25 17:50:23.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"fdea239c77021c92392718fdeba1db43c2de6819","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-25 19:20:04.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"a49a0054c5d3cb739a2523fbc397dfbbe2e64b20","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-25 19:20:04.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/f15a565add3746678ac23fec74101c0d : SUCCESS in 7m 16s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/7baa00bde9ef4135891add1ed7a7a7ac : SUCCESS in 6m 54s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/a7e732f8cd77457195ca9b456339bb4f : SUCCESS in 33m 55s\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/6e4a08ac4da74bffbd891300b8d15a2e : SUCCESS in 1h 07m 18s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/d5cec97f0914480cb328a09cb17ec560 : SUCCESS in 1h 13m 22s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/ec10d00912da42e0b5311f574df2470d : SUCCESS in 1h 03m 50s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/f7592b8dc8ff4764a03fb6884a6bc61a : SUCCESS in 44m 25s\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/a1ce4157cd2a4a95909f1f3d9f06594a : SUCCESS in 26m 10s\n- puppet-nova-tripleo-standalone https://zuul.opendev.org/t/openstack/build/6d3c165d44234bf3b17bd66c2bf6f0d0 : SUCCESS in 1h 28m 28s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"0795c4b69e2f9530dad37729da7abaf9ac8fd708","author":{"_account_id":17216,"name":"Martin Schuppert","email":"mschuppert@redhat.com","username":"mcschupp"},"date":"2019-10-26 08:49:34.000000000","message":"Patch Set 1: Cherry Picked\n\nThis patchset was cherry picked to branch stable/stein as commit 75d67bd6c8ff0fd147e1ac7516ede2bc60d960b2","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"57ff2a154c0e50da4a833ffabfef27a8d0db4fb6","revisions":{"57ff2a154c0e50da4a833ffabfef27a8d0db4fb6":{"kind":"REWORK","_number":1,"created":"2019-10-24 21:00:51.000000000","uploader":{"_account_id":17216,"name":"Martin Schuppert","email":"mschuppert@redhat.com","username":"mcschupp"},"ref":"refs/changes/21/691121/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-nova","ref":"refs/changes/21/691121/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-nova refs/changes/21/691121/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-nova refs/changes/21/691121/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-nova refs/changes/21/691121/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-nova refs/changes/21/691121/1"}}},"commit":{"parents":[{"commit":"6a0a609d57d5ff82b417ad823511154372095ef9","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/6a0a609d57d5ff82b417ad823511154372095ef9"}]}],"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-24 21:00:51.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","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-nova/commit/57ff2a154c0e50da4a833ffabfef27a8d0db4fb6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-nova/commit/57ff2a154c0e50da4a833ffabfef27a8d0db4fb6"}]},"branch":"refs/heads/stable/train"}},"requirements":[],"submit_records":[],"submit_requirements":[]}