)]}'
{"id":"openstack%2Fnova~814735","triplet_id":"openstack%2Fnova~master~Id96f1795034490c13125ebbab49b029fb96af1c7","project":"openstack/nova","branch":"master","topic":"bug/1948963","attention_set":{},"removed_from_attention_set":{"4690":{"account":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"last_update":"2021-11-09 13:59:03.000000000","reason":"Change was submitted"},"11604":{"account":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"last_update":"2021-11-09 13:59:03.000000000","reason":"Change was submitted"},"15334":{"account":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"last_update":"2021-11-09 13:59:03.000000000","reason":"Change was submitted"},"9708":{"account":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"last_update":"2021-11-09 13:59:03.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"Id96f1795034490c13125ebbab49b029fb96af1c7","subject":"Fix interference in db unit test","status":"MERGED","created":"2021-10-20 10:23:41.000000000","updated":"2021-11-09 14:22:19.000000000","submitted":"2021-11-09 13:59:03.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":13,"unresolved_comment_count":1,"has_review_started":true,"submission_id":"814735-bug/1948963","meta_rev_id":"d267eb8f1d75f31ce1822335979c5f725cfa4d19","_number":814735,"virtual_id_number":814735,"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":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-11-09 13:59:03.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"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":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"all":[{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":2,"date":"2021-11-08 11:36:53.000000000","_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2021-11-08 13:50:04.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"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":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2021-11-08 13:50:04.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"recommended":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"all":[{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2021-11-09 14:22:19.000000000","post_submit":true,"permitted_voting_range":{"min":1,"max":2},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","value":1,"default_value":0,"optional":true}},"removable_reviewers":[{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"}],"reviewers":{"CC":[{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}],"REVIEWER":[{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-10-20 10:39:38.000000000","updated_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"reviewer":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"state":"REVIEWER"},{"updated":"2021-10-20 10:39:38.000000000","updated_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2021-10-20 10:39:48.000000000","updated_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"reviewer":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"state":"REVIEWER"},{"updated":"2021-10-20 11:08:25.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2021-10-20 11:08:59.000000000","updated_by":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-10-20 12:52:05.000000000","updated_by":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"reviewer":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"state":"CC"},{"updated":"2021-10-26 16:14:23.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":"2021-11-05 01:06:39.000000000","updated_by":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-11-08 11:36:53.000000000","updated_by":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"reviewer":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"state":"REVIEWER"}],"messages":[{"id":"40acf2c5c1566d4a6b311d7a01110fced2767e29","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-10-20 10:23:41.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"12bf6207dce8580f5624862394d1ec21e54d0963","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-10-20 10:37:01.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"3aa2379beef6a01b7f83602169f953d31546ee4e","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-10-20 10:40:30.000000000","message":"Uploaded patch set 3: Commit message was updated.","accounts_in_message":[],"_revision_number":3},{"id":"72fa3808743a16f9de23aef86a8799044f794493","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-10-20 11:08:25.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/0be8df9d819f4613bdf0e3b222fdbe13 : SUCCESS in 15m 46s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/7f7aa38203ce40b992d25d6453cc6114 : SUCCESS in 13m 43s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/aa6eab86b5a14f889440f1d17d2245a0 : SUCCESS in 8m 13s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/5db60fe4bb9c43b7b12edce91881502c : SUCCESS in 12m 17s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/c320b4a72ddc43be81d28697992b4778 : SUCCESS in 13m 16s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/98371ebb61c04a888366962f8313e42e : SUCCESS in 22m 20s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a7842aa8cc3c4b8690671b6a0a365f3d : SUCCESS in 10m 28s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/75d8e8063b734863ae4cbfa5730ec61e : SUCCESS in 6m 30s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/d3d8cb1ad688498590da1ca7375a12cd : SUCCESS in 21m 51s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/f6857913f28d4ae0864c01645a83bd6f : SUCCESS in 19m 57s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/6b19007c042042febc7dbed9c8cd0a32 : SUCCESS in 20m 29s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"478791ed58d527d811022571ed36ab121e303fcf","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-10-20 11:08:58.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/8adc13a6b3054f38a5e48443538bdcb3 : SUCCESS in 22m 52s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/9f8a27ab185f4313b8220a2f95a03449 : SUCCESS in 20m 32s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"39c08f37b74cb4496549b61ac56fd542dbc1efcb","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-10-20 11:08:59.000000000","message":"Patch Set 3:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- EMC_PowerFlex_NOVA https://elab-os-logsrv.delllabs.net/35/814735/3/check/EMC_PowerFlex_NOVA/a1ff76e/EMC_PowerFlex_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC PowerFlex CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_PowerFlex_CI","accounts_in_message":[],"_revision_number":3},{"id":"12a9a7366ba93fc61fd4fac213a756dec0d1e826","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-10-20 11:09:17.000000000","message":"Patch Set 3:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- EMC_PowerFlex_NOVA https://elab-os-logsrv.delllabs.net/35/814735/3/check/EMC_PowerFlex_NOVA/95ad88c/EMC_PowerFlex_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC PowerFlex CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_PowerFlex_CI","accounts_in_message":[],"_revision_number":3},{"id":"7b528f10441ba96b74f4fc1c6e5be3ccf1e52a01","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-10-20 12:52:05.000000000","message":"Patch Set 3:\n\nFor rechecking only on the Cloudbase Nova Hyper-V CI, add a review comment with run-Cloudbase Nova Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/814735/3/iscsi : FAILURE in 2h 10m 10s","accounts_in_message":[],"_revision_number":3},{"id":"481db1ed22728722ea99fae15ce6deb1e8c0ef40","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-10-20 19:17:20.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"1c7b9653409fc736ed77666b06bd61f0f3fe4836","author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"date":"2021-10-26 16:14:23.000000000","message":"Patch Set 3: Review-Priority+1","accounts_in_message":[],"_revision_number":3},{"id":"d6c2be2bfb27a655778f9bdaaa7604d0b178a01c","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-10-27 14:59:34.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"401c02db84abe21c77a2baab4ca9ccf3979786b5","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-10-27 15:01:36.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"c704f88311a899ff30bfab477230ddc35c9d81ed","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-10-27 15:32:53.000000000","message":"Patch Set 4:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/5ecc7975fdff4699bf310287acc6b7ce : SUCCESS in 28m 20s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/18784bd8a70a402bb6052e6206b49241 : SUCCESS in 30m 41s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"ccad41fa1d627c03d93b3ebd13445eee1fac1427","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-10-27 16:23:00.000000000","message":"Patch Set 4: Code-Review+2\n\n(7 comments)","accounts_in_message":[],"_revision_number":4},{"id":"75e7a1273c6165abcdb208e07b2bd1535efc40f2","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-10-27 17:08:58.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/0c2b703289a44b96bba1b11ac48807e3 : SUCCESS in 17m 14s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/2860dfa929cd4f5591ff011ed6967cff : SUCCESS in 13m 58s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2576aa1f993842f3ab19538bb8facc3f : SUCCESS in 9m 53s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/a82cc98d5230476bafcf9fe057cadf43 : SUCCESS in 12m 40s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/1337de1a9ff84cfc99c8a7f9956cebc8 : SUCCESS in 13m 59s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/71af8fe590174bffb3f3441bde5d2554 : SUCCESS in 15m 21s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/92978281f97240348b65094dde657d8b : SUCCESS in 10m 46s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/376d5cf6e9a74e2da78bac83866790c0 : SUCCESS in 8m 46s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/84cb27d5dc40420989d241226be1391b : SUCCESS in 21m 42s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/a87b1f751c034bf0aa78a070d21b8b12 : SUCCESS in 22m 19s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/0b5bd2d62fa9409a9c2c44d589e80717 : SUCCESS in 22m 07s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"7e5d6af2d6437fc5bb05ed2cddcf23d2cd8c7eca","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-10-27 17:09:53.000000000","message":"Patch Set 4:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- EMC_PowerFlex_NOVA https://elab-os-logsrv.delllabs.net/35/814735/4/check/EMC_PowerFlex_NOVA/92a0927/EMC_PowerFlex_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC PowerFlex CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_PowerFlex_CI","accounts_in_message":[],"_revision_number":4},{"id":"92fad9b7fd88ea8cb6de49d75189537344dc47ff","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-10-27 17:20:04.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"23d0878fbd5cc9493d056c5ed31a7e4e6e6f8572","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-10-27 17:20:18.000000000","message":"Patch Set 5:\n\n(4 comments)","accounts_in_message":[],"_revision_number":5},{"id":"2252ac85f015f0ec7a88666e6b46fe02b9a036dc","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-10-27 17:52:20.000000000","message":"Patch Set 5:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/020ebb1e2da64b8596b49c20d624c9be : SUCCESS in 26m 45s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/8db0664c0e6147b7915d15df7412f20c : SUCCESS in 24m 24s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"e985f60bcf975b8098606401cb7ad2a72e057de6","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-10-27 18:41:14.000000000","message":"Patch Set 5: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/e0dbfaeaed0f4ed8a47830b61914ad86 : SUCCESS in 24m 10s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/992eed5e48eb4bdaac32b6c9f5f4f696 : SUCCESS in 29m 15s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/083d3734367c479ea18a187c367c5763 : FAILURE in 8m 16s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/639ca89f60a84db6bf962b0a29422616 : SUCCESS in 15m 08s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/75477808342846cfa46accd51590fe48 : SUCCESS in 13m 45s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/1bead110cc4d43a88fb99263aae8307c : SUCCESS in 14m 54s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e21a074c767a4503a250c37f04e119b4 : SUCCESS in 11m 03s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/b873ece8d0fc43b5997cd1179bf42c8e : SUCCESS in 8m 54s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/c0dd51af3d094fd6b77c51c5e2f66238 : SUCCESS in 22m 02s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/60e51cb0ec1c4244b528e514d6926d6a : SUCCESS in 21m 35s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/d0ed812f0cc742e1b7ff8deefad02330 : SUCCESS in 21m 14s (non-voting)\n\nWarning:\n  Comments left for invalid file nova/tests/functional/db/test_connection_switch.py","accounts_in_message":[],"_revision_number":5},{"id":"894e9609a3201724653b75b5ae0bae56b785c66d","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-10-27 19:20:23.000000000","message":"Patch Set 5:\n\nFor rechecking only on the Cloudbase Nova Hyper-V CI, add a review comment with run-Cloudbase Nova Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/814735/5/iscsi : FAILURE in 1h 57m 54s","accounts_in_message":[],"_revision_number":5},{"id":"ff490de59c7157b4f7bdddeb88cb568570435a9f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-11-04 16:29:49.000000000","message":"Uploaded patch set 6: Patch Set 5 was rebased.","accounts_in_message":[],"_revision_number":6},{"id":"4aa313a1323981fb9d5d25ed83c5357678e3e80b","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-11-04 16:49:04.000000000","message":"Patch Set 6:\n\nFor rechecking only on the Cloudbase Nova Hyper-V CI, add a review comment with run-Cloudbase Nova Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/814735/6/iscsi : FAILURE in 17m 05s","accounts_in_message":[],"_revision_number":6},{"id":"372c73bc7611078ba2569b37da2d999fa6d31b35","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-11-04 17:24:21.000000000","message":"Patch Set 6:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/50ffcbe5ed3c484eb18c7972380f4be0 : SUCCESS in 37m 44s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/b279619c59e444169c11a98ca3dfdbb9 : SUCCESS in 50m 28s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"bdc7c407d55b8042ab267c2e91293174f8f9bd83","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-11-04 17:27:19.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/70716b3707784ec49d43c158b6988bde : SUCCESS in 16m 11s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/53bb515624b84ca7bd0b9662677d9f1c : SUCCESS in 15m 20s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5ad0563d4c0a45ffbb4af90d6e770d0e : SUCCESS in 9m 45s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/5097762f1fd54e6b8e341bbfc3ee0e3f : SUCCESS in 13m 38s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/5ca91262f4184968a29f4c5a4d3b04a5 : SUCCESS in 20m 31s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/81ef87164ca04645bc0a2b7d31de002c : SUCCESS in 15m 26s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/77777e9ca6754c00bdce33a37b009888 : SUCCESS in 12m 34s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/5bf1c0ce24d348568c8cdff22a244e69 : SUCCESS in 7m 25s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/f50b18d583a542a9a0933e7b5847efe0 : SUCCESS in 24m 17s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/0d8ea6aaed7a428fa7d909a837e70d27 : SUCCESS in 24m 25s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/0a0a9696966b47bba8edfa183ae6cdba : SUCCESS in 22m 23s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"e433f78f18db07f51838c78160397539a32172b9","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-11-04 17:28:01.000000000","message":"Patch Set 6:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- EMC_PowerFlex_NOVA https://elab-os-logsrv.delllabs.net/35/814735/6/check/EMC_PowerFlex_NOVA/51dc145/EMC_PowerFlex_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC PowerFlex CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_PowerFlex_CI","accounts_in_message":[],"_revision_number":6},{"id":"72de8289c4d205c9b691d8fe1323f955c8e385d0","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-11-05 01:06:39.000000000","message":"Patch Set 6:\n\nBuild succeeded\n\n- dsvm-nova http://207.189.188.190/logs/814735/814735/6/check-not-vote/ext-nova-zuul/3e60804 : SUCCESS in 57m 49s","accounts_in_message":[],"_revision_number":6},{"id":"a2474033223a12a16f2d1b2d5f9f7a3c12341b2c","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-11-05 10:52:07.000000000","message":"Uploaded patch set 7: Patch Set 6 was rebased.","accounts_in_message":[],"_revision_number":7},{"id":"1bdb7540d93c5f8acedf2466e544f4524c9d59fb","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-11-05 10:58:18.000000000","message":"Patch Set 7:\n\nBuild failed\n\n- dsvm-nova http://207.189.188.190/logs/814735/814735/7/check-not-vote/ext-nova-zuul/6707bca : FAILURE in 2s","accounts_in_message":[],"_revision_number":7},{"id":"d70a963df9d3fa264ae58b860d360fc8016108e9","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-11-05 11:22:58.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/b80b9e32249a44b9b38c962b374864d9 : SUCCESS in 16m 35s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/e059bf9597b0476b89df3b1e6eafc9f1 : SUCCESS in 15m 38s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c37d60aeea4941aeb2f145a1fbb3107d : SUCCESS in 8m 42s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/0fd03e1757344e9a87d827b062b3f1c6 : SUCCESS in 12m 23s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/e0e5f8f523434dcd844f104d9832159d : SUCCESS in 13m 56s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/7f7fe3f7ebd24c96a245b9664f0697ab : SUCCESS in 13m 22s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/02ebc62197d74aeca69f96f5456b522f : SUCCESS in 9m 58s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/cc14be2d1a794603a4884cdfb6789d5c : SUCCESS in 5m 42s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/f175e51ac124416fae8dadb447922521 : SUCCESS in 24m 16s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/b3875187f0454281b83bc270f69dd7cf : SUCCESS in 20m 18s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/3440d063bbb046bfbade7afe44c4b09a : SUCCESS in 20m 28s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"d8ec888276a98ff66565ebcadbe804fe58d0a6c1","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-11-05 11:29:44.000000000","message":"Patch Set 7:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/9fe4249eb09b437ca99bc296b09f2a93 : FAILURE in 27m 49s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/ddd985794bc64c029125ac1df1a1c7f1 : SUCCESS in 30m 24s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"04a44142687b4f5bc1b0ca45693520bba966f19a","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-11-05 13:18:27.000000000","message":"Patch Set 7:\n\nBuild succeeded. For checking Nova with Ceph, add a comment with run-Cloudbase Nova Ceph Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/814735/7/iscsi : SUCCESS in 2h 22m 42s","accounts_in_message":[],"_revision_number":7},{"id":"5bbefdc3c0d61c627a2c4260dc882b04c76bbdcc","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-11-08 11:36:53.000000000","message":"Patch Set 7: Code-Review+2","accounts_in_message":[],"_revision_number":7},{"id":"8ebf43219cf2470ec5bbbdb5babeb498db9a77b9","author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"date":"2021-11-08 13:50:04.000000000","message":"Patch Set 7: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":7},{"id":"9c70fa6e37bf0d6925950a420ded544837a44465","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-11-09 08:42:31.000000000","message":"Patch Set 7: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":7},{"id":"c7829ef66c4ebed590ac34463d1f93ec8d5fc2b6","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-11-09 13:59:03.000000000","message":"Patch Set 7: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/1ba3f1b0b89d45b3bd09c71ff3aec6cf : SUCCESS in 1h 25m 54s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/03e92984de8c401c930ac586d8cf3943 : SUCCESS in 14m 52s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/9b81c4678e964379b017e23ee6a0b045 : SUCCESS in 18m 15s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/6ff673ba858244cb882fe1ab2802d721 : SUCCESS in 14m 21s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/164f09b828f9423abea60747713c8401 : SUCCESS in 15m 06s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e1321878032b4651ba4b8148d7d02e3f : SUCCESS in 10m 57s\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/80a6cb03db3242e58a5154091757b076 : SUCCESS in 24m 35s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/2fbc69f758474e42ac6123c26b79dee7 : SUCCESS in 20m 28s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/c9d3584f82b743f69482e29a3e36b040 : SUCCESS in 7m 54s","accounts_in_message":[],"_revision_number":7},{"id":"1e2f987ff387b0e32cf7606322fceaa242f7ced3","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-11-09 13:59:03.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":7},{"id":"5ca092e3e71f8fa14e87de559de41207056c93cb","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-11-09 14:22:19.000000000","message":"Patch Set 7:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6554ca01048046588535e82a518499c7 : SUCCESS in 2m 08s","accounts_in_message":[],"_revision_number":7}],"current_revision_number":7,"current_revision":"a7bccff06b871765fb87ee3e9edd9458c7f73701","revisions":{"f5af46d1ee0baf541a9547a48dd32b393ab01a2b":{"kind":"REWORK","_number":1,"created":"2021-10-20 10:23:41.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/35/814735/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/814735/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/814735/1"}}},"commit":{"parents":[{"commit":"e14eef0719eceef35e7e96b3e3d242ec79a80969","subject":"Merge \"Fix the wrong exception used to retry detach API calls\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e14eef0719eceef35e7e96b3e3d242ec79a80969"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-20 10:19:44.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-20 10:22:02.000000000","tz":120},"subject":"DNM: turn off two unit test cases","message":"DNM: turn off two unit test cases\n\nThe two unit test case causes errors like\n  oslo_db.sqlalchemy.enginefacade.AlreadyStartedError:\n  this TransactionFactory is already started\n\nIf the db tests run selectively. So far I only found the offending test\ncases by bisecting the recent patches, but I don\u0027t know why these are\nbreaking our test execution and why only if db tests are run\nselectively.\n\nChange-Id: Id96f1795034490c13125ebbab49b029fb96af1c7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f5af46d1ee0baf541a9547a48dd32b393ab01a2b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f5af46d1ee0baf541a9547a48dd32b393ab01a2b"}]},"branch":"refs/heads/master"},"88ef67c5b06604a147e5231f87c8c7431b474a4b":{"kind":"REWORK","_number":2,"created":"2021-10-20 10:37:01.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/35/814735/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/814735/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/814735/2"}}},"commit":{"parents":[{"commit":"e14eef0719eceef35e7e96b3e3d242ec79a80969","subject":"Merge \"Fix the wrong exception used to retry detach API calls\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e14eef0719eceef35e7e96b3e3d242ec79a80969"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-20 10:19:44.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-20 10:33:31.000000000","tz":120},"subject":"Fix db migration unit test","message":"Fix db migration unit test\n\nTwo unit test case causes added in\nI0647bb8545c1464b521a1d866cf5ee674aea2eae causes errors like\n  oslo_db.sqlalchemy.enginefacade.AlreadyStartedError:\n  this TransactionFactory is already started\n\nwhen the db tests run selectively with tox -e py38 nova.tests.unit.db\nbut does not cause errors if the whole unit test suit is run.\n\nIn other test cases I found that migration.db_sync call was done in a\nway that the migration._get_engine is mocked to use a local engine\ninstead of a global one. So I modified these test cases to do the same\nand it seems it fixed the problem. Still I don\u0027t understand why this\nchange helps or how the original error happens.\n\nChange-Id: Id96f1795034490c13125ebbab49b029fb96af1c7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/88ef67c5b06604a147e5231f87c8c7431b474a4b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/88ef67c5b06604a147e5231f87c8c7431b474a4b"}]},"branch":"refs/heads/master"},"a14125feb0e6cfcdbf18d223235d736d9ae48a13":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2021-10-20 10:40:30.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/35/814735/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/814735/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/814735/3"}}},"commit":{"parents":[{"commit":"e14eef0719eceef35e7e96b3e3d242ec79a80969","subject":"Merge \"Fix the wrong exception used to retry detach API calls\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e14eef0719eceef35e7e96b3e3d242ec79a80969"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-20 10:19:44.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-20 10:38:46.000000000","tz":120},"subject":"Fix db migration unit test","message":"Fix db migration unit test\n\nTwo unit test cases added in\nI0647bb8545c1464b521a1d866cf5ee674aea2eae cause errors like\n  oslo_db.sqlalchemy.enginefacade.AlreadyStartedError:\n  this TransactionFactory is already started\n\nwhen the db tests run selectively with tox -e py38 nova.tests.unit.db\nbut does not cause errors if the whole unit test suit is run.\n\nIn other test cases I found that migration.db_sync call was done in a\nway that the migration._get_engine is mocked to use a local engine\ninstead of a global one. So I modified these test cases to do the same\nand it seems it fixed the problem. Still I don\u0027t understand why this\nchange helps or how the original error happens.\n\nChange-Id: Id96f1795034490c13125ebbab49b029fb96af1c7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a14125feb0e6cfcdbf18d223235d736d9ae48a13"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a14125feb0e6cfcdbf18d223235d736d9ae48a13"}]},"branch":"refs/heads/master"},"44d1f22a98eae2fdd00f9d96422a3a291bcfda1a":{"kind":"REWORK","_number":4,"created":"2021-10-27 14:59:34.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/35/814735/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/814735/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/814735/4"}}},"commit":{"parents":[{"commit":"5b60083e2eb297ec52463190c22661de8cdba3ef","subject":"Refactor Database fixture","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/5b60083e2eb297ec52463190c22661de8cdba3ef"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-27 14:29:29.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-27 14:55:11.000000000","tz":120},"subject":"Fix interference in db unit test","message":"Fix interference in db unit test\n\nWe discovered that two unit test cases added in\nI0647bb8545c1464b521a1d866cf5ee674aea2eae cause errors like\n  oslo_db.sqlalchemy.enginefacade.AlreadyStartedError:\n  this TransactionFactory is already started\n\nwhen the db tests run selectively with tox -e py38 nova.tests.unit.db\nbut does not cause errors if the whole unit test suit is run.\n\nThis error happens because our db code uses two global transaction\nfactory, one for the api DB and one for the main DB. There was a global\nflag SESSION_CONFIGURED in our Database fixture that guarded against\ndouble initialization of the factory. But the faulty test cases in\nquestion does not use our Database fixture but use the\nOpportunisticDBTestMixin from oslo_db. Obviously that fixture does not\nknow about our SESSION_CONFIGURED global. So if one of the offending\ntest case run first in an executor then that could initialize the\ntransaction factory globally and a later test that uses our Database\nfixture will try to configure it again leading to the error. For some\nunknown reason if these tests were run in the opposite order the faulty\nre-initialization did not happen. Probably the OpportunisticDBTestMixin\nwas able to prevent that.\n\nA previous patch already removed the global SESSION_CONFIGURED flag\nfrom our fixture and replaced it with a per DB specific patch_factory\ncalls that allow resetting the state of the factory at the end of each\ntest case. This would already solve the current test case issue as only\nour offending test cases would initialize the global factory without\ncleanup and we have one test case per DB. So there would be no\ninterference. However if in the future we add similar test cases then\nthose can still interfere through the global factory.\n\nSo this patch fixes the two offending test case. Also it extends the\nDatabasePoisonFixture used for the NoDbTestCase tesst. The poison now\ndetects if the test case starts any transaction factory.\n\nThis poison caught another offender test case,\ntest_db_sync_with_special_symbols_in_connection_string, that was marked\nNoDb but actually using the database. It is now changed to declare\nitself as a test that sets up the DB manually and also it is changed to\nuse the Database fixture instead of touching the global factory\ndirectly.\n\nChange-Id: Id96f1795034490c13125ebbab49b029fb96af1c7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/44d1f22a98eae2fdd00f9d96422a3a291bcfda1a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/44d1f22a98eae2fdd00f9d96422a3a291bcfda1a"}]},"branch":"refs/heads/master"},"a7b5fe6e3b5460e84533f6ebf146cde5c1efa3e3":{"kind":"REWORK","_number":5,"created":"2021-10-27 17:20:04.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/35/814735/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/814735/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/814735/5"}}},"commit":{"parents":[{"commit":"5538e2ceed0472f75c75ed5f5e38191523acc6e8","subject":"Refactor Database fixture","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/5538e2ceed0472f75c75ed5f5e38191523acc6e8"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-27 14:29:29.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-27 17:16:32.000000000","tz":120},"subject":"Fix interference in db unit test","message":"Fix interference in db unit test\n\nWe discovered that two unit test cases added in\nI0647bb8545c1464b521a1d866cf5ee674aea2eae cause errors like\n  oslo_db.sqlalchemy.enginefacade.AlreadyStartedError:\n  this TransactionFactory is already started\n\nwhen the db tests run selectively with tox -e py38 nova.tests.unit.db\nbut does not cause errors if the whole unit test suit is run.\n\nThis error happened because our db code uses two global transaction\nfactory, one for the api DB and one for the main DB. There was a global\nflag SESSION_CONFIGURED in our Database fixture that guarded against\ndouble initialization of the factory. But the faulty test cases in\nquestion do not use our Database fixture but use the\nOpportunisticDBTestMixin from oslo_db. Obviously that fixture does not\nknow about our SESSION_CONFIGURED global. So if one of the offending\ntest case ran first in an executor then that initialized the\ntransaction factory globally and a later test that uses our Database\nfixture tried to configure it again leading to the error. For some\nunknown reason if these tests were run in the opposite order the faulty\nre-initialization did not happen. Probably the OpportunisticDBTestMixin\nwas able to prevent that.\n\nA previous patch already removed the global SESSION_CONFIGURED flag\nfrom our fixture and replaced it with a per DB specific patch_factory\ncalls that allow resetting the state of the factory at the end of each\ntest case. This would already solve the current test case issue as only\nour offending test cases would initialize the global factory without\ncleanup and we have one test case per DB. So there would be no\ninterference. However if in the future we add similar test cases then\nthose can still interfere through the global factory.\n\nSo this patch fixes the two offending test case. Also it extends the\nDatabasePoisonFixture used for the NoDbTestCase tesst. The poison now\ndetects if the test case starts any transaction factory.\n\nThis poison caught another offender test case,\ntest_db_sync_with_special_symbols_in_connection_string, that was marked\nNoDb but actually using the database. It is now changed to declare\nitself as a test that sets up the DB manually and also it is changed to\nuse the Database fixture instead of touching the global factory\ndirectly.\n\nCloses-Bug: #1948963\n\nChange-Id: Id96f1795034490c13125ebbab49b029fb96af1c7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a7b5fe6e3b5460e84533f6ebf146cde5c1efa3e3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a7b5fe6e3b5460e84533f6ebf146cde5c1efa3e3"}]},"branch":"refs/heads/master"},"16c8552dac71dd2e9e724ee8047d68831e6766f0":{"kind":"TRIVIAL_REBASE","_number":6,"created":"2021-11-04 16:29:49.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/35/814735/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/814735/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/814735/6"}}},"commit":{"parents":[{"commit":"c7ccbfe403f100b5cb420fb12ac6244c4a7ae381","subject":"Refactor Database fixture","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c7ccbfe403f100b5cb420fb12ac6244c4a7ae381"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-27 14:29:29.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-11-04 16:18:56.000000000","tz":60},"subject":"Fix interference in db unit test","message":"Fix interference in db unit test\n\nWe discovered that two unit test cases added in\nI0647bb8545c1464b521a1d866cf5ee674aea2eae cause errors like\n  oslo_db.sqlalchemy.enginefacade.AlreadyStartedError:\n  this TransactionFactory is already started\n\nwhen the db tests run selectively with tox -e py38 nova.tests.unit.db\nbut does not cause errors if the whole unit test suit is run.\n\nThis error happened because our db code uses two global transaction\nfactory, one for the api DB and one for the main DB. There was a global\nflag SESSION_CONFIGURED in our Database fixture that guarded against\ndouble initialization of the factory. But the faulty test cases in\nquestion do not use our Database fixture but use the\nOpportunisticDBTestMixin from oslo_db. Obviously that fixture does not\nknow about our SESSION_CONFIGURED global. So if one of the offending\ntest case ran first in an executor then that initialized the\ntransaction factory globally and a later test that uses our Database\nfixture tried to configure it again leading to the error. For some\nunknown reason if these tests were run in the opposite order the faulty\nre-initialization did not happen. Probably the OpportunisticDBTestMixin\nwas able to prevent that.\n\nA previous patch already removed the global SESSION_CONFIGURED flag\nfrom our fixture and replaced it with a per DB specific patch_factory\ncalls that allow resetting the state of the factory at the end of each\ntest case. This would already solve the current test case issue as only\nour offending test cases would initialize the global factory without\ncleanup and we have one test case per DB. So there would be no\ninterference. However if in the future we add similar test cases then\nthose can still interfere through the global factory.\n\nSo this patch fixes the two offending test case. Also it extends the\nDatabasePoisonFixture used for the NoDbTestCase tesst. The poison now\ndetects if the test case starts any transaction factory.\n\nThis poison caught another offender test case,\ntest_db_sync_with_special_symbols_in_connection_string, that was marked\nNoDb but actually using the database. It is now changed to declare\nitself as a test that sets up the DB manually and also it is changed to\nuse the Database fixture instead of touching the global factory\ndirectly.\n\nCloses-Bug: #1948963\n\nChange-Id: Id96f1795034490c13125ebbab49b029fb96af1c7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/16c8552dac71dd2e9e724ee8047d68831e6766f0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/16c8552dac71dd2e9e724ee8047d68831e6766f0"}]},"branch":"refs/heads/master"},"a7bccff06b871765fb87ee3e9edd9458c7f73701":{"kind":"TRIVIAL_REBASE","_number":7,"created":"2021-11-05 10:52:07.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/35/814735/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/814735/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/814735/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/814735/7"}}},"commit":{"parents":[{"commit":"063ed41174c27c20871696f7c6dcb1eddb4310ee","subject":"Use ReplaceEngineFacade fixture","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/063ed41174c27c20871696f7c6dcb1eddb4310ee"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-10-27 14:29:29.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-11-05 10:41:27.000000000","tz":60},"subject":"Fix interference in db unit test","message":"Fix interference in db unit test\n\nWe discovered that two unit test cases added in\nI0647bb8545c1464b521a1d866cf5ee674aea2eae cause errors like\n  oslo_db.sqlalchemy.enginefacade.AlreadyStartedError:\n  this TransactionFactory is already started\n\nwhen the db tests run selectively with tox -e py38 nova.tests.unit.db\nbut does not cause errors if the whole unit test suit is run.\n\nThis error happened because our db code uses two global transaction\nfactory, one for the api DB and one for the main DB. There was a global\nflag SESSION_CONFIGURED in our Database fixture that guarded against\ndouble initialization of the factory. But the faulty test cases in\nquestion do not use our Database fixture but use the\nOpportunisticDBTestMixin from oslo_db. Obviously that fixture does not\nknow about our SESSION_CONFIGURED global. So if one of the offending\ntest case ran first in an executor then that initialized the\ntransaction factory globally and a later test that uses our Database\nfixture tried to configure it again leading to the error. For some\nunknown reason if these tests were run in the opposite order the faulty\nre-initialization did not happen. Probably the OpportunisticDBTestMixin\nwas able to prevent that.\n\nA previous patch already removed the global SESSION_CONFIGURED flag\nfrom our fixture and replaced it with a per DB specific patch_factory\ncalls that allow resetting the state of the factory at the end of each\ntest case. This would already solve the current test case issue as only\nour offending test cases would initialize the global factory without\ncleanup and we have one test case per DB. So there would be no\ninterference. However if in the future we add similar test cases then\nthose can still interfere through the global factory.\n\nSo this patch fixes the two offending test case. Also it extends the\nDatabasePoisonFixture used for the NoDbTestCase tesst. The poison now\ndetects if the test case starts any transaction factory.\n\nThis poison caught another offender test case,\ntest_db_sync_with_special_symbols_in_connection_string, that was marked\nNoDb but actually using the database. It is now changed to declare\nitself as a test that sets up the DB manually and also it is changed to\nuse the Database fixture instead of touching the global factory\ndirectly.\n\nCloses-Bug: #1948963\n\nChange-Id: Id96f1795034490c13125ebbab49b029fb96af1c7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a7bccff06b871765fb87ee3e9edd9458c7f73701"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a7bccff06b871765fb87ee3e9edd9458c7f73701"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"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","applied_by":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}