)]}'
{"id":"openstack%2Fwhitebox-tempest-plugin~778752","triplet_id":"openstack%2Fwhitebox-tempest-plugin~master~Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a","project":"openstack/whitebox-tempest-plugin","branch":"master","topic":"vol_encrypt","hashtags":[],"change_id":"Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a","subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","status":"MERGED","created":"2021-03-04 17:06:14.000000000","updated":"2021-03-24 00:24:03.000000000","submitted":"2021-03-24 00:24:03.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":16,"unresolved_comment_count":7,"has_review_started":true,"submission_id":"778752-vol_encrypt","meta_rev_id":"b3dcccbc0a82a8b559a954c1c3a379ffb1edf9e6","_number":778752,"virtual_id_number":778752,"owner":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-03-24 00:24:03.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},{"value":0,"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"}],"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":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2021-03-23 22:34:05.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},{"value":1,"date":"2021-03-23 17:37:52.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":1,"date":"2021-03-23 18:01:08.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},{"value":0,"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"}],"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":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2021-03-23 22:34:05.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},{"value":0,"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"reviewers":{"REVIEWER":[{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},{"_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":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-03-04 17:10:27.000000000","updated_by":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"reviewer":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"state":"REVIEWER"},{"updated":"2021-03-04 17:10:27.000000000","updated_by":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2021-03-04 17:10:27.000000000","updated_by":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"reviewer":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"state":"REVIEWER"},{"updated":"2021-03-04 17:10:27.000000000","updated_by":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"reviewer":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"state":"REVIEWER"},{"updated":"2021-03-04 20:56:44.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"e343b558b4e58dca44ba723058109f33b6d7f1f7","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-04 17:06:14.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"272ac01b5dd6abd13dcae15f00400ace024b6e45","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-04 17:11:10.000000000","message":"Patch Set 1:\n\n(overcloud) [stack@undercloud-0 tempest-dir]$ tempest run --regex whitebox_tempest_plugin.api.compute.test_volume_encryption.TestQEMUVolumeEncrption                                                                                         \n{0} whitebox_tempest_plugin.api.compute.test_volume_encryption.TestQEMUVolumeEncrption.test_qemu_volume_encryption [7.363968s] ... ok                                                                                                        \n\n\u003d\u003d\u003d\u003d\u003d\u003d\nTotals\n\u003d\u003d\u003d\u003d\u003d\u003d\nRan: 1 tests in 7.3640 sec.\n - Passed: 1\n - Skipped: 0\n - Expected Fail: 0\n - Unexpected Success: 0\n - Failed: 0\nSum of execute time for each test: 7.3640 sec.","accounts_in_message":[],"_revision_number":1},{"id":"8b99e80bf07046642c31e48dc5c5c78342d7f135","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-04 17:16:50.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"5f2f1a4bf2b1ffdb82ec1d177deebc9e4649a300","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-04 18:59:00.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"8bab1793eeb2bb530f8391c1eadb125262096535","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-04 20:55:15.000000000","message":"Patch Set 3: Code-Review-2\n\nAh, the test does not pass in reality as it\u0027s finished in 7 seconds and when i checked, it only created one volume and then finished execution marking it as passed, so the issue still remains :(","accounts_in_message":[],"_revision_number":3},{"id":"bfa5b99ee6259e79e7f63954bc0a06718b1954dc","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-04 20:56:44.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\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b8acdedd9e7c4c21ab15d5b00e3f4100 : SUCCESS in 5m 05s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/a540ecbad7de4a8f8b27340a076cb883 : SUCCESS in 4m 43s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/3a1e8cbb62504731bc2858a154c28169 : SUCCESS in 5m 21s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/9fd02f7ca81048378fe04b23003642b0 : FAILURE in 1h 48m 04s","accounts_in_message":[],"_revision_number":3},{"id":"17bc999161094e26e86ea412d47e62fb1924690d","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-05 16:56:15.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"4cc68b5eac30e96edffd46d8b0fbcd3427cf2d9b","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-05 16:59:06.000000000","message":"Patch Set 4: -Code-Review\n\nThis looks good now and ready for review/merge\n\n[stack@undercloud-0 tempest-dir]$ tempest run --regex whitebox_tempest_plugin.api.compute.test_volume_encryption.TestQEMUVolumeEncryption\n{0} whitebox_tempest_plugin.api.compute.test_volume_encryption.TestQEMUVolumeEncryption.test_qemu_volume_encryption [33.292740s] ... ok\n\n\u003d\u003d\u003d\u003d\u003d\u003d\nTotals\n\u003d\u003d\u003d\u003d\u003d\u003d\nRan: 1 tests in 33.2927 sec.\n - Passed: 1\n - Skipped: 0\n - Expected Fail: 0\n - Unexpected Success: 0\n - Failed: 0\nSum of execute time for each test: 33.2927 sec.\n\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nWorker Balance\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n - Worker 0 (1 tests) \u003d\u003e 0:00:33.292740","accounts_in_message":[],"_revision_number":4},{"id":"dac37e71e22401baa83c5a092a3b383cfda0a5ce","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-05 18:48:05.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/dce9ef754aec4717bea33fb86e2f979e : SUCCESS in 3m 35s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/59a3d8cba2a6435a961c91b4a9f59fba : SUCCESS in 4m 47s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/913b0cc0710442efb8f2fa3ae161bbc7 : SUCCESS in 4m 17s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/9ee796b5885f4773b1d8ca6e0aff11b2 : SUCCESS in 1h 46m 07s","accounts_in_message":[],"_revision_number":4},{"id":"633584b112b28f157c765d2a691b1572567ba317","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-03-05 18:59:20.000000000","message":"Patch Set 4: Code-Review-1\n\nCan we dig into what\u0027s causing the failure mentioned in the commit message? I could get behind changing the inheritance order, but having to duplicate a helper method because we\u0027ve changed our inheritance (which... why is that even necessary?) is a hard no from me.","accounts_in_message":[],"_revision_number":4},{"id":"604952703a768420a649ac507e0b1d234a8d7ab1","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-05 19:22:51.000000000","message":"Patch Set 4:\n\n\u003e Patch Set 4: Code-Review-1\n\u003e \n\u003e Can we dig into what\u0027s causing the failure mentioned in the commit message? I could get behind changing the inheritance order, but having to duplicate a helper method because we\u0027ve changed our inheritance (which... why is that even necessary?) is a hard no from me.\nTwo things:\n1. Why did we change the inheritance order?\nThe current order processes the cleanUp/tearDown in a way that the user creating the secret is not deleting it and we get the policy error (permission denied)\n2. Why duplicate the helper method?\nNot just bc we\u0027ve changed our inheritance, with the order change, now os_primary creds are used to create the server and the self.server_client is unable to get the [\u0027server\u0027][\u0027OS-xxxx-host\u0027] instead it can be obtained directl from server json server[\u0027OS-xxx\u0027]\n\n#  using the existing get_server_xml\n(Pdb) server[\u0027server\u0027][\u0027OS-EXT-SRV-ATTR:host\u0027]\n*** KeyError: \u0027OS-EXT-SRV-ATTR:host\u0027","accounts_in_message":[],"_revision_number":4},{"id":"92a35154df2fea5ee761cb55f5bc333aee212f0a","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2021-03-05 20:19:56.000000000","message":"Patch Set 4: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":4},{"id":"b2ea03f07fd8c412b8176c80b5834962833bace1","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-08 14:41:37.000000000","message":"Patch Set 4:\n\n(3 comments)\n\nAlso, this job failed and should give u a better view as to why the order is important:\nhttps://zuul.opendev.org/t/openstack/build/b81ef269f2df4c95af63084db36afd40\n\nChange-Url: https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/779043","accounts_in_message":[],"_revision_number":4},{"id":"269ae75670992add8a44fca2d748ec6818d7f242","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-12 15:20:31.000000000","message":"Patch Set 4:\n\nCan we get this merged before the Barbican job?","accounts_in_message":[],"_revision_number":4},{"id":"6eed270c312c2ab708dc852064d298d533cc3308","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-16 14:24:09.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"3ceab3909378a4cb10142b2af9293d3da275fa73","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-16 14:51:27.000000000","message":"Patch Set 5:\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"d3b1a9e49a187c80d1dc87d3140653c44017d1df","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-16 16:31:27.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-pep8 https://zuul.opendev.org/t/openstack/build/78be32e3511d4a8c842fa474774d80b1 : SUCCESS in 4m 19s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/4e62c1022dab4af183084780da9b4c97 : SUCCESS in 4m 46s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/d3438c37e4ba4044aa005bb7be390af5 : SUCCESS in 4m 13s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/2128bc39ccf04144bd75cf0bb6455005 : FAILURE in 2h 02m 53s","accounts_in_message":[],"_revision_number":5},{"id":"4a384c51f4d45c24f483a569e81bf23a7bb524b0","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-16 16:35:26.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"c0fc3a83575afc0f3651865e7a557f6dd728a3c8","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-16 16:36:26.000000000","message":"Uploaded patch set 7: Commit message was updated.","accounts_in_message":[],"_revision_number":7},{"id":"1e49cd19756399f8769414c7b894e2c804fe024d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-16 18:38:25.000000000","message":"Patch Set 7: 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-pep8 https://zuul.opendev.org/t/openstack/build/db052dcab4c143408c39defcd2245a89 : SUCCESS in 5m 08s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/ce1a95120e2b4c858481790e3e657581 : SUCCESS in 4m 31s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/de8fdd025db842c99d21386baf2f7ffd : SUCCESS in 4m 21s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/ef4a356cc8024dcabbb3fb9eaea7af46 : FAILURE in 1h 56m 06s","accounts_in_message":[],"_revision_number":7},{"id":"4c3bdc6e4609d9bcb447da94533188cbc13e3b56","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-16 19:09:13.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"0c27ff0a7b3c8a50ff84431e1122b20317aa3667","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-16 19:10:06.000000000","message":"Patch Set 8:\n\n(1 comment)","accounts_in_message":[],"_revision_number":8},{"id":"5ec80d29c939da5bb58fa867308ca717ea423023","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-16 19:14:20.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"466a59c69885533ab6d1c54cc8eab170067efff9","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-16 21:09:38.000000000","message":"Patch Set 9: 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-pep8 https://zuul.opendev.org/t/openstack/build/8c3f5d6fb7454421824ccaf94cbc4d36 : SUCCESS in 3m 33s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/f046b1711f2f40beaa7372acc7272dc8 : SUCCESS in 3m 25s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/ee2a49c5584141599d9d9c069549789b : SUCCESS in 3m 22s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/ed7de6577cac480b89be4de659e043dc : FAILURE in 1h 53m 35s","accounts_in_message":[],"_revision_number":9},{"id":"58334778c1ced94c3b223e941f1abb34f3f9b04e","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-16 21:17:56.000000000","message":"Uploaded patch set 10.","accounts_in_message":[],"_revision_number":10},{"id":"3d6121ba3032d1946e7549208caca9ca08b44ca9","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-16 23:19:03.000000000","message":"Patch Set 10: 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-pep8 https://zuul.opendev.org/t/openstack/build/2c59b95164ad4d209662df5e81428234 : SUCCESS in 3m 27s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/447655ead25047a6a686b11619de895b : SUCCESS in 3m 49s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/d575470f60b94d119397f71b43f5b5df : SUCCESS in 3m 44s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/400a9c580fad49e5bc79223ec1d4ed0e : FAILURE in 1h 58m 15s","accounts_in_message":[],"_revision_number":10},{"id":"4b7e123085cade1c80bf5d2510cd136a86a90531","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-17 09:18:03.000000000","message":"Uploaded patch set 11.","accounts_in_message":[],"_revision_number":11},{"id":"5ed6276ec70d9809784d1ebec9d8f625ee54b67d","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-17 09:18:03.000000000","message":"Patch Set 11:\n\n(2 comments)","accounts_in_message":[],"_revision_number":11},{"id":"671b5583394e08ae6fac26c2ae33fad5447cc3b9","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-17 11:12:57.000000000","message":"Patch Set 11: Verified-1\n\n(1 comment)\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-pep8 https://zuul.opendev.org/t/openstack/build/9055e1d3f33245fbbc5d97c40e83c87b : FAILURE in 5m 43s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/98ad593c1745470a96dbfc1965a8689d : SUCCESS in 6m 26s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/90a66851a44a482c9b71d80445c78328 : SUCCESS in 3m 36s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/fe81fd2c8f7341f6a01badaccaea6386 : FAILURE in 1h 51m 48s","accounts_in_message":[],"_revision_number":11},{"id":"ca11adc40ab03eaaece5edec6eb63cc54d6569e2","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-17 11:25:15.000000000","message":"Uploaded patch set 12.","accounts_in_message":[],"_revision_number":12},{"id":"72cf4c1a96e0ae4468994c1981cf718731ce9624","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-17 11:25:55.000000000","message":"Patch Set 11:\n\nThis is passing both with fixed_key and Barbican now, I\u0027ll fix the pep8 issue but there\u0027s plenty wrong with this plugin outside of this change at the moment that needs to be fixed before this can land.","accounts_in_message":[],"_revision_number":11},{"id":"9b8c797bb40abcb374050b65dd77eb6dd090e042","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-17 12:29:55.000000000","message":"Patch Set 12:\n\n(1 comment)","accounts_in_message":[],"_revision_number":12},{"id":"697837338cdd8b9aa84c2b0dc7e1f63b5d72ba93","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-17 12:44:15.000000000","message":"Uploaded patch set 13: Patch Set 12 was rebased.","accounts_in_message":[],"_revision_number":13},{"id":"b8750bdc4835e677e8584b7abeb86c73a8e3abac","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-17 12:44:36.000000000","message":"Patch Set 13: Verified-1\n\nThis change depends on a change with an invalid configuration.","accounts_in_message":[],"_revision_number":13},{"id":"3d87ccac25f0013fdc353e82fbe38085135816eb","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-17 12:45:41.000000000","message":"Uploaded patch set 14: Patch Set 13 was rebased.","accounts_in_message":[],"_revision_number":14},{"id":"aee0493f0821afdc6ffd1cf0f04d0b3530ca010a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-17 15:15:23.000000000","message":"Patch Set 14: 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-pep8 https://zuul.opendev.org/t/openstack/build/a9c9a134cf0f485ebef25aa9c68285c2 : SUCCESS in 4m 15s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/c31c081e764140af9e5c8d6e2341adc5 : SUCCESS in 4m 37s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/cbff4b46aaad4a4caf14fa1dd3268210 : SUCCESS in 4m 07s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/ed51035bf80b4594ad4892b3c2662f26 : FAILURE in 2h 25m 27s","accounts_in_message":[],"_revision_number":14},{"id":"df23cd25d4680398726c9a0bba981ad02addc28e","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-17 16:05:48.000000000","message":"Patch Set 14:\n\nrecheck","accounts_in_message":[],"_revision_number":14},{"id":"3da7517485c80aaeab979dbb9ac6adde331c6436","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-17 17:57:20.000000000","message":"Patch Set 14:\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-pep8 https://zuul.opendev.org/t/openstack/build/39eea91f10bb491f96bb3f348a1daff3 : SUCCESS in 3m 56s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/2efdde2985de463faf6379913896f877 : SUCCESS in 3m 37s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/7ceed00523644c98bb2e587fb027b1ff : SUCCESS in 4m 07s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/95c6ffcd134b48de8c9a0574f914d689 : FAILURE in 1h 48m 56s","accounts_in_message":[],"_revision_number":14},{"id":"21ec7d9d75540ee39e7f7bb26277c1a41a6b578a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-18 12:06:43.000000000","message":"Uploaded patch set 15: Patch Set 14 was rebased.","accounts_in_message":[],"_revision_number":15},{"id":"56907189817837f6d06ff54be8b415268c6f1bdf","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-18 14:22:35.000000000","message":"Patch Set 15: 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-pep8 https://zuul.opendev.org/t/openstack/build/9a5bd48452b7480cbbcca4ea7c475842 : SUCCESS in 4m 48s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/3334d90e4c564716a740f4c7a71d5a49 : SUCCESS in 5m 22s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/f2d862f1f12846e8a6a8c8a0df4d38b1 : SUCCESS in 5m 08s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/7a1f17f5dacb4825964a32d42f83bb69 : FAILURE in 1h 50m 04s","accounts_in_message":[],"_revision_number":15},{"id":"edd648d9fab1cff57a8260f0eac7b676a5b32df3","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-18 19:57:32.000000000","message":"Patch Set 16: Patch Set 15 was rebased","accounts_in_message":[],"_revision_number":16},{"id":"9726f989e30b1851ad55d18e01663acfcb18095d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-18 22:02:04.000000000","message":"Patch Set 16: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/678c5d0f9e5c42e6b25204848233f818 : SUCCESS in 4m 42s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/857b9a76ac8f424b8fa4ba8f77b707f9 : SUCCESS in 4m 23s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/c9e323f781b24c5694a5a5d21317e5ef : SUCCESS in 5m 47s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/09518d745a124307a4380ab27f188eff : SUCCESS in 1h 57m 10s","accounts_in_message":[],"_revision_number":16},{"id":"3ced71cef3c6082f3528fb0ac975cc8c8b36ecf5","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2021-03-23 16:10:18.000000000","message":"Patch Set 16: Code-Review+1\n\nLGTM, would like to just run this against our downstream jobs to +2.","accounts_in_message":[],"_revision_number":16},{"id":"1568762302a76e2aa83740bec197c7847773a781","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2021-03-23 17:37:52.000000000","message":"Patch Set 16: Code-Review+1\n\n(3 comments)\n\n+1 also pending downstream results","accounts_in_message":[],"_revision_number":16},{"id":"d66c2968bb8af91032f2809bb12f676ad0b74a17","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-23 18:01:08.000000000","message":"Patch Set 16: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":16},{"id":"23443fea255ca6ddcbf3cc93dc8ea74122fe5f65","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2021-03-23 22:34:05.000000000","message":"Patch Set 16: Code-Review+2 Workflow+1\n\nDownstream looks good:\n(.tempest) (overcloud) [stack@undercloud-0 tempest-dir]$ tempest run --serial --regex whitebox_tempest_plugin.api.compute.\u0027(test_volume|test_virtio)\u0027\n{0} whitebox_tempest_plugin.api.compute.test_virtio_scsi_attach.VirtioSCSIDisk.test_attach_multiple_scsi_disks [68.453981s] ... ok\n{0} whitebox_tempest_plugin.api.compute.test_virtio_scsi_attach.VirtioSCSIDisk.test_boot_with_multiple_disks [54.359669s] ... ok\n{0} whitebox_tempest_plugin.api.compute.test_volume_encryption.TestQEMUVolumeEncryption.test_qemu_volume_encryption [31.226996s] ... ok\n{0} whitebox_tempest_plugin.api.compute.test_volume_negative.VolumesAdminNegativeTest.test_detach_failure [33.401967s] ... ok\n\n\u003d\u003d\u003d\u003d\u003d\u003d\nTotals\n\u003d\u003d\u003d\u003d\u003d\u003d\nRan: 4 tests in 236.3341 sec.\n - Passed: 4\n - Skipped: 0\n - Expected Fail: 0\n - Unexpected Success: 0\n - Failed: 0\nSum of execute time for each test: 187.4426 sec.\n\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nWorker Balance\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n - Worker 0 (4 tests) \u003d\u003e 0:03:56.334101","accounts_in_message":[],"_revision_number":16},{"id":"bac7a9da7637b211d8b21c685f376d654e529b8b","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-23 22:34:26.000000000","message":"Patch Set 16: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":16},{"id":"fc963b12a1f5d05f274b8ea09769a113ce246f52","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-24 00:24:03.000000000","message":"Patch Set 16: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/56cfb9fe65734127ba5094b3a2712d4c : SUCCESS in 4m 08s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/fb4f419e676e4c90807ec2e578d14f21 : SUCCESS in 4m 43s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/e41a25c734314c7e8749737e2ca5a20a : SUCCESS in 3m 41s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/7b245c677d2d4f869626426cdc119ec4 : SUCCESS in 1h 47m 13s","accounts_in_message":[],"_revision_number":16},{"id":"b3dcccbc0a82a8b559a954c1c3a379ffb1edf9e6","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-24 00:24:03.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":16}],"current_revision_number":16,"current_revision":"04e54d1ff91d139a7fc2e01bd4185e4e766d1646","revisions":{"faa7a3ada9b3471815d542e0f6db0678cc994368":{"kind":"REWORK","_number":1,"created":"2021-03-04 17:06:14.000000000","uploader":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"ref":"refs/changes/52/778752/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/1"}}},"commit":{"parents":[{"commit":"4a0382f40891b7056b5e435317db67ff72a7233f","subject":"Merge \"Added emulator thread pinning tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/4a0382f40891b7056b5e435317db67ff72a7233f"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"subject":"Change order of inheritance in TestQEMUVolumeEncrption class","message":"Change order of inheritance in TestQEMUVolumeEncrption class\n\nChange 00eb0f1 added the volume encryption tests based on a key\nmanager or fixed key present. Using a key manager like Barbican fails\nthis test with the following error \"Invalid volume: Unable to delete\nencryption key for volume.\" Changing the order of inheritance\nresolves this issue\n\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/faa7a3ada9b3471815d542e0f6db0678cc994368"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/faa7a3ada9b3471815d542e0f6db0678cc994368"}]},"branch":"refs/heads/master"},"c680cfb7df9a67abad860ad3882cf27e6c2490e5":{"kind":"REWORK","_number":2,"created":"2021-03-04 17:16:50.000000000","uploader":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"ref":"refs/changes/52/778752/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/2"}}},"commit":{"parents":[{"commit":"4a0382f40891b7056b5e435317db67ff72a7233f","subject":"Merge \"Added emulator thread pinning tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/4a0382f40891b7056b5e435317db67ff72a7233f"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:16:38.000000000","tz":-300},"subject":"WIP: Change order of inheritance in TestQEMUVolumeEncryption class","message":"WIP: Change order of inheritance in TestQEMUVolumeEncryption class\n\nChange 00eb0f1 added the volume encryption tests based on a key\nmanager or fixed key present. Using a key manager like Barbican fails\nthis test with the following error \"Invalid volume: Unable to delete\nencryption key for volume.\" Changing the order of inheritance\nresolves this issue\n\nAlso, fixed a typo TestQEMUVolumeEncr[y]ption\n\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/c680cfb7df9a67abad860ad3882cf27e6c2490e5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/c680cfb7df9a67abad860ad3882cf27e6c2490e5"}]},"branch":"refs/heads/master"},"419b57825f4ab3b118008df5ef1b817580e495fd":{"kind":"REWORK","_number":3,"created":"2021-03-04 18:59:00.000000000","uploader":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"ref":"refs/changes/52/778752/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/3"}}},"commit":{"parents":[{"commit":"befaafc1c24a8e9f68f75850865e5ae4a8a93cd1","subject":"Add skip check to volume encryption tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/befaafc1c24a8e9f68f75850865e5ae4a8a93cd1"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 18:58:40.000000000","tz":-300},"subject":"Change order of inheritance in TestQEMUVolumeEncryption class","message":"Change order of inheritance in TestQEMUVolumeEncryption class\n\nChange 00eb0f1 added the volume encryption tests based on a key\nmanager or fixed key present. Using a key manager like Barbican fails\nthis test with the following error \"Invalid volume: Unable to delete\nencryption key for volume.\" Changing the order of inheritance\nresolves this issue\n\nAlso, fixed a typo TestQEMUVolumeEncr[y]ption\n\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/419b57825f4ab3b118008df5ef1b817580e495fd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/419b57825f4ab3b118008df5ef1b817580e495fd"}]},"branch":"refs/heads/master"},"cd78ef2841f0418b5016b88c496e0df97b09adca":{"kind":"REWORK","_number":4,"created":"2021-03-05 16:56:15.000000000","uploader":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"ref":"refs/changes/52/778752/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/4"}}},"commit":{"parents":[{"commit":"befaafc1c24a8e9f68f75850865e5ae4a8a93cd1","subject":"Add skip check to volume encryption tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/befaafc1c24a8e9f68f75850865e5ae4a8a93cd1"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-05 16:53:43.000000000","tz":-300},"subject":"Change order of inheritance in TestQEMUVolumeEncryption class","message":"Change order of inheritance in TestQEMUVolumeEncryption class\n\nChange 00eb0f1 added the volume encryption tests based on a key\nmanager or fixed key present. Using a key manager like Barbican fails\nthis test with the following error \"Invalid volume: Unable to delete\nencryption key for volume.\" Changing the order of inheritance\nresolves this issue and as a result redefining get_server_xml method\n\nAlso, fixed a typo TestQEMUVolumeEncr[y]ption\n\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/cd78ef2841f0418b5016b88c496e0df97b09adca"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/cd78ef2841f0418b5016b88c496e0df97b09adca"}]},"branch":"refs/heads/master"},"81248b1cf7d457937a63065ef5b96d9a442cabf6":{"kind":"REWORK","_number":5,"created":"2021-03-16 14:24:09.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/5"}}},"commit":{"parents":[{"commit":"befaafc1c24a8e9f68f75850865e5ae4a8a93cd1","subject":"Add skip check to volume encryption tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/befaafc1c24a8e9f68f75850865e5ae4a8a93cd1"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-16 14:22:37.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\ninternally and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/81248b1cf7d457937a63065ef5b96d9a442cabf6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/81248b1cf7d457937a63065ef5b96d9a442cabf6"}]},"branch":"refs/heads/master"},"70d94691e167b05c28db281d39d2931833cdc9dd":{"kind":"REWORK","_number":6,"created":"2021-03-16 16:35:26.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/6"}}},"commit":{"parents":[{"commit":"befaafc1c24a8e9f68f75850865e5ae4a8a93cd1","subject":"Add skip check to volume encryption tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/befaafc1c24a8e9f68f75850865e5ae4a8a93cd1"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-16 16:35:17.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\ninternally and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/70d94691e167b05c28db281d39d2931833cdc9dd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/70d94691e167b05c28db281d39d2931833cdc9dd"}]},"branch":"refs/heads/master"},"8b6fcf441e8dc405357ab9e15a803df1e81be773":{"kind":"NO_CODE_CHANGE","_number":7,"created":"2021-03-16 16:36:26.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/7"}}},"commit":{"parents":[{"commit":"befaafc1c24a8e9f68f75850865e5ae4a8a93cd1","subject":"Add skip check to volume encryption tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/befaafc1c24a8e9f68f75850865e5ae4a8a93cd1"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-16 16:35:59.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\nentirely and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/8b6fcf441e8dc405357ab9e15a803df1e81be773"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/8b6fcf441e8dc405357ab9e15a803df1e81be773"}]},"branch":"refs/heads/master"},"70e0a36f49a225ba422732baadbed23e48b45026":{"kind":"REWORK","_number":8,"created":"2021-03-16 19:09:13.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/8"}}},"commit":{"parents":[{"commit":"befaafc1c24a8e9f68f75850865e5ae4a8a93cd1","subject":"Add skip check to volume encryption tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/befaafc1c24a8e9f68f75850865e5ae4a8a93cd1"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-16 19:08:59.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\nentirely and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/70e0a36f49a225ba422732baadbed23e48b45026"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/70e0a36f49a225ba422732baadbed23e48b45026"}]},"branch":"refs/heads/master"},"4b4b4754aa28c29ddfa2145eef35e0a9178e3744":{"kind":"REWORK","_number":9,"created":"2021-03-16 19:14:20.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/9"}}},"commit":{"parents":[{"commit":"befaafc1c24a8e9f68f75850865e5ae4a8a93cd1","subject":"Add skip check to volume encryption tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/befaafc1c24a8e9f68f75850865e5ae4a8a93cd1"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-16 19:13:20.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\nentirely and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/4b4b4754aa28c29ddfa2145eef35e0a9178e3744"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/4b4b4754aa28c29ddfa2145eef35e0a9178e3744"}]},"branch":"refs/heads/master"},"840d6bd9cffcc017f5ee48fa44cbc2d4bbb05ab2":{"kind":"REWORK","_number":10,"created":"2021-03-16 21:17:56.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/10"}}},"commit":{"parents":[{"commit":"befaafc1c24a8e9f68f75850865e5ae4a8a93cd1","subject":"Add skip check to volume encryption tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/befaafc1c24a8e9f68f75850865e5ae4a8a93cd1"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-16 21:17:36.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\nentirely and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/840d6bd9cffcc017f5ee48fa44cbc2d4bbb05ab2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/840d6bd9cffcc017f5ee48fa44cbc2d4bbb05ab2"}]},"branch":"refs/heads/master"},"4d5a8db502e7b2cfeff0aac009d90d777d335242":{"kind":"REWORK","_number":11,"created":"2021-03-17 09:18:03.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/11","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/11","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/11 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/11 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/11 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/11"}}},"commit":{"parents":[{"commit":"befaafc1c24a8e9f68f75850865e5ae4a8a93cd1","subject":"Add skip check to volume encryption tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/befaafc1c24a8e9f68f75850865e5ae4a8a93cd1"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-17 09:17:39.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\nentirely and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/4d5a8db502e7b2cfeff0aac009d90d777d335242"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/4d5a8db502e7b2cfeff0aac009d90d777d335242"}]},"branch":"refs/heads/master"},"17dc555d0ed8c12b7e30b0a19a1456c0e713c73a":{"kind":"REWORK","_number":12,"created":"2021-03-17 11:25:15.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/12","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/12","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/12 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/12 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/12 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/12"}}},"commit":{"parents":[{"commit":"befaafc1c24a8e9f68f75850865e5ae4a8a93cd1","subject":"Add skip check to volume encryption tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/befaafc1c24a8e9f68f75850865e5ae4a8a93cd1"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-17 11:25:05.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\nentirely and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/17dc555d0ed8c12b7e30b0a19a1456c0e713c73a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/17dc555d0ed8c12b7e30b0a19a1456c0e713c73a"}]},"branch":"refs/heads/master"},"2f320e947b55f7dce7129ceb80c140dcf0e07382":{"kind":"TRIVIAL_REBASE","_number":13,"created":"2021-03-17 12:44:15.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/13","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/13","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/13 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/13 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/13 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/13"}}},"commit":{"parents":[{"commit":"5ce1a5619d6e81f34d3233a82a72fd433468541c","subject":"zuul: Ensure subnode n-cpu is correctly configured","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/5ce1a5619d6e81f34d3233a82a72fd433468541c"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-17 12:43:37.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\nentirely and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/2f320e947b55f7dce7129ceb80c140dcf0e07382"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/2f320e947b55f7dce7129ceb80c140dcf0e07382"}]},"branch":"refs/heads/master"},"6a1bdb7754f6c59d44c5914790bdefa9aa0b963e":{"kind":"TRIVIAL_REBASE","_number":14,"created":"2021-03-17 12:45:41.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/14","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/14","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/14 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/14 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/14 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/14"}}},"commit":{"parents":[{"commit":"9f90797c39d52129d917cb28bbe009e2765a0da5","subject":"zuul: Ensure subnode n-cpu is correctly configured","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/9f90797c39d52129d917cb28bbe009e2765a0da5"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-17 12:45:27.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\nentirely and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/6a1bdb7754f6c59d44c5914790bdefa9aa0b963e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/6a1bdb7754f6c59d44c5914790bdefa9aa0b963e"}]},"branch":"refs/heads/master"},"77723aa3ca65e6a29fe95e3d8ecb74c33fad7f02":{"kind":"TRIVIAL_REBASE","_number":15,"created":"2021-03-18 12:06:43.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/15","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/15","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/15 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/15 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/15 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/15"}}},"commit":{"parents":[{"commit":"948dffad74c2e0ab185f03eedf4220941d7463bc","subject":"test_virtio_scsi_attach: Improve the volume id lookups","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/948dffad74c2e0ab185f03eedf4220941d7463bc"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-18 12:06:29.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\nentirely and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/77723aa3ca65e6a29fe95e3d8ecb74c33fad7f02"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/77723aa3ca65e6a29fe95e3d8ecb74c33fad7f02"}]},"branch":"refs/heads/master"},"04e54d1ff91d139a7fc2e01bd4185e4e766d1646":{"kind":"TRIVIAL_REBASE","_number":16,"created":"2021-03-18 19:57:32.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/52/778752/16","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/52/778752/16","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/16 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/16 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/16 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/52/778752/16"}}},"commit":{"parents":[{"commit":"c6252d4e2ad4b91ff510287354c144ffb599a99e","subject":"test_virtio_scsi_attach: Improve the volume id lookups","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/c6252d4e2ad4b91ff510287354c144ffb599a99e"}]}],"author":{"name":"Archit Modi","email":"amodi@redhat.com","date":"2021-03-04 17:00:06.000000000","tz":-300},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-18 19:57:32.000000000","tz":0},"subject":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption","message":"Remove manager.EncryptionScenarioTest from TestQEMUVolumeEncryption\n\nIntroduced by Iceca320e5b93f7c64318648a94e775cc5d321c06 this test class\npreviously inherited from both compute api base and scenario manager\ntest classes.\n\nWhen executed against an environment with Barbican deployed this would\nlead to failures to delete secrets as the clients used by both of these\ninherited classes differed, resulting in an unauthorised request to\ndelete a secret being made that failed.\n\nTo eliminate this the use of the scenario manager base class is removed\nentirely and the logic required to create a volume type, encrypted\nvolume type and finally an encrypted volume is copied over verbatim. A\nTODO is left in place to have this code refactored into a shared place\nsomewhere within Tempest in the future so both the API and Scenario\nclasses can have a unified approach to this.\n\nCo-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e\nChange-Id: Id91887bb5d75c36c9a3c6fa4ea297c20b442fd4a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/04e54d1ff91d139a7fc2e01bd4185e4e766d1646"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/04e54d1ff91d139a7fc2e01bd4185e4e766d1646"}]},"branch":"refs/heads/master","description":"Rebase"}},"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":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"}}]}],"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":{}}}]}