)]}'
{"id":"openstack%2Fnova~864935","triplet_id":"openstack%2Fnova~stable%2Fxena~Ic1352ec31996577a5d0ad18a057339df3e49de25","project":"openstack/nova","branch":"stable/xena","topic":"bug/1970467","attention_set":{},"removed_from_attention_set":{"11604":{"account":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"last_update":"2022-12-05 17:04:11.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"Ic1352ec31996577a5d0ad18a057339df3e49de25","subject":"Remove double mocking... again","status":"MERGED","created":"2022-11-17 19:08:21.000000000","updated":"2022-12-05 17:04:11.000000000","submitted":"2022-12-05 17:04:11.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":2,"unresolved_comment_count":1,"has_review_started":true,"submission_id":"864935-bug/1970467","meta_rev_id":"748245eca9efe3b7553ed3230f72b2283abd41a6","_number":864935,"virtual_id_number":864935,"owner":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2022-12-05 17:04:11.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"}],"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":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2022-12-01 18:14:33.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":2,"date":"2022-12-01 18:14:33.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"}],"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":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":1,"date":"2022-12-01 18:14:33.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"}],"REVIEWER":[{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},{"_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":"2022-11-17 19:08:21.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"state":"CC"},{"updated":"2022-11-17 19:26:36.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":"2022-11-17 19:31:48.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":"2022-11-24 17:44:12.000000000","updated_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"reviewer":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"state":"REVIEWER"},{"updated":"2022-12-01 15:43:06.000000000","updated_by":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"reviewer":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"state":"REVIEWER"},{"updated":"2022-12-01 18:14:33.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"state":"CC"}],"messages":[{"id":"964f8bca5d9a88175ac9d9e7fe7ed04e625e9738","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2022-11-17 19:08:21.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e239b9ea2a14e9eed67984da1e2609f151bf6d6c","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-11-17 19:26:36.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/997802dbf469484787bd88c870751f00\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/557ffef32d244992b603dbe11e6c2f99 : FAILURE in 11m 53s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/313454448cf74c508f323e40414288f9 : FAILURE in 12m 41s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"7391f996886244ee25ce48f480cb6436062799fb","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-11-17 19:31:48.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\nhttps://zuul.opendev.org/t/openstack/buildset/b65bb7b4631e4e03a9f2b1d3f6e2d8ce\n\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/2be24a2e6fe842b28250424a507e5f9d : FAILURE in 21m 50s (non-voting)\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/6dd4e32edb7f4e5eba1b2dede437b5aa : FAILURE in 5m 59s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e6c627c8e90c418aaede373e7a88f2a9 : SUCCESS in 7m 30s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/0269446ab5c54c41b5d51b2c5bc3dd04 : FAILURE in 5m 13s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/c532132ec78d4a2fb407d411cb6e1a61 : FAILURE in 7m 03s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/68779aafc31346cf9b9f94f5c37dc08f : FAILURE in 6m 25s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f8e79376084743daa799acae87d2c695 : SUCCESS in 8m 20s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/ad738131a14340269ff59c40f7305e5b : FAILURE in 5m 33s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/465bc8d53f73441485485ca12ac92339 : FAILURE in 10m 52s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/cfabab46ac2f4cd79ce8a21aaaede00d : FAILURE in 11m 24s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/86c78691be46453581ca6261480b413a : FAILURE in 11m 07s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"4e7b8967a19245cb5218f6074dfdf68d3277816d","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2022-11-18 11:44:15.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.","accounts_in_message":[],"_revision_number":2},{"id":"90e08543ec7a3f9ed17b9e2b0f6f52395e73b977","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-11-18 12:09:15.000000000","message":"Patch Set 2:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/ba1268f596274c5aa8c4c5a03014bb39\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/ec9ee1dbfedd462484af8f2e8c8a76d5 : FAILURE in 12m 34s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/933bc523428a4a84a415bfcbd3570242 : FAILURE in 14m 20s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"2be8451fc4677c016d66eb6051bd7f3fde1186df","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2022-11-18 12:12:05.000000000","message":"Uploaded patch set 3: New patch set was added with same tree, parent, and commit message as Patch Set 2.","accounts_in_message":[],"_revision_number":3},{"id":"d102b14ce9863982938a00e761a20c91fb9b32f7","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-11-18 12:33:20.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/f04427a1856a4ddfb9051d1e8d58aabf\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/1336c036e4c1444c873cb0eb10731f4b : FAILURE in 11m 16s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/03e74e8bdda64c02a899e1a5d741f1f0 : FAILURE in 12m 01s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"3bf5f23c6eb7f70376639eb49644e453e10e8295","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-11-18 12:35:54.000000000","message":"Patch Set 3: 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\nhttps://zuul.opendev.org/t/openstack/buildset/1a65f6f38aa345aab2e1300caf51b17b\n\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/2484a6559ee246c8ae12706ebe9c76f1 : FAILURE in 21m 58s (non-voting)\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/25dd377b455247cf8f334e50a8149d5d : FAILURE in 5m 31s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/3e2ceca897bc4735912bc55d6f30929b : FAILURE in 7m 22s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/152a2ed898b84785ab22af6d2b052c59 : FAILURE in 5m 26s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/b695d258357d402da7af0193a73eba93 : FAILURE in 6m 30s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/82e2734a357f498e8475b6fc27145803 : FAILURE in 5m 56s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/fc61566db53f47259ebf8b2ae24c4231 : SUCCESS in 9m 45s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/a2c68fa3ce8d4379bacaab2475c900e0 : FAILURE in 5m 06s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/0038a9e3ba484f998145e317915080af : FAILURE in 11m 53s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/485c9e0725604eb1b295adce82475c3b : FAILURE in 11m 41s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/4af34f5a30e64a5a9159ca72f7e7f88a : FAILURE in 12m 50s (non-voting)\n\nWarning:\n  Comments left for invalid file nova/tests/functional/libvirt/test_pci_sriov_servers.py","accounts_in_message":[],"_revision_number":3},{"id":"f36ec35f3549b71c2c6644339991acddae4d75b6","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2022-11-21 17:08:27.000000000","message":"Uploaded patch set 4: Patch Set 3 was rebased.","accounts_in_message":[],"_revision_number":4},{"id":"8063ae9950f200467e9f9a4b374ee6d423ae4fee","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-11-21 17:27:37.000000000","message":"Patch Set 4:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/468dbd3cae0849a8995dea47c9e3e088\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/336c57fb50b148e9a9b3f36342434bfe : FAILURE in 7m 32s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/51e7e8ed13334ba08b194e3feeb47025 : FAILURE in 9m 57s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"deafa2fce232ce3deae6de9e16a1214cb37d2dc6","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-11-21 17:33:49.000000000","message":"Patch Set 4: 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\nhttps://zuul.opendev.org/t/openstack/buildset/c6a4bd7a70b54b39896b86a45f45449d\n\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/9d9fc0c1d5d2463a9d4b39016398786b : FAILURE in 24m 16s (non-voting)\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/ccd73abd09b745d3bc0a6a87cdff583d : FAILURE in 6m 21s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/ec1dc672512b4a10b42421ce4160712d : SUCCESS in 8m 07s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/91159944d12b41c6a480ab9aad008c26 : FAILURE in 6m 02s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/8cfc3b92c7dd49f593e491ee12a3c26a : FAILURE in 5m 51s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/321eb9e852844793b482defcff8657b5 : FAILURE in 6m 23s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/7d41efc6f5034ec5928d52b982a24f9e : SUCCESS in 10m 03s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/3f591edd7efc49019672e8fd5b419744 : FAILURE in 5m 48s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/0f36cdc0b5224802a2efa1f711ace56c : FAILURE in 11m 52s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/c06ae8f6787c46579fb5c4801c3777b9 : FAILURE in 11m 11s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/bc761ea4292147868e307c0beac73c76 : FAILURE in 11m 37s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"dc2362d4b72375030ba3f4a3759362202dcde0a2","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2022-11-21 18:49:26.000000000","message":"Patch Set 4: Workflow-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"f3af8d524249d8f37cebd4412d21aebfd1e15a5e","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2022-11-22 11:54:07.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"699bfa846f7264db27d2147b58b240219648d665","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-11-22 12:31:56.000000000","message":"Patch Set 5:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/9bc331fddb70492cb505d93925c2cbbc\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/6ce6506c71d1428b85b370025dadac54 : SUCCESS in 26m 25s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/3fe3ed90069f4a8bb0c1852bf70f2a5d : SUCCESS in 29m 53s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"51da8326ba0d7b59a94a04af10cf0807151a3a1b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-11-22 12:37:06.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/cb6684180a9c40879b1f890f3bb8786a\n\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/f114d64829e84c1b9776f7a2fb89f9e4 : FAILURE in 36m 25s (non-voting)\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/1d38d7b2536b4491a7dc0244af83a826 : SUCCESS in 14m 25s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/77cae58aca7a4dde944ec7659d23e690 : SUCCESS in 7m 54s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/397227023d8d4b678998b193863ce740 : SUCCESS in 12m 09s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/3b3293744dad4414848201f5eb4e18ea : SUCCESS in 14m 06s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/d6330963251546f09e61770a0a2d568e : SUCCESS in 11m 44s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/64fd36d699924e949e65091b50597815 : SUCCESS in 9m 19s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/1ac64e1740df44fa9815e19e258d2ada : FAILURE in 5m 07s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/64273da5af4b4e24b132491727044bff : SUCCESS in 20m 44s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/da900368dcd34179ac57bbddb20ce0b9 : SUCCESS in 23m 42s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/384b0dc6a652453b8837a56bc251a755 : SUCCESS in 23m 21s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"242e39f26aeb2ed4fb25759eba39c68829205d35","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2022-11-24 17:44:12.000000000","message":"Patch Set 5: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"c50ea71455e93c64148cb4b7468545beee48e8c6","author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"date":"2022-12-01 15:43:06.000000000","message":"Patch Set 5: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":5},{"id":"bc3aa17459e887c470bdf40bea239772696e92e7","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2022-12-01 18:14:33.000000000","message":"Uploaded patch set 6: New patch set was added with same tree, parent, and commit message as Patch Set 5.","accounts_in_message":[],"_revision_number":6},{"id":"50c85420afd05f4d9c4fa65a8bd636a07243650a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-12-01 18:43:13.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/ee27e45c9d1a47f28a32c2073d08704e\n\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/db1e411d1a0a4ae1962e0ee933cf24d2 : FAILURE in 26m 11s (non-voting)\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/cedb4c45e03843dc9aee220ea922d9ee : SUCCESS in 15m 36s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/95a38a3c28d4490d9d5840011d2986d2 : SUCCESS in 9m 08s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/a4bece0bb37d400fb83d407cb6deac18 : SUCCESS in 12m 43s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/30a0752a556e45a89189d1d110bd23b0 : SUCCESS in 12m 56s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/368fc619186d4ba0a834e292a414251e : SUCCESS in 11m 47s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2b945548bf1040ebaca4e032c1f441cd : SUCCESS in 9m 33s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/a1913521d7fb44b199f37644be299c31 : SUCCESS in 6m 04s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/32d56bec1e414a6481163b2f71c43432 : SUCCESS in 19m 26s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/9d83a5fb42d04d41a00526b7487ca104 : SUCCESS in 19m 59s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/2ecc89ca17de45b6b3892a9ec6e61ef6 : SUCCESS in 20m 20s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"3c2e9305abca5cd7f237a81c0bea7f752f7e0357","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-12-01 19:18:47.000000000","message":"Patch Set 6:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/ba146efbfe2c4e7996ce68fc90fc111a\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/66b62a4da6b34d2a99b5a21422e769d9 : SUCCESS in 30m 10s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/80a12758625c4da5ba8dffd94de73c15 : SUCCESS in 26m 18s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"d2756705a713f8abf2f7fc39ea2bf4924f5ec571","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-12-05 15:25:24.000000000","message":"Patch Set 6: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":6},{"id":"5b7ee123ea1c2b2495747a6bd323e5cb26f0060e","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-12-05 17:04:11.000000000","message":"Patch Set 6: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/a4453fc6645442d7b110d4194dfdc289\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/dd163d0106d94aff8a649eeffcf1e52f : SUCCESS in 8m 00s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/5becdee7dc854f6d83fd72bc5a80dbd3 : SUCCESS in 10m 25s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/656fa98523ad4097ac968668c57bd514 : SUCCESS in 11m 23s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/0489af961baa4f3799c63f9ad5e3081b : SUCCESS in 9m 25s\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/136757dd1b2944e9949764e9c0d0f98a : SUCCESS in 22m 47s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/562b210007af49bbae75b16ef1e699f4 : SUCCESS in 22m 44s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/764e87edfb5a470497fef9d0b87631b5 : SUCCESS in 5m 40s","accounts_in_message":[],"_revision_number":6},{"id":"748245eca9efe3b7553ed3230f72b2283abd41a6","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-12-05 17:04:11.000000000","message":"Change has been successfully merged\n\n5 is the latest approved patch-set.\nNo files were changed between the latest approved patch-set and the submitted one.\n","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"b36bc9247ff195d983865860aef8e90bf49334e9","revisions":{"a4d33ed1e2aed68d9ac7856c0c3f56142fa9abe3":{"kind":"REWORK","_number":1,"created":"2022-11-17 19:08:21.000000000","uploader":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"ref":"refs/changes/35/864935/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/864935/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/864935/1"}}},"commit":{"parents":[{"commit":"2a9baccf83a6877b85a108ea5fc373feb82d5da8","subject":"Remove double mocking","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/2a9baccf83a6877b85a108ea5fc373feb82d5da8"}]}],"author":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2022-08-08 17:44:41.000000000","tz":120},"committer":{"name":"Sean Mooney","email":"work@seanmooney.info","date":"2022-11-17 19:05:06.000000000","tz":0},"subject":"Remove double mocking... again","message":"Remove double mocking... again\n\nI thought we fixed all the double mocking issues with\nI3998d0d49583806ac1c3ae64f1b1fe343cefd20d but I was wrong.\n\nWhile we used both mock and unittest.mock the fixtures.MockPatch\nused the mock lib instead of the unittest.mock lib.\nThe path Ibf4f36136f2c65adad64f75d665c00cf2de4b400 (Remove the PowerVM driver)\nremoved the last user of mock lib from nova. So it is also\nremoved the mock from test-requirements. This triggered that\nfixtures.MockPatch athat started using unittest.mock too.\n\nBefore Ibf4f36136f2c65adad64f75d665c00cf2de4b400 a function can be mocked\ntwice once with unittest.mock and once with fixtures.MockPatch (still\nusing mock). However after that patch both path of such double\nmocking goes through unittest.mock and the second one fails.\n\nSo this patch fixes double mocking so far hidden behind\nfixtures.MockPatch.\n\nThis patch made the py310 and functional-py310 jobs voting  on master\nhowever that has been dropped as part of the backport.\n\nConflicts:\n    nova/tests/fixtures/nova.py\n    nova/tests/unit/api/openstack/compute/test_quotas.py\n    nova/tests/unit/api/openstack/compute/test_server_group_quotas.py\n\nConflicts are due to lack of unified limits feature in xena\n\nChange-Id: Ic1352ec31996577a5d0ad18a057339df3e49de25\n(cherry picked from commit bf654e3a4a8f690ad0bec0955690bf4fadf98dba)\n(cherry picked from commit 69667a817cb65c3efbe4e3ada0e8c69c0a106087)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a4d33ed1e2aed68d9ac7856c0c3f56142fa9abe3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a4d33ed1e2aed68d9ac7856c0c3f56142fa9abe3"}]},"branch":"refs/heads/stable/xena"},"b639cb74b83e58b4f36eeeb961f6ace6d46f2aca":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2022-11-18 11:44:15.000000000","uploader":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"ref":"refs/changes/35/864935/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/864935/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/864935/2"}}},"commit":{"parents":[{"commit":"776d6fcd44d310a42c99e8bed195479d15f01806","subject":"Remove double mocking","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/776d6fcd44d310a42c99e8bed195479d15f01806"}]}],"author":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2022-08-08 17:44:41.000000000","tz":120},"committer":{"name":"Sean Mooney","email":"work@seanmooney.info","date":"2022-11-18 11:43:49.000000000","tz":0},"subject":"Remove double mocking... again","message":"Remove double mocking... again\n\nI thought we fixed all the double mocking issues with\nI3998d0d49583806ac1c3ae64f1b1fe343cefd20d but I was wrong.\n\nWhile we used both mock and unittest.mock the fixtures.MockPatch\nused the mock lib instead of the unittest.mock lib.\nThe path Ibf4f36136f2c65adad64f75d665c00cf2de4b400 (Remove the PowerVM driver)\nremoved the last user of mock lib from nova. So it is also\nremoved the mock from test-requirements. This triggered that\nfixtures.MockPatch athat started using unittest.mock too.\n\nBefore Ibf4f36136f2c65adad64f75d665c00cf2de4b400 a function can be mocked\ntwice once with unittest.mock and once with fixtures.MockPatch (still\nusing mock). However after that patch both path of such double\nmocking goes through unittest.mock and the second one fails.\n\nSo this patch fixes double mocking so far hidden behind\nfixtures.MockPatch.\n\nThis patch made the py310 and functional-py310 jobs voting  on master\nhowever that has been dropped as part of the backport.\n\nConflicts:\n    nova/tests/fixtures/nova.py\n    nova/tests/unit/api/openstack/compute/test_quotas.py\n    nova/tests/unit/api/openstack/compute/test_server_group_quotas.py\n\nConflicts are due to lack of unified limits feature in xena\n\nChange-Id: Ic1352ec31996577a5d0ad18a057339df3e49de25\n(cherry picked from commit bf654e3a4a8f690ad0bec0955690bf4fadf98dba)\n(cherry picked from commit 69667a817cb65c3efbe4e3ada0e8c69c0a106087)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b639cb74b83e58b4f36eeeb961f6ace6d46f2aca"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b639cb74b83e58b4f36eeeb961f6ace6d46f2aca"}]},"branch":"refs/heads/stable/xena"},"f6b3a5448fdd0f6bb02897f021fb9d916d8c21db":{"kind":"NO_CHANGE","_number":3,"created":"2022-11-18 12:12:05.000000000","uploader":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"ref":"refs/changes/35/864935/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/864935/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/864935/3"}}},"commit":{"parents":[{"commit":"1aead6dae22d5e9aa3229bc91b53d15846465b99","subject":"Remove double mocking","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1aead6dae22d5e9aa3229bc91b53d15846465b99"}]}],"author":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2022-08-08 17:44:41.000000000","tz":120},"committer":{"name":"Sean Mooney","email":"work@seanmooney.info","date":"2022-11-18 12:11:28.000000000","tz":0},"subject":"Remove double mocking... again","message":"Remove double mocking... again\n\nI thought we fixed all the double mocking issues with\nI3998d0d49583806ac1c3ae64f1b1fe343cefd20d but I was wrong.\n\nWhile we used both mock and unittest.mock the fixtures.MockPatch\nused the mock lib instead of the unittest.mock lib.\nThe path Ibf4f36136f2c65adad64f75d665c00cf2de4b400 (Remove the PowerVM driver)\nremoved the last user of mock lib from nova. So it is also\nremoved the mock from test-requirements. This triggered that\nfixtures.MockPatch athat started using unittest.mock too.\n\nBefore Ibf4f36136f2c65adad64f75d665c00cf2de4b400 a function can be mocked\ntwice once with unittest.mock and once with fixtures.MockPatch (still\nusing mock). However after that patch both path of such double\nmocking goes through unittest.mock and the second one fails.\n\nSo this patch fixes double mocking so far hidden behind\nfixtures.MockPatch.\n\nThis patch made the py310 and functional-py310 jobs voting  on master\nhowever that has been dropped as part of the backport.\n\nConflicts:\n    nova/tests/fixtures/nova.py\n    nova/tests/unit/api/openstack/compute/test_quotas.py\n    nova/tests/unit/api/openstack/compute/test_server_group_quotas.py\n\nConflicts are due to lack of unified limits feature in xena\n\nChange-Id: Ic1352ec31996577a5d0ad18a057339df3e49de25\n(cherry picked from commit bf654e3a4a8f690ad0bec0955690bf4fadf98dba)\n(cherry picked from commit 69667a817cb65c3efbe4e3ada0e8c69c0a106087)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f6b3a5448fdd0f6bb02897f021fb9d916d8c21db"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f6b3a5448fdd0f6bb02897f021fb9d916d8c21db"}]},"branch":"refs/heads/stable/xena"},"d09447086e24c67f6a7c28e6260aefd9be1ad75b":{"kind":"TRIVIAL_REBASE","_number":4,"created":"2022-11-21 17:08:27.000000000","uploader":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"ref":"refs/changes/35/864935/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/864935/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/864935/4"}}},"commit":{"parents":[{"commit":"2f195284795e4bdea1f63c7184bd342d3920fbf1","subject":"Remove double mocking","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/2f195284795e4bdea1f63c7184bd342d3920fbf1"}]}],"author":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2022-08-08 17:44:41.000000000","tz":120},"committer":{"name":"Sean Mooney","email":"work@seanmooney.info","date":"2022-11-21 17:05:55.000000000","tz":0},"subject":"Remove double mocking... again","message":"Remove double mocking... again\n\nI thought we fixed all the double mocking issues with\nI3998d0d49583806ac1c3ae64f1b1fe343cefd20d but I was wrong.\n\nWhile we used both mock and unittest.mock the fixtures.MockPatch\nused the mock lib instead of the unittest.mock lib.\nThe path Ibf4f36136f2c65adad64f75d665c00cf2de4b400 (Remove the PowerVM driver)\nremoved the last user of mock lib from nova. So it is also\nremoved the mock from test-requirements. This triggered that\nfixtures.MockPatch athat started using unittest.mock too.\n\nBefore Ibf4f36136f2c65adad64f75d665c00cf2de4b400 a function can be mocked\ntwice once with unittest.mock and once with fixtures.MockPatch (still\nusing mock). However after that patch both path of such double\nmocking goes through unittest.mock and the second one fails.\n\nSo this patch fixes double mocking so far hidden behind\nfixtures.MockPatch.\n\nThis patch made the py310 and functional-py310 jobs voting  on master\nhowever that has been dropped as part of the backport.\n\nConflicts:\n    nova/tests/fixtures/nova.py\n    nova/tests/unit/api/openstack/compute/test_quotas.py\n    nova/tests/unit/api/openstack/compute/test_server_group_quotas.py\n\nConflicts are due to lack of unified limits feature in xena\n\nChange-Id: Ic1352ec31996577a5d0ad18a057339df3e49de25\n(cherry picked from commit bf654e3a4a8f690ad0bec0955690bf4fadf98dba)\n(cherry picked from commit 69667a817cb65c3efbe4e3ada0e8c69c0a106087)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/d09447086e24c67f6a7c28e6260aefd9be1ad75b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/d09447086e24c67f6a7c28e6260aefd9be1ad75b"}]},"branch":"refs/heads/stable/xena"},"0a938d568e038360a9e633bb169a54e35410be0b":{"kind":"REWORK","_number":5,"created":"2022-11-22 11:54:07.000000000","uploader":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"ref":"refs/changes/35/864935/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/864935/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/864935/5"}}},"commit":{"parents":[{"commit":"b648352115d79924d59b59670213f066cc253396","subject":"Remove double mocking","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b648352115d79924d59b59670213f066cc253396"}]}],"author":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2022-08-08 17:44:41.000000000","tz":120},"committer":{"name":"Sean Mooney","email":"work@seanmooney.info","date":"2022-11-21 20:25:27.000000000","tz":0},"subject":"Remove double mocking... again","message":"Remove double mocking... again\n\nI thought we fixed all the double mocking issues with\nI3998d0d49583806ac1c3ae64f1b1fe343cefd20d but I was wrong.\n\nWhile we used both mock and unittest.mock the fixtures.MockPatch\nused the mock lib instead of the unittest.mock lib.\nThe path Ibf4f36136f2c65adad64f75d665c00cf2de4b400 (Remove the PowerVM driver)\nremoved the last user of mock lib from nova. So it is also\nremoved the mock from test-requirements. This triggered that\nfixtures.MockPatch athat started using unittest.mock too.\n\nBefore Ibf4f36136f2c65adad64f75d665c00cf2de4b400 a function can be mocked\ntwice once with unittest.mock and once with fixtures.MockPatch (still\nusing mock). However after that patch both path of such double\nmocking goes through unittest.mock and the second one fails.\n\nSo this patch fixes double mocking so far hidden behind\nfixtures.MockPatch.\n\nThis patch made the py310 and functional-py310 jobs voting  on master\nhowever that has been dropped as part of the backport.\n\nConflicts:\n    nova/tests/fixtures/nova.py\n    nova/tests/unit/api/openstack/compute/test_quotas.py\n    nova/tests/unit/api/openstack/compute/test_server_group_quotas.py\n\nConflicts are due to lack of unified limits feature in xena\n\nChange-Id: Ic1352ec31996577a5d0ad18a057339df3e49de25\n(cherry picked from commit bf654e3a4a8f690ad0bec0955690bf4fadf98dba)\n(cherry picked from commit 69667a817cb65c3efbe4e3ada0e8c69c0a106087)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0a938d568e038360a9e633bb169a54e35410be0b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0a938d568e038360a9e633bb169a54e35410be0b"}]},"branch":"refs/heads/stable/xena"},"b36bc9247ff195d983865860aef8e90bf49334e9":{"kind":"NO_CHANGE","_number":6,"created":"2022-12-01 18:14:33.000000000","uploader":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"ref":"refs/changes/35/864935/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/864935/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/864935/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/864935/6"}}},"commit":{"parents":[{"commit":"54c7c97cb83609cbcba44856d412dae11fe8643b","subject":"Remove double mocking","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/54c7c97cb83609cbcba44856d412dae11fe8643b"}]}],"author":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2022-08-08 17:44:41.000000000","tz":120},"committer":{"name":"Sean Mooney","email":"work@seanmooney.info","date":"2022-12-01 18:14:01.000000000","tz":0},"subject":"Remove double mocking... again","message":"Remove double mocking... again\n\nI thought we fixed all the double mocking issues with\nI3998d0d49583806ac1c3ae64f1b1fe343cefd20d but I was wrong.\n\nWhile we used both mock and unittest.mock the fixtures.MockPatch\nused the mock lib instead of the unittest.mock lib.\nThe path Ibf4f36136f2c65adad64f75d665c00cf2de4b400 (Remove the PowerVM driver)\nremoved the last user of mock lib from nova. So it is also\nremoved the mock from test-requirements. This triggered that\nfixtures.MockPatch athat started using unittest.mock too.\n\nBefore Ibf4f36136f2c65adad64f75d665c00cf2de4b400 a function can be mocked\ntwice once with unittest.mock and once with fixtures.MockPatch (still\nusing mock). However after that patch both path of such double\nmocking goes through unittest.mock and the second one fails.\n\nSo this patch fixes double mocking so far hidden behind\nfixtures.MockPatch.\n\nThis patch made the py310 and functional-py310 jobs voting  on master\nhowever that has been dropped as part of the backport.\n\nConflicts:\n    nova/tests/fixtures/nova.py\n    nova/tests/unit/api/openstack/compute/test_quotas.py\n    nova/tests/unit/api/openstack/compute/test_server_group_quotas.py\n\nConflicts are due to lack of unified limits feature in xena\n\nChange-Id: Ic1352ec31996577a5d0ad18a057339df3e49de25\n(cherry picked from commit bf654e3a4a8f690ad0bec0955690bf4fadf98dba)\n(cherry picked from commit 69667a817cb65c3efbe4e3ada0e8c69c0a106087)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b36bc9247ff195d983865860aef8e90bf49334e9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b36bc9247ff195d983865860aef8e90bf49334e9"}]},"branch":"refs/heads/stable/xena"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}},{"label":"Review-Priority","status":"MAY"}]}],"submit_requirements":[]}
