)]}'
{"id":"openstack%2Fwhitebox-tempest-plugin~781431","triplet_id":"openstack%2Fwhitebox-tempest-plugin~master~I2e53a2ded276c6f7f3d59c95904a0ee195ec2fab","project":"openstack/whitebox-tempest-plugin","branch":"master","topic":"change-781431-1","hashtags":[],"change_id":"I2e53a2ded276c6f7f3d59c95904a0ee195ec2fab","subject":"test_virtio_scsi_attach: Improve the volume id lookups","status":"MERGED","created":"2021-03-18 12:06:43.000000000","updated":"2021-03-22 20:00:44.000000000","submitted":"2021-03-22 20:00:44.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":3,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"781431-change-781431-1","meta_rev_id":"1acd4725e04f9f75a108ff24e7b1bfcf71d255e8","_number":781431,"virtual_id_number":781431,"owner":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},{"value":0,"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},{"value":0,"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":0,"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-03-22 20:00:44.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"recommended":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"all":[{"value":0,"date":"2021-03-19 18:37:00.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},{"value":1,"date":"2021-03-18 15:36:10.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},{"value":1,"date":"2021-03-18 15:14:30.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":2,"date":"2021-03-19 13:39:29.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"all":[{"value":0,"date":"2021-03-19 18:37:00.000000000","_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},{"value":0,"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},{"value":0,"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":1,"date":"2021-03-19 13:40:20.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":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":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-18 12:56:12.000000000","updated_by":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"reviewer":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"state":"REVIEWER"},{"updated":"2021-03-18 13:04:56.000000000","updated_by":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"reviewer":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"state":"REVIEWER"},{"updated":"2021-03-18 13:04:56.000000000","updated_by":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"reviewer":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"state":"REVIEWER"},{"updated":"2021-03-18 14:26:39.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":"2ad1f373c0fdb22d2480748e991927e74489b350","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 1.","accounts_in_message":[],"_revision_number":1},{"id":"1ad781b17413d4d392c7459b4cfa552809d06f9e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-18 14:26:39.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/fe476152dfef4e008561cf41203bd9ca : SUCCESS in 7m 11s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/46dafcbd12114323acf7924849b99e69 : SUCCESS in 5m 54s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/0f02121d1d954be9ae4b1069aa59e7b4 : SUCCESS in 6m 00s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/0d23a2da8b464ff595e08e0c97fd7c1b : FAILURE in 1h 53m 44s","accounts_in_message":[],"_revision_number":1},{"id":"604a3a3d998c049ac080632a66261290a9fc4aed","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2021-03-18 14:59:40.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"09f259c224effa33d8b1571c8639e34c309af966","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-18 15:14:30.000000000","message":"Patch Set 2: Code-Review+1\n\nLGTM now thanks James and apologies for not running this against an actual env before pushing!","accounts_in_message":[],"_revision_number":2},{"id":"8cd6b30ac33b5c191c3b41904987d0f45345d4a2","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-18 15:33:13.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"026abcc40edf93bde7e5b5f392d61e4ac636589e","author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"date":"2021-03-18 15:36:10.000000000","message":"Patch Set 2: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"9e4896d3668ec66d07333b3644a98f6a5d185b67","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-18 16:49:19.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/eb8bb454506a4d8e8b80901223924788 : SUCCESS in 4m 30s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/a44781d8ceab4f98be7d142622c2ea71 : SUCCESS in 4m 00s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/8b9620d610f8407fa94ccb0b5b4895a8 : SUCCESS in 3m 53s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/48ded38a45964e02a69827729c676ed7 : SUCCESS in 1h 41m 56s","accounts_in_message":[],"_revision_number":2},{"id":"2f4fe16c4b55de36908bd2f2fa17c45ac916aeb0","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2021-03-18 16:58:38.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"9b80870275d4a735b7f940de9e6fc2f65c867b53","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2021-03-19 13:39:29.000000000","message":"Patch Set 2: Code-Review+2\n\nLGTM, thanks for the improvements Lee!","accounts_in_message":[],"_revision_number":2},{"id":"af51883fc4c8a5bd36efc293069bb7b25fc34562","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2021-03-19 13:40:20.000000000","message":"Patch Set 2: Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"3223023f24bf3bb779f88f35fbac932c3c4866dc","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-03-19 18:24:02.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\n*kicks Zuul*","accounts_in_message":[],"_revision_number":2},{"id":"4528abb4611d94007a35857684c953a79d3ae860","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-03-19 18:37:00.000000000","message":"Patch Set 2: -Code-Review -Workflow\n\n*facepalm* the patch below this one isn\u0027t approved","accounts_in_message":[],"_revision_number":2},{"id":"fdd1efb0744dcdcf92fd2dfc4e945092c3ae9ba5","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-19 20:09:31.000000000","message":"Patch Set 2:\n\n\u003e Patch Set 2: -Code-Review -Workflow\n\u003e \n\u003e *facepalm* the patch below this one isn\u0027t approved\n\nFWIW you don\u0027t need to do this, clean rebases retain the votes.","accounts_in_message":[],"_revision_number":2},{"id":"91f17e588e80aab91165517e63712c99d23c3d47","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-22 17:53:34.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"bce085228d86337535935028792f5ef4cb97ecb2","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-22 20:00:44.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/d0ce41d7dcc9432981e22a6a7243e96d : SUCCESS in 4m 22s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/13c72fc995f14a1188ee9e7f5dcef5a6 : SUCCESS in 4m 31s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/ad1ee6ef288f4dc3b84fc478b6486821 : SUCCESS in 5m 10s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/7ddab0ff880d47e68f169674fa3b3bdd : SUCCESS in 2h 03m 04s","accounts_in_message":[],"_revision_number":2},{"id":"1acd4725e04f9f75a108ff24e7b1bfcf71d255e8","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-22 20:00:44.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"c6252d4e2ad4b91ff510287354c144ffb599a99e","revisions":{"948dffad74c2e0ab185f03eedf4220941d7463bc":{"kind":"REWORK","_number":1,"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/31/781431/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/31/781431/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/31/781431/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/31/781431/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/31/781431/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/31/781431/1"}}},"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":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-18 12:04:12.000000000","tz":0},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-18 12:06:24.000000000","tz":0},"subject":"test_virtio_scsi_attach: Improve the volume id lookups","message":"test_virtio_scsi_attach: Improve the volume id lookups\n\nThis lookup previously looked at *all* volumes the specific user had\naccess to within the environment. A better approach is just to look for\nspecific volume ids that are attached to our test instance.\n\nChange-Id: I2e53a2ded276c6f7f3d59c95904a0ee195ec2fab\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/948dffad74c2e0ab185f03eedf4220941d7463bc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/948dffad74c2e0ab185f03eedf4220941d7463bc"}]},"branch":"refs/heads/master"},"c6252d4e2ad4b91ff510287354c144ffb599a99e":{"kind":"REWORK","_number":2,"created":"2021-03-18 14:59:40.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/31/781431/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/31/781431/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/31/781431/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/31/781431/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/31/781431/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/31/781431/2"}}},"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":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-18 12:04:12.000000000","tz":0},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2021-03-18 14:47:41.000000000","tz":0},"subject":"test_virtio_scsi_attach: Improve the volume id lookups","message":"test_virtio_scsi_attach: Improve the volume id lookups\n\nThis lookup previously looked at *all* volumes the specific user had\naccess to within the environment. A better approach is just to look for\nspecific volume ids that are attached to our test instance.\n\nChange-Id: I2e53a2ded276c6f7f3d59c95904a0ee195ec2fab\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/c6252d4e2ad4b91ff510287354c144ffb599a99e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/c6252d4e2ad4b91ff510287354c144ffb599a99e"}]},"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":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":{}}}]}