)]}'
{"id":"openstack%2Fnova~953121","triplet_id":"openstack%2Fnova~master~I440fff6663d0663fb1630dd096f352403982aa37","project":"openstack/nova","branch":"master","attention_set":{},"removed_from_attention_set":{"11604":{"account":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"last_update":"2025-06-27 11:47:08.000000000","reason":"\u003cGERRIT_ACCOUNT_11604\u003e replied on the change","reason_account":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"}},"4393":{"account":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"last_update":"2025-06-25 16:25:26.000000000","reason":"\u003cGERRIT_ACCOUNT_4393\u003e replied on the change","reason_account":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"}},"9708":{"account":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"last_update":"2025-06-27 16:42:15.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"I440fff6663d0663fb1630dd096f352403982aa37","subject":"Note on RPC error decorators around build_and_run_instance","status":"MERGED","created":"2025-06-23 14:52:45.000000000","updated":"2025-06-27 16:44:04.000000000","submitted":"2025-06-27 16:42:15.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":5,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"953121","meta_rev_id":"7a37d73707960b11a2b8319c11856f7f085f54be","_number":953121,"virtual_id_number":953121,"owner":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2025-06-27 16:42:15.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":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"all":[{"value":2,"date":"2025-06-27 11:47:08.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":2,"date":"2025-06-25 16:25:26.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"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":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"all":[{"value":1,"date":"2025-06-27 11:47:08.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"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},"Review-Priority":{"all":[{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-06-23 15:04:08.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2025-06-23 16:15:24.000000000","updated_by":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"reviewer":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"state":"CC"},{"updated":"2025-06-23 16:44: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":"CC"},{"updated":"2025-06-23 17:19:07.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"},{"updated":"2025-06-25 16:25:26.000000000","updated_by":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"reviewer":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"state":"REVIEWER"}],"messages":[{"id":"635b6bf6df5989c0711994353ae51aa6f14f1e04","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2025-06-23 14:52:45.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"782b0a98e21c6ca98c1fca7fdf9f1040a6f8e6f3","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2025-06-23 15:04:08.000000000","message":"Patch Set 1: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"985ad1288f229138ddb3bc3ffb09e21d57b313cd","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2025-06-23 16:15:24.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"01ef98d90c69e313eccc1185002fc5641c83f4b3","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-23 16:44:04.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/bb5fcf0cc6d44bf08ba8bc823f967cbf\n\n- openstack-tox-py310-arm64 https://zuul.opendev.org/t/openstack/build/c851a123211b40b5a269d5e4d205c986 : SUCCESS in 32m 44s (non-voting)\n- openstack-tox-py312-arm64 https://zuul.opendev.org/t/openstack/build/2f2bf99c97a74553bedd12ba3ed09035 : SUCCESS in 36m 47s (non-voting)\n- openstack-tox-py313-arm64 https://zuul.opendev.org/t/openstack/build/402b7d7369ae46ffbda740540f4ba3a0 : SUCCESS in 42m 49s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"31d5cc1a26a4c6784ad75b09c302e37ad6793f0c","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-23 17:19:07.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\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/c97ff1b70d4b4f4e9944b4cde2ed4a50\n\n- grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/de077cc90fe44bf9b803124452be968a : SUCCESS in 52m 19s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/776a71a91a3748fdb01d2cd426848673 : SUCCESS in 1h 36m 58s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/725444a628d847cf9e42f306d33b11c6 : SUCCESS in 1h 02m 33s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/bd4a36ad7d014751903504890620cfd3 : SUCCESS in 23m 15s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/220076c5ff2c40d3830c8a3e43e96f41 : FAILURE in 8m 33s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/28dec2d506f84e24bee97856e61af61d : SUCCESS in 11m 39s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/88c279ecf783442b83a89b824c5df66e : SUCCESS in 12m 56s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/721fe31e39164fe881510fe0ce506c8d : SUCCESS in 15m 43s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a720aac332d94073bc30038d5ea785cd : SUCCESS in 11m 47s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/72b0b6d1aec847cabd035de3a82e4e71 : SUCCESS in 1h 10m 16s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/5984583677e64e23a2a7571c4aba8032 : SUCCESS in 1h 43m 02s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/da7da26711ee470892b885e33a3e18c9 : SUCCESS in 46m 03s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/fe50241d1cf148da8b56fdcada1b51be : SUCCESS in 43m 23s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/bccea1cbcd604f9695f8e5e15de2f999 : FAILURE in 2h 00m 52s\n- nova-next https://zuul.opendev.org/t/openstack/build/a6fe5e1b88454757bf5b18d77aa32098 : SUCCESS in 2h 19m 17s\n- nova-ovs-hybrid-plug https://zuul.opendev.org/t/openstack/build/04b813fce1be496ca6c380be4bad09de : SUCCESS in 43m 42s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/3c45867f67a34353812672f586d621c4 : SUCCESS in 4m 42s (non-voting)\n- nova-tox-functional-py310 https://zuul.opendev.org/t/openstack/build/69093845e9394f7f995e1c3915440fa2 : SUCCESS in 27m 22s\n- nova-tox-functional-py312 https://zuul.opendev.org/t/openstack/build/9d7f8fa3c0ee4332bb2dfbaf51892069 : SUCCESS in 28m 53s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/cc0bfdfcf19c4495a8b89b81599b8abe : SUCCESS in 1h 31m 17s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/7f90f29538a24a2f9a6cf0489628d2df : SUCCESS in 54m 45s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/ee2b5e28d1f8494bb997f36677d1fa39 : SUCCESS in 24m 51s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/ba363b501b9544ada97a852b12c8ffd5 : SUCCESS in 24m 34s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"52b9251ef9951f1ad3c5f2b3aa7fc62cc06c2ec2","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2025-06-25 16:01:23.000000000","message":"Uploaded patch set 2.\n\nOutdated Votes:\n* Code-Review+1 (copy condition: \"changekind:TRIVIAL_REBASE OR is:MIN\")\n* Verified-1\n","accounts_in_message":[],"_revision_number":2},{"id":"63a05a2936d43126f44c905f6a03b48451c50678","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2025-06-25 16:01:32.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"40222b5ca56e4ea6d03e69d75e40743b8e66b0d8","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2025-06-25 16:25:26.000000000","message":"Patch Set 2: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"28874538560d35632f43e3dfc27ec2d96ec0f0ba","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-25 18:07:06.000000000","message":"Patch Set 2:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/a005843b74204ff78d7194d6ec9c98e5\n\n- openstack-tox-py310-arm64 https://zuul.opendev.org/t/openstack/build/e1681cbcf73346ebb524d7aa4168f2fc : SUCCESS in 37m 23s (non-voting)\n- openstack-tox-py312-arm64 https://zuul.opendev.org/t/openstack/build/e5a56628360f4ea99071593fe24fc318 : SUCCESS in 24m 27s (non-voting)\n- openstack-tox-py313-arm64 https://zuul.opendev.org/t/openstack/build/c62aefb6d9404fe993c887ca98afea8e : FAILURE in 44m 15s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"37d1bdc31637c1a0ba21806b0695678a386d6a63","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-25 18:20:38.000000000","message":"Patch Set 2: 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\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/52fe243663ca4d5b9c89381dde04f84c\n\n- grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/d0ee69c406864046a47a125ec8f344d2 : SUCCESS in 1h 06m 15s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/85cc0a1fcd4f41bd9f134ad53597cd37 : SUCCESS in 1h 00m 56s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/a88338048a674af7b90d3063bdb137f5 : SUCCESS in 56m 16s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/53df91d399de49a28c7f31028f382025 : SUCCESS in 24m 04s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/967979057f33497c99265160678eebb4 : SUCCESS in 8m 21s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/4992ed2fe00c43e7b86d6c1602c4370c : SUCCESS in 14m 48s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/7817532aea31463c8b57ba425e6bbbd4 : SUCCESS in 15m 56s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/df7ffd5ad9d34802b574975df49b3e96 : SUCCESS in 23m 22s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/55c3da7ae2b6419b91e3ca6758aabac7 : SUCCESS in 13m 44s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/d6545f9323b24661950f09b4d662928d : SUCCESS in 1h 31m 52s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/91aa2b89f7804bcfafee79a2106daeb8 : SUCCESS in 2h 10m 29s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/fd9721d732c742cc8a12cb8c14b77572 : POST_FAILURE in 9m 15s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/870b579ee140493ca1d8b2b1d06528bd : SUCCESS in 27m 15s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/86489ff234a14eeba2bb1b78c9675767 : FAILURE in 12m 21s\n- nova-next https://zuul.opendev.org/t/openstack/build/6d208a69ea394151bd63582e333dfa46 : SUCCESS in 1h 14m 35s\n- nova-ovs-hybrid-plug https://zuul.opendev.org/t/openstack/build/9f8eda0b2520484d92c04fd23392f4f5 : SUCCESS in 46m 59s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/b697aeebef0842c89241dc84b1263a5c : SUCCESS in 4m 45s (non-voting)\n- nova-tox-functional-py310 https://zuul.opendev.org/t/openstack/build/a2da509c3ae54bac944bb910f74ee2c2 : SUCCESS in 26m 03s\n- nova-tox-functional-py312 https://zuul.opendev.org/t/openstack/build/0d98ddd942d34013b8122d4ac385f706 : SUCCESS in 32m 42s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/b06ab97d4ef94eb79bb233ec2e8f6ba5 : SUCCESS in 1h 19m 40s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/2a933bb046ee4bc0bc36990e3f080130 : SUCCESS in 59m 14s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/ad1d0c520cd04796805e80926c00e66c : SUCCESS in 25m 08s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/10706bebc85e475991acf2d6fce9cddc : SUCCESS in 13m 51s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"c3a9ecb36b578b00853a1a87f298c2645393a567","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2025-06-27 07:16:53.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"7de3b2070cf071ff6f8d5e41cc98ac7b658ae7d2","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-27 08:28:38.000000000","message":"Patch Set 2:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/2f73ba7998bf49ac91d2bf32475b0927\n\n- openstack-tox-py310-arm64 https://zuul.opendev.org/t/openstack/build/3f00b244da24400885e0005324021ffa : SUCCESS in 21m 29s (non-voting)\n- openstack-tox-py312-arm64 https://zuul.opendev.org/t/openstack/build/c2155b517e014b0988f04c2f51227185 : SUCCESS in 20m 37s (non-voting)\n- openstack-tox-py313-arm64 https://zuul.opendev.org/t/openstack/build/f216ade43a6948618936675889e68f0f : SUCCESS in 43m 09s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"62c15656c2dbad6996718b5471ae5b53bdb380aa","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-27 09:55:10.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/8c0476bc9d4c4f758d8f51518e37fb5c\n\n- grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/5f1676909e784ee09347b722430bc9c3 : SUCCESS in 1h 02m 23s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/0bab1987fe344db597b3bdeb1f46174e : SUCCESS in 1h 46m 20s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/8f56d8cb35334b6d9552c19cc5602988 : SUCCESS in 58m 11s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/2205946fb6b84353b13a77ef649e3906 : SUCCESS in 20m 56s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/76484dad78cf41e38f9d07b9aa78e53c : SUCCESS in 7m 06s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/4d62c5f2850f44b58c4181673974aeb9 : SUCCESS in 8m 21s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/3c0a5e6de69b4045b97250af48f5b91f : SUCCESS in 13m 19s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/fc2cd431c76c481e832e5c0c5d952f79 : SUCCESS in 16m 22s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/53988e4463484bf1943ad8747eb637ae : SUCCESS in 10m 23s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/b6c6a250ecff41ac986475d657cdced4 : SUCCESS in 1h 12m 22s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/a9cb72a7199f44dfb81acba5972ef645 : SUCCESS in 1h 54m 04s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/e3c4f494bf8f4cb2a63f90db7fcca8db : SUCCESS in 25m 38s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/5155fc0e3e164b94a1ab457cc8d830f0 : SUCCESS in 52m 00s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/bec51586b9a646f58ad908b68e4a1fec : SUCCESS in 2h 12m 47s\n- nova-next https://zuul.opendev.org/t/openstack/build/18b3513ea486403490d71ca6f7c49f7e : SUCCESS in 2h 37m 02s\n- nova-ovs-hybrid-plug https://zuul.opendev.org/t/openstack/build/8127b6d24efa4228a008f0c31bd27888 : SUCCESS in 22m 30s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/f76cd423a3a54920a8a26aef9e108b6c : SUCCESS in 6m 20s (non-voting)\n- nova-tox-functional-py310 https://zuul.opendev.org/t/openstack/build/13115c697910439b995e0795077ac594 : SUCCESS in 28m 32s\n- nova-tox-functional-py312 https://zuul.opendev.org/t/openstack/build/e803226b93fa4540957152f2eced66b7 : SUCCESS in 31m 40s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/3dcb26ff6ed3446ba07c062a7f7d50dd : SUCCESS in 1h 19m 25s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/4f2f438a1a8648db9d064b6cd661ba2f : SUCCESS in 56m 53s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/3ab84cb6363146c6a735509ce27b2581 : SUCCESS in 26m 37s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/741dd68d0a7b4103a8024b67f03c565c : SUCCESS in 34m 16s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"5942affb2131bb6d2e94b4ab04c6434c97e47bc9","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2025-06-27 11:47:08.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"2d0f41e7fd3f20f686988ecf8cfc8690a8fb0d17","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-27 12:20:35.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"af6f4b3203e2e46f7fa5dd6410e4e0d1f93795c2","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-27 16:42:15.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/7227d0e2532d4d2992478357c6b74cb2\n\n- grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/5ebb94032aef49f4b276ab20e01d5253 : SUCCESS in 1h 04m 58s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/03c2f0d3540e4818b4da1c57f35c9ba4 : SUCCESS in 1h 52m 47s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/3cbdee5663cb47f2a8fd493f1eaa1786 : SUCCESS in 1h 03m 51s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/df4bd5169cec4bca8ed8dd0e588f1a60 : SUCCESS in 10m 31s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/386554214f5745bdb29117084912ab34 : SUCCESS in 12m 30s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/8a96e300e7f9457a85e05019fe1ca373 : SUCCESS in 14m 50s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/8003cde463884520be115749c8bb3061 : SUCCESS in 13m 39s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/1978cb83220944a2af55bfa15a8fae87 : SUCCESS in 29m 46s\n- nova-tox-functional-py310 https://zuul.opendev.org/t/openstack/build/9203741231d146ea92c631f36e1c9f89 : SUCCESS in 25m 14s\n- nova-tox-functional-py312 https://zuul.opendev.org/t/openstack/build/7e02f40880e9411c8c68420c675e6ba7 : SUCCESS in 31m 34s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/ee0241af0f1a4cae9e94a71357d4d873 : SUCCESS in 1h 14m 47s\n- nova-next https://zuul.opendev.org/t/openstack/build/908e2986141c486e8f63778b2e465692 : SUCCESS in 2h 10m 39s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/e120f7eb892d4cfe91112ab38da281a6 : SUCCESS in 4m 28s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/bc2f98fcda4e4bd3815ccc7b61e28776 : SUCCESS in 1h 42m 41s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/8ed8e6c1a5ea42edbc0703c5e5f28c1b : SUCCESS in 53m 03s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/40aa302ba00543a6a51a554ef35b692d : SUCCESS in 36m 46s","accounts_in_message":[],"_revision_number":2},{"id":"14711892bc501dd826d719d382ca2ec7dad19d82","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-27 16:42:15.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":2},{"id":"7a37d73707960b11a2b8319c11856f7f085f54be","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-27 16:44:04.000000000","message":"Patch Set 2:\n\nBuild succeeded (promote pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/4cf28ae1194e4a789a776a55701333e0\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f09527a4f6004abfb7dfe8da63d677b9 : SUCCESS in 49s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"1cd1c472bd8a2dafe4219c7b61b8d75a67f1bf16","revisions":{"a593b176d8bd70c6d0a618c0b331769b376ab838":{"kind":"REWORK","_number":1,"created":"2025-06-23 14:52:45.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/21/953121/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/21/953121/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/953121/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/953121/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/953121/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/21/953121/1"}}},"commit":{"parents":[{"commit":"41773f8c6515021eb037e6d9d385b34e89191c8c","subject":"Merge \"Fix disable memballoon device\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/41773f8c6515021eb037e6d9d385b34e89191c8c"}]}],"author":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2025-06-23 12:42:10.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2025-06-23 14:46:50.000000000","tz":120},"subject":"Note on RPC error decorators around build_and_run_instance","message":"Note on RPC error decorators around build_and_run_instance\n\nThe RPC handler build_and_run_instance has a normal set of error handler\ndecorators but that call immediately spawns a separate thread and does not\nwait for its result so it cannot get any exceptions. Therefore the error\nhandling should happen inside the thread. And it does. Mostly.\n\nThe _locked_do_build_and_run_instance is the function run by the thread\nand it calls _do_build_and_run_instance. The _do_build_and_run_instance\nhas `except Exception` handling dropping the active exception and returning\nan error value instead. So most of the exceptions are handled by\n_do_build_and_run_instance without going through any of the error\ndecorators.\n\nWhatever exception still bubbles up from _do_build_and_run_instance\nare going through the error handling decorators top of\n_do_build_and_run_instance.\n\nThe only source of exception outside of _do_build_and_run_instance but\nwithin _locked_do_build_and_run_instance is\ndelete_allocation_for_instance. Exception from that call does not\ngo though any error handling decorators.\n\nThese error decorators has multiple tasks:\n* wrap_exception calls _emit_legacy_exception_notification and\n  _emit_versioned_exception_notification so sending error\n  notifications\n* reverts_task_state sets instance.task_state to None\n* wrap_instance_fault records InstanceFault object to the DB\n\nSo there are two cases when error happens but the decorators are not\ncalled:\n* _do_build_and_run_instance catches the exception, does its own error\n  handling and cleanup and then returns an error value. The task state\n  revert and the InstanceFault creation is replicated to the individual\n  cleanup sections here.\n  Note that notification sending is not missing either as that is\n  handled by the _build_and_run_instance called from\n  _do_build_and_run_instance.\n\n* delete_allocation_for_instance raises when called from\n  _locked_do_build_and_run_instance. We reach this call if\n  _do_build_and_run_instance failed already and did its own cleanup. So\n  again task state are already reset and InstanceFault already recorded\n  for the original fault and notification already sent.\n\nSo the only changes here is to replace the error handling decorators on\nbuild_and_run_instance with a note that they are deeper in the stack so\nthat readers will not think error handling happen on top of the call\nstack.\n\nThere was two unit test cases that partially relied on the wrong\nassumption that exceptions propagate to build_and_run_instance.\nThey could make that wrong assumption because the\nSpawnIsSynchronousFixture simulates the GreenThread interface wrongly.\nThe goal of that fixture is to make any spawn (and spawn_n) call\nsynchronous meaning the function passed in is executed right away before\nspawn returns. That is OK. But the fixture forgot that if the function\nraises then such exception is now raised by spawn. Such a thing never\nhappens with a normal spawn. The exception (and the function return\nvalue) only propagated when GreenThread.wait() is called.\n\nSo this patch fixes the fixture and corrects the two unit tests as well.\n\nChange-Id: I440fff6663d0663fb1630dd096f352403982aa37\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a593b176d8bd70c6d0a618c0b331769b376ab838"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a593b176d8bd70c6d0a618c0b331769b376ab838"}]},"branch":"refs/heads/master"},"1cd1c472bd8a2dafe4219c7b61b8d75a67f1bf16":{"kind":"REWORK","_number":2,"created":"2025-06-25 16:01:23.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/21/953121/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/21/953121/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/953121/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/953121/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/953121/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/21/953121/2"}}},"commit":{"parents":[{"commit":"41773f8c6515021eb037e6d9d385b34e89191c8c","subject":"Merge \"Fix disable memballoon device\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/41773f8c6515021eb037e6d9d385b34e89191c8c"}]}],"author":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2025-06-23 12:42:10.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2025-06-25 15:59:02.000000000","tz":120},"subject":"Note on RPC error decorators around build_and_run_instance","message":"Note on RPC error decorators around build_and_run_instance\n\nThe RPC handler build_and_run_instance has a normal set of error handler\ndecorators but that call immediately spawns a separate thread and does not\nwait for its result so it cannot get any exceptions. Therefore the error\nhandling should happen inside the thread. And it does. Mostly.\n\nThe _locked_do_build_and_run_instance is the function run by the thread\nand it calls _do_build_and_run_instance. The _do_build_and_run_instance\nhas `except Exception` handling dropping the active exception and returning\nan error value instead. So most of the exceptions are handled by\n_do_build_and_run_instance without going through any of the error\ndecorators.\n\nWhatever exception still bubbles up from _do_build_and_run_instance\nare going through the error handling decorators top of\n_do_build_and_run_instance.\n\nThe only source of exception outside of _do_build_and_run_instance but\nwithin _locked_do_build_and_run_instance is\ndelete_allocation_for_instance. Exception from that call does not\ngo though any error handling decorators.\n\nThese error decorators has multiple tasks:\n* wrap_exception calls _emit_legacy_exception_notification and\n  _emit_versioned_exception_notification so sending error\n  notifications\n* reverts_task_state sets instance.task_state to None\n* wrap_instance_fault records InstanceFault object to the DB\n\nSo there are two cases when error happens but the decorators are not\ncalled:\n* _do_build_and_run_instance catches the exception, does its own error\n  handling and cleanup and then returns an error value. The task state\n  revert and the InstanceFault creation is replicated to the individual\n  cleanup sections here.\n  Note that notification sending is not missing either as that is\n  handled by the _build_and_run_instance called from\n  _do_build_and_run_instance.\n\n* delete_allocation_for_instance raises when called from\n  _locked_do_build_and_run_instance. We reach this call if\n  _do_build_and_run_instance failed already and did its own cleanup. So\n  again task state are already reset and InstanceFault already recorded\n  for the original fault and notification already sent.\n\nSo the only changes here is to add a note to build_and_run_instance so\nthat the real error handling are deeper in the stack so that readers will\nnot think error handling happen on top of the call stack.\n\nThere was two unit test cases that partially relied on the wrong\nassumption that exceptions propagate to build_and_run_instance.\nThey could make that wrong assumption because the\nSpawnIsSynchronousFixture simulates the GreenThread interface wrongly.\nThe goal of that fixture is to make any spawn (and spawn_n) call\nsynchronous meaning the function passed in is executed right away before\nspawn returns. That is OK. But the fixture forgot that if the function\nraises then such exception is now raised by spawn. Such a thing never\nhappens with a normal spawn. The exception (and the function return\nvalue) only propagated when GreenThread.wait() is called.\n\nSo this patch fixes the fixture and corrects the two unit tests as well.\n\nChange-Id: I440fff6663d0663fb1630dd096f352403982aa37\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1cd1c472bd8a2dafe4219c7b61b8d75a67f1bf16"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1cd1c472bd8a2dafe4219c7b61b8d75a67f1bf16"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"CLOSED","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"}},{"label":"Workflow","status":"MAY","applied_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"}},{"label":"Review-Priority","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dMAX"],"failing_atoms":["label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dMAX"],"failing_atoms":["label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Review-Priority","description":"Review Priority","status":"NOT_APPLICABLE","is_legacy":false,"applicability_expression_result":{"fulfilled":false,"status":"FAIL"},"submittability_expression_result":{"expression":"is:true","fulfilled":true,"status":"NOT_EVALUATED","passing_atoms":[],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dMAX"],"failing_atoms":["label:Workflow\u003dMIN"],"atom_explanations":{}}}]}