)]}'
{"id":"openstack%2Fmanila-tempest-plugin~736195","triplet_id":"openstack%2Fmanila-tempest-plugin~master~I7607019e47ce9e8c6584f69c2b2d3228873f24df","project":"openstack/manila-tempest-plugin","branch":"master","topic":"bug/1883795","hashtags":[],"change_id":"I7607019e47ce9e8c6584f69c2b2d3228873f24df","subject":"Stop forcing creation of \"isolated\" credentials","status":"MERGED","created":"2020-06-17 12:47:43.000000000","updated":"2020-07-01 00:46:15.000000000","submitted":"2020-07-01 00:46:15.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":7,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"736195-1593564375826-f3086ce0","meta_rev_id":"965e8e5a3a0575e6b8bc31a744a89c61a47e284f","_number":736195,"virtual_id_number":736195,"owner":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":12017,"name":"DellEMC VNX CI","email":"emc.vnx.ci@emc.com","username":"emc-vnx-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},{"value":0,"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},{"value":0,"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},{"value":0,"date":"2020-06-30 10:49:14.000000000","_account_id":25243,"name":"NetApp CI","username":"NetApp-ci","inactive":true,"tags":["SERVICE_USER"]},{"value":0,"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"value":0,"_account_id":21884,"name":"DellEMC Unity CI","email":"emc.unity.ci@emc.com","username":"emc-unity-ci"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-07-01 00:46:15.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"all":[{"value":0,"_account_id":12017,"name":"DellEMC VNX CI","email":"emc.vnx.ci@emc.com","username":"emc-vnx-ci","tags":["SERVICE_USER"]},{"value":2,"date":"2020-06-30 21:01:43.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},{"value":0,"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},{"value":0,"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},{"value":0,"_account_id":25243,"name":"NetApp CI","username":"NetApp-ci","inactive":true,"tags":["SERVICE_USER"]},{"value":2,"date":"2020-06-30 23:25:32.000000000","_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"value":0,"_account_id":21884,"name":"DellEMC Unity CI","email":"emc.unity.ci@emc.com","username":"emc-unity-ci"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"all":[{"value":0,"_account_id":12017,"name":"DellEMC VNX CI","email":"emc.vnx.ci@emc.com","username":"emc-vnx-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},{"value":0,"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},{"value":0,"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},{"value":0,"_account_id":25243,"name":"NetApp CI","username":"NetApp-ci","inactive":true,"tags":["SERVICE_USER"]},{"value":1,"date":"2020-06-30 23:25:32.000000000","_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"value":0,"_account_id":21884,"name":"DellEMC Unity CI","email":"emc.unity.ci@emc.com","username":"emc-unity-ci"},{"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":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"_account_id":12017,"name":"DellEMC VNX CI","email":"emc.vnx.ci@emc.com","username":"emc-vnx-ci","tags":["SERVICE_USER"]},{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},{"_account_id":21884,"name":"DellEMC Unity CI","email":"emc.unity.ci@emc.com","username":"emc-unity-ci"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":25243,"name":"NetApp CI","username":"NetApp-ci","inactive":true,"tags":["SERVICE_USER"]},{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-06-23 08:45:33.000000000","updated_by":{"_account_id":12017,"name":"DellEMC VNX CI","email":"emc.vnx.ci@emc.com","username":"emc-vnx-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":12017,"name":"DellEMC VNX CI","email":"emc.vnx.ci@emc.com","username":"emc-vnx-ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2020-06-23 14:22:19.000000000","updated_by":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"reviewer":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"state":"REVIEWER"},{"updated":"2020-06-29 23:45:17.000000000","updated_by":{"_account_id":21884,"name":"DellEMC Unity CI","email":"emc.unity.ci@emc.com","username":"emc-unity-ci"},"reviewer":{"_account_id":21884,"name":"DellEMC Unity CI","email":"emc.unity.ci@emc.com","username":"emc-unity-ci"},"state":"REVIEWER"},{"updated":"2020-06-30 10:49:14.000000000","updated_by":{"_account_id":25243,"name":"NetApp CI","username":"NetApp-ci","inactive":true,"tags":["SERVICE_USER"]},"reviewer":{"_account_id":25243,"name":"NetApp CI","username":"NetApp-ci","inactive":true,"tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2020-06-30 21:01:43.000000000","updated_by":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"reviewer":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"state":"REVIEWER"},{"updated":"2020-06-30 23:25:32.000000000","updated_by":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"reviewer":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"state":"REVIEWER"},{"updated":"2020-07-01 00:46:15.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":"4fa57264cc989a8b49b20cf3a6a59fb6fe723599","author":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"date":"2020-06-17 12:47:43.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"3840bbfae8649148322c536c5f0d5e18d5ac04ab","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-17 14:57:18.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/89f6c53a089448ac998c5fdcb3c0733e : SUCCESS in 3m 13s\n- manila-tempest-plugin-lvm https://zuul.opendev.org/t/openstack/build/7bf74d5370454ab9a34fd53bc42c03e1 : SUCCESS in 1h 10m 29s\n- manila-tempest-minimal-dsvm-dummy https://zuul.opendev.org/t/openstack/build/de76eda6f46c44e689d3d66d3597e799 : SUCCESS in 46m 43s\n- manila-tempest-plugin-zfsonlinux https://zuul.opendev.org/t/openstack/build/7c23829962bf43c8936b16cc3eb69dc8 : SUCCESS in 1h 20m 59s (non-voting)\n- manila-tempest-plugin-cephfs-native https://zuul.opendev.org/t/openstack/build/8adcbcf14df54f48921c012facf02db5 : FAILURE in 1h 28m 06s (non-voting)\n- manila-tempest-plugin-container https://zuul.opendev.org/t/openstack/build/a08fe6a1969d4406a239ec6716955f2f : SUCCESS in 54m 32s (non-voting)\n- manila-tempest-plugin-generic https://zuul.opendev.org/t/openstack/build/865183c338fb448aa74e1fc3935c7b62 : FAILURE in 2h 07m 40s (non-voting)\n- manila-tempest-minimal-dsvm-cephfs-nfs https://zuul.opendev.org/t/openstack/build/a6947637cfb34c9e9461861df638e8a9 : SUCCESS in 1h 15m 42s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"b9ad43388e50b1967b5c76f06c477afd76a6bebd","author":{"_account_id":25243,"name":"NetApp CI","username":"NetApp-ci","inactive":true,"tags":["SERVICE_USER"]},"date":"2020-06-17 15:11:31.000000000","message":"Patch Set 1:\n\nBuild failed.\n\n- manila-cDOT-manila-tempest-plugin-ss https://logs.openstack.netapp.com/logs/95/736195/1/upstream-check/manila-cDOT-manila-tempest-plugin-ss/b65bf5e/ : FAILURE in 13m 54s\n- manila-cDOT-manila-tempest-plugin-no-ss https://logs.openstack.netapp.com/logs/95/736195/1/upstream-check/manila-cDOT-manila-tempest-plugin-no-ss/cfa66c3/ : FAILURE in 13m 13s\n\nComment \u0027run-NetApp CI\u0027 to rerun these tests. https://wiki.openstack.org/wiki/ThirdPartySystems/NetApp_CI","accounts_in_message":[],"_revision_number":1},{"id":"50f1e7a3199a910cdbf41850cee80dc9381bf72d","author":{"_account_id":12017,"name":"DellEMC VNX CI","email":"emc.vnx.ci@emc.com","username":"emc-vnx-ci","tags":["SERVICE_USER"]},"date":"2020-06-17 15:54:09.000000000","message":"Patch Set 1:\n\nBuild succeed.\n\n- DellEMC_VNX_Manila http://publiclogs.emc.com/95/736195/1/check/DellEMC_VNX_Manila/93bbc2b/DellEMC_VNX_Manila/49 : SUCCESS in 56m 36s\n\nLeave a comment with \u0027run-DellEMC VNX CI\u0027 to trigger a recheck. For more information about this CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_VNX_CI","accounts_in_message":[],"_revision_number":1},{"id":"212e886112f89fd3ea25ada83a928666553f00b3","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2020-06-19 23:13:54.000000000","message":"Patch Set 1:\n\n(5 comments)\n\nHmmm, this change made me look up this whole test credential business with tempest. It feels like we didn\u0027t account for tempest\u0027s ability to provide us with different tenants for the same test class! Which is why we keep spawning new isolated credentials to do stuff. However, if we relied on the tenants tempest creates with test case annotations, we don\u0027t need to create any more isolated creds!\n\nLucio: do you mind me updating this patch? I\u0027ll commit some cleanup and add you as co-author.","accounts_in_message":[],"_revision_number":1},{"id":"47d34a9b964820f71109636b8eddb51b6498f235","author":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"date":"2020-06-20 00:03:26.000000000","message":"Patch Set 1:\n\n(1 comment)\n\n\u003e do you mind me updating this patch?\n\nPlease go ahead, thank you!","accounts_in_message":[],"_revision_number":1},{"id":"2434adf4e0f77df995e1c61d24ccd6bcaa921db1","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2020-06-20 00:12:34.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"1aecced2219fa7bd499077741007a92723f4386d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-20 02:10:41.000000000","message":"Patch Set 2: 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/80f3f8750aa3434d9a59d5526187242e : FAILURE in 4m 19s\n- manila-tempest-plugin-lvm https://zuul.opendev.org/t/openstack/build/adb70d0d38b640128e2318f17204d4f6 : SUCCESS in 53m 20s\n- manila-tempest-minimal-dsvm-dummy https://zuul.opendev.org/t/openstack/build/523e8697e1b243998a75d6a2240ff67d : FAILURE in 54m 02s\n- manila-tempest-plugin-zfsonlinux https://zuul.opendev.org/t/openstack/build/43bc6999bedf49c8b665b720f9d99343 : SUCCESS in 1h 12m 07s (non-voting)\n- manila-tempest-plugin-cephfs-native https://zuul.opendev.org/t/openstack/build/db951a47fa4b444ba0ac0569013b1786 : FAILURE in 1h 26m 24s (non-voting)\n- manila-tempest-plugin-container https://zuul.opendev.org/t/openstack/build/ac7fc29af56845c39a8dd12508e4649f : FAILURE in 36m 19s (non-voting)\n- manila-tempest-plugin-generic https://zuul.opendev.org/t/openstack/build/a3ca9baafbe3440a94f332ff25be69d6 : FAILURE in 1h 56m 40s (non-voting)\n- manila-tempest-minimal-dsvm-cephfs-nfs https://zuul.opendev.org/t/openstack/build/d8eb0aa8ded64065993ca60396150d14 : FAILURE in 51m 17s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"b83b9b80d24fa57b598e51b9fbaacc1bfbf657c5","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2020-06-23 05:49:16.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"3f8f189093597f5f0f85794fd676efa331a05d74","author":{"_account_id":21884,"name":"DellEMC Unity CI","email":"emc.unity.ci@emc.com","username":"emc-unity-ci"},"date":"2020-06-23 07:32:35.000000000","message":"Patch Set 3:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- DellEMC_Unity_Manila_dhss_true http://publiclogs.emc.com/95/736195/3/check/DellEMC_Unity_Manila_dhss_true/1187103/DellEMC_Unity_Manila_dhss_true/83 : FAILURE in 1h 43m 10s\n- DellEMC_Unity_Manila_dhss_false http://publiclogs.emc.com/95/736195/3/check/DellEMC_Unity_Manila_dhss_false/855be98/DellEMC_Unity_Manila_dhss_false/82 : FAILURE in 47m 39s\n\nLeave a comment with \u0027run-DellEMC Unity CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_Unity_CI","accounts_in_message":[],"_revision_number":3},{"id":"9ab09714c0a3ea953c55edc251708817e166290a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-23 07:45:20.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/98b42ff172174b0694bb4c7b431a8f61 : SUCCESS in 3m 53s\n- manila-tempest-plugin-lvm https://zuul.opendev.org/t/openstack/build/574b53b7ecbb4081ba53302f475cddb0 : SUCCESS in 1h 03m 25s\n- manila-tempest-minimal-dsvm-dummy https://zuul.opendev.org/t/openstack/build/4157592f29a743018104bc88f73217ce : SUCCESS in 47m 34s\n- manila-tempest-plugin-zfsonlinux https://zuul.opendev.org/t/openstack/build/975025297dbc40e19cc8b92703880de1 : FAILURE in 1h 32m 33s (non-voting)\n- manila-tempest-plugin-cephfs-native https://zuul.opendev.org/t/openstack/build/611317b0836c4fd1bc7a6b516e91d427 : SUCCESS in 52m 53s (non-voting)\n- manila-tempest-plugin-container https://zuul.opendev.org/t/openstack/build/8845c13a6dd045c8bcb45908a2483def : SUCCESS in 1h 10m 39s (non-voting)\n- manila-tempest-plugin-generic https://zuul.opendev.org/t/openstack/build/ed425d203c20440094d44e21a0cebe88 : FAILURE in 1h 55m 33s (non-voting)\n- manila-tempest-minimal-dsvm-cephfs-nfs https://zuul.opendev.org/t/openstack/build/93db199cf012487db76a7a0ea96fd49f : FAILURE in 45m 41s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"2a0ecf6e0589810a506246679c410bd688b20ef0","author":{"_account_id":12017,"name":"DellEMC VNX CI","email":"emc.vnx.ci@emc.com","username":"emc-vnx-ci","tags":["SERVICE_USER"]},"date":"2020-06-23 08:45:33.000000000","message":"Patch Set 3:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- DellEMC_VNX_Manila http://publiclogs.emc.com/95/736195/3/check/DellEMC_VNX_Manila/0e2588a/DellEMC_VNX_Manila/54 : FAILURE in 59m 59s\n\nLeave a comment with \u0027run-DellEMC VNX CI\u0027 to trigger a recheck. For more information about this CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_VNX_CI","accounts_in_message":[],"_revision_number":3},{"id":"20f8f5e6af20712d69a2d8671303dbfbb5a26829","author":{"_account_id":21884,"name":"DellEMC Unity CI","email":"emc.unity.ci@emc.com","username":"emc-unity-ci"},"date":"2020-06-23 09:20:43.000000000","message":"Patch Set 3:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- DellEMC_Unity_Manila_dhss_true http://publiclogs.emc.com/95/736195/3/check/DellEMC_Unity_Manila_dhss_true/b3fc74a/DellEMC_Unity_Manila_dhss_true/84 : FAILURE in 1h 35m 14s\n- DellEMC_Unity_Manila_dhss_false http://publiclogs.emc.com/95/736195/3/check/DellEMC_Unity_Manila_dhss_false/ef33736/DellEMC_Unity_Manila_dhss_false/83 : FAILURE in 47m 44s\n\nLeave a comment with \u0027run-DellEMC Unity CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_Unity_CI","accounts_in_message":[],"_revision_number":3},{"id":"eea85bcc00544d05806765f2fdcce44bf063e993","author":{"_account_id":25243,"name":"NetApp CI","username":"NetApp-ci","inactive":true,"tags":["SERVICE_USER"]},"date":"2020-06-23 09:56:36.000000000","message":"Patch Set 3:\n\nBuild failed.\n\n- manila-cDOT-manila-tempest-plugin-ss https://logs.openstack.netapp.com/logs/95/736195/3/upstream-check/manila-cDOT-manila-tempest-plugin-ss/de6a192/ : SUCCESS in 2h 05m 31s\n- manila-cDOT-manila-tempest-plugin-no-ss https://logs.openstack.netapp.com/logs/95/736195/3/upstream-check/manila-cDOT-manila-tempest-plugin-no-ss/224cdcd/ : FAILURE in 1m 10s\n\nComment \u0027run-NetApp CI\u0027 to rerun these tests. https://wiki.openstack.org/wiki/ThirdPartySystems/NetApp_CI","accounts_in_message":[],"_revision_number":3},{"id":"0cba659931234157c6357412776c50f4e69dee7d","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2020-06-23 14:22:19.000000000","message":"Patch Set 3: Workflow-1\n\nLooks like some more modifications are necessary - quota tests are now failing","accounts_in_message":[],"_revision_number":3},{"id":"a25a8dd564059b994b6ac8530bcd7f0a0ba5daf8","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2020-06-29 21:24:26.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"7bd0d512cc79aa8c14327585ae93b7d6bfae25c2","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2020-06-29 21:37:57.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"d21a1654c54f144377ebb735051ea6a3a2cbebd7","author":{"_account_id":21884,"name":"DellEMC Unity CI","email":"emc.unity.ci@emc.com","username":"emc-unity-ci"},"date":"2020-06-29 22:14:55.000000000","message":"Patch Set 5:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- DellEMC_Unity_Manila_dhss_true http://publiclogs.emc.com/95/736195/5/check/DellEMC_Unity_Manila_dhss_true/f18d76e/DellEMC_Unity_Manila_dhss_true/114 : FAILURE in 36m 43s\n- DellEMC_Unity_Manila_dhss_false http://publiclogs.emc.com/95/736195/5/check/DellEMC_Unity_Manila_dhss_false/9aa7366/DellEMC_Unity_Manila_dhss_false/114 : FAILURE in 29m 28s\n\nLeave a comment with \u0027run-DellEMC Unity CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_Unity_CI","accounts_in_message":[],"_revision_number":5},{"id":"138bbdf7c240b8e756bdaf7d325b7d979b941993","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2020-06-29 23:07:58.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"58ffa3ee15b746f3116aed76568b3bf4d7c1b56b","author":{"_account_id":21884,"name":"DellEMC Unity CI","email":"emc.unity.ci@emc.com","username":"emc-unity-ci"},"date":"2020-06-29 23:45:17.000000000","message":"Patch Set 6:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- DellEMC_Unity_Manila_dhss_true http://publiclogs.emc.com/95/736195/6/check/DellEMC_Unity_Manila_dhss_true/e677938/DellEMC_Unity_Manila_dhss_true/115 : FAILURE in 37m 10s\n- DellEMC_Unity_Manila_dhss_false http://publiclogs.emc.com/95/736195/6/check/DellEMC_Unity_Manila_dhss_false/e0fa787/DellEMC_Unity_Manila_dhss_false/115 : FAILURE in 28m 09s\n\nLeave a comment with \u0027run-DellEMC Unity CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_Unity_CI","accounts_in_message":[],"_revision_number":6},{"id":"53a559e0a010918868ab3cf7d72934db023adb7d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-30 01:09:43.000000000","message":"Patch Set 6: 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/2d28116893624f908975af484af4e75c : SUCCESS in 3m 42s\n- manila-tempest-plugin-lvm https://zuul.opendev.org/t/openstack/build/8496724e97c344a38a24a6e1381c1b24 : FAILURE in 42m 07s\n- manila-tempest-minimal-dsvm-dummy https://zuul.opendev.org/t/openstack/build/566016590d854cf7b2185ec43b115d26 : FAILURE in 44m 50s\n- manila-tempest-plugin-zfsonlinux https://zuul.opendev.org/t/openstack/build/c7a3b1e6db944e29ab02049a6a23ffe3 : FAILURE in 39m 46s (non-voting)\n- manila-tempest-plugin-cephfs-native https://zuul.opendev.org/t/openstack/build/adfcb1813f364f6c868c836ae2bf0275 : FAILURE in 1h 07m 20s (non-voting)\n- manila-tempest-plugin-container https://zuul.opendev.org/t/openstack/build/5c510add83144dfaa65ce0f202c8a3bb : FAILURE in 34m 01s (non-voting)\n- manila-tempest-plugin-generic https://zuul.opendev.org/t/openstack/build/f2f4b70c25994227b92e3be5f598afa9 : FAILURE in 2h 00m 05s (non-voting)\n- manila-tempest-minimal-dsvm-cephfs-nfs https://zuul.opendev.org/t/openstack/build/4c45e04c7e8648b98955cb844780db54 : FAILURE in 44m 37s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"bb0d010b1f60e99c74c6dd3647985b43d510119e","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2020-06-30 06:29:45.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"66e802da30239337231bf894f340920b3f4e7734","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-30 08:36:33.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c966fb4f4541436789eb12ca4045ce4a : SUCCESS in 4m 57s\n- manila-tempest-plugin-lvm https://zuul.opendev.org/t/openstack/build/a04a956202ef4646b1202bb14dd01cca : SUCCESS in 1h 11m 50s\n- manila-tempest-minimal-dsvm-dummy https://zuul.opendev.org/t/openstack/build/f8360ecba12a4e03af6faee933021368 : SUCCESS in 48m 47s\n- manila-tempest-plugin-zfsonlinux https://zuul.opendev.org/t/openstack/build/967591c137524257850dc9c325a78fb7 : SUCCESS in 1h 20m 25s (non-voting)\n- manila-tempest-plugin-cephfs-native https://zuul.opendev.org/t/openstack/build/85bf39392ad84599aba00ef843c07ca3 : SUCCESS in 1h 28m 28s (non-voting)\n- manila-tempest-plugin-container https://zuul.opendev.org/t/openstack/build/9bf7dd2b79744d18a28d9a890a0e8d2e : SUCCESS in 59m 07s (non-voting)\n- manila-tempest-plugin-generic https://zuul.opendev.org/t/openstack/build/3719df025bbd45e19dcd8645bbc4e37d : FAILURE in 1h 58m 54s (non-voting)\n- manila-tempest-minimal-dsvm-cephfs-nfs https://zuul.opendev.org/t/openstack/build/7541fcd016504ed7be1b602ba75a8133 : SUCCESS in 1h 05m 56s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"2a60dd1be730244612a14afd7b6a04f5e3aa5240","author":{"_account_id":25243,"name":"NetApp CI","username":"NetApp-ci","inactive":true,"tags":["SERVICE_USER"]},"date":"2020-06-30 10:49:14.000000000","message":"Patch Set 7:\n\nBuild succeeded.\n\n- manila-cDOT-manila-tempest-plugin-ss https://logs.openstack.netapp.com/logs/95/736195/7/upstream-check/manila-cDOT-manila-tempest-plugin-ss/5099770/ : SUCCESS in 2h 12m 23s\n- manila-cDOT-manila-tempest-plugin-no-ss https://logs.openstack.netapp.com/logs/95/736195/7/upstream-check/manila-cDOT-manila-tempest-plugin-no-ss/adeebd8/ : SUCCESS in 1h 33m 35s\n\nComment \u0027run-NetApp CI\u0027 to rerun these tests. https://wiki.openstack.org/wiki/ThirdPartySystems/NetApp_CI","accounts_in_message":[],"_revision_number":7},{"id":"dd8d748aa84c2c6003970ac7c1a5be94dc084322","author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"date":"2020-06-30 21:01:43.000000000","message":"Patch Set 7: Code-Review+2\n\nGreat refactoring, simplifies how test classes handle different credentials. Thanks for working on this Goutham and Lucio.\nLooks good to me and CI is green.\nThanks!","accounts_in_message":[],"_revision_number":7},{"id":"286d94f84ef48739c40dfd5a5e85c77de4c9e6d5","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2020-06-30 23:25:32.000000000","message":"Patch Set 7: Code-Review+2 Workflow+1\n\nThis is a lot cleaner and will be easier to understand and maintain.\n\nThanks, Goutham!","accounts_in_message":[],"_revision_number":7},{"id":"280551ce3357666af5a5f5c47698cb37d92cbff7","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-30 23:25:43.000000000","message":"Patch Set 7: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":7},{"id":"d9f899afb081e85852942b2d506c3baaa5f6d69c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-01 00:46:15.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":7},{"id":"166d7d79f8f61dd68181e3ae3e4c799223e4c7e5","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-01 00:46:15.000000000","message":"Patch Set 7: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2c00c8b2ea5142b384db7f1d5315cec5 : SUCCESS in 4m 36s\n- manila-tempest-minimal-dsvm-dummy https://zuul.opendev.org/t/openstack/build/de511c13abb6453786ea9c94eeafb706 : SUCCESS in 34m 24s\n- manila-tempest-plugin-lvm https://zuul.opendev.org/t/openstack/build/f058eab263d94998b91766703ba5c0dd : SUCCESS in 1h 18m 57s","accounts_in_message":[],"_revision_number":7}],"current_revision_number":7,"current_revision":"af44826a3a6a9c3c59e36b7b3b881c2798af8d3f","revisions":{"a38cde6a8327617bde1b7b969432ec59a2845c70":{"kind":"REWORK","_number":1,"created":"2020-06-17 12:47:43.000000000","uploader":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"ref":"refs/changes/95/736195/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/manila-tempest-plugin","ref":"refs/changes/95/736195/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/1"}}},"commit":{"parents":[{"commit":"8257d9e9c095a6d145217217f5c84edfa29384bc","subject":"Merge \"Add CephFS Native scenario tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/8257d9e9c095a6d145217217f5c84edfa29384bc"}]}],"author":{"name":"Lucio Seki","email":"luciomitsuru.seki@fit-tecnologia.org.br","date":"2020-06-17 12:27:33.000000000","tz":-180},"committer":{"name":"Lucio Seki","email":"luciomitsuru.seki@fit-tecnologia.org.br","date":"2020-06-17 12:46:13.000000000","tz":-180},"subject":"Skip tests that create dynamic credentials if it\u0027s disabled","message":"Skip tests that create dynamic credentials if it\u0027s disabled\n\nSome tests create dynamic credentials even with\nCONF.auth.use_dynamic_credentials\u003dFalse.\nSome of them will fail cleaning up the resources they create.\nOthers don\u0027t fail, though they shouldn\u0027t create dynamic creds at all.\n\nThis patch makes those tests to be skipped when\nCONF.auth.use_dynamic_credentials\u003dFalse.\n\nChange-Id: I7607019e47ce9e8c6584f69c2b2d3228873f24df\nCloses-Bug: #1883795\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/a38cde6a8327617bde1b7b969432ec59a2845c70"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/a38cde6a8327617bde1b7b969432ec59a2845c70"}]},"branch":"refs/heads/master"},"1e6d7ad2c72948f4d6fc86e6e303f0f4e8bb4f4c":{"kind":"REWORK","_number":2,"created":"2020-06-20 00:12:34.000000000","uploader":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"ref":"refs/changes/95/736195/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/manila-tempest-plugin","ref":"refs/changes/95/736195/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/2"}}},"commit":{"parents":[{"commit":"8257d9e9c095a6d145217217f5c84edfa29384bc","subject":"Merge \"Add CephFS Native scenario tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/8257d9e9c095a6d145217217f5c84edfa29384bc"}]}],"author":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-20 00:11:32.000000000","tz":-420},"committer":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-20 00:12:27.000000000","tz":-420},"subject":"Use dynamic credentials only where necessary","message":"Use dynamic credentials only where necessary\n\nA number of tests are creating an \"isolated\"\ntest credential to test cross project separation.\nThis isn\u0027t necessary, since the base class already\nrequests projects to run the test. So we can just\nuse different project contexts where appropriate.\n\nIn the quota tests, we can\u0027t work with projects\nwe\u0027ve gotten from tempest (whether dynamic\nor pre-provisioned). This is because these\ntests can be destructive if they accidentally\nmalfunction. So, we request tempest to give\nus isolated credentials with the class level\ndirective \"force_tenant_isolation\u003dTrue\".\n\nThis may not suffice in the negative quota tests\nwhich may leave test credentials in a bad state\nif they malfunction - so we\u0027ll request new\ntest credentials dynamically only in these tests.\n\nWhenever we require \"force_tenant_isolation\u003dTrue\",\nwe\u0027d do well to skip the tests if tempest is\nconfigured to not use dynamic credentials, because\ntempest will gladly ignore the explicit\nconfiguration to disallow dynamic credentials\nand give us new credentials, which might be\nfrustrating to test runners.\n\nCo-Authored-By: Lucio Seki \u003cluciomitsuru.seki@fit-tecnologia.org.br\u003e\nChange-Id: I7607019e47ce9e8c6584f69c2b2d3228873f24df\nCloses-Bug: #1883795\nSigned-off-by: Goutham Pacha Ravi \u003cgouthampravi@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/1e6d7ad2c72948f4d6fc86e6e303f0f4e8bb4f4c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/1e6d7ad2c72948f4d6fc86e6e303f0f4e8bb4f4c"}]},"branch":"refs/heads/master"},"b6a9977d74ef506a57030862ba17baf35ad431ac":{"kind":"REWORK","_number":3,"created":"2020-06-23 05:49:16.000000000","uploader":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"ref":"refs/changes/95/736195/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/manila-tempest-plugin","ref":"refs/changes/95/736195/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/3"}}},"commit":{"parents":[{"commit":"8257d9e9c095a6d145217217f5c84edfa29384bc","subject":"Merge \"Add CephFS Native scenario tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/8257d9e9c095a6d145217217f5c84edfa29384bc"}]}],"author":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-20 00:11:32.000000000","tz":-420},"committer":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-23 05:49:07.000000000","tz":-420},"subject":"Use dynamic credentials only where necessary","message":"Use dynamic credentials only where necessary\n\nA number of tests are creating an \"isolated\"\ntest credential to test cross project separation.\nThis isn\u0027t necessary, since the base class already\nrequests projects to run the test. So we can just\nuse different project contexts where appropriate.\n\nIn the quota tests, we can\u0027t work with projects\nwe\u0027ve gotten from tempest (whether dynamic\nor pre-provisioned). This is because these\ntests can be destructive if they accidentally\nmalfunction. So, we request tempest to give\nus isolated credentials with the class level\ndirective \"force_tenant_isolation\u003dTrue\".\n\nThis may not suffice in the negative quota tests\nwhich may leave test credentials in a bad state\nif they malfunction - so we\u0027ll request new\ntest credentials dynamically only in these tests.\n\nWhenever we require \"force_tenant_isolation\u003dTrue\",\nwe\u0027d do well to skip the tests if tempest is\nconfigured to not use dynamic credentials, because\ntempest will gladly ignore the explicit\nconfiguration to disallow dynamic credentials\nand give us new credentials, which might be\nfrustrating to test runners.\n\nCo-Authored-By: Lucio Seki \u003cluciomitsuru.seki@fit-tecnologia.org.br\u003e\nChange-Id: I7607019e47ce9e8c6584f69c2b2d3228873f24df\nCloses-Bug: #1883795\nSigned-off-by: Goutham Pacha Ravi \u003cgouthampravi@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/b6a9977d74ef506a57030862ba17baf35ad431ac"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/b6a9977d74ef506a57030862ba17baf35ad431ac"}]},"branch":"refs/heads/master"},"353cc6583e1b304a93ebbd77044588994c37147f":{"kind":"REWORK","_number":4,"created":"2020-06-29 21:24:26.000000000","uploader":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"ref":"refs/changes/95/736195/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/manila-tempest-plugin","ref":"refs/changes/95/736195/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/4"}}},"commit":{"parents":[{"commit":"8257d9e9c095a6d145217217f5c84edfa29384bc","subject":"Merge \"Add CephFS Native scenario tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/8257d9e9c095a6d145217217f5c84edfa29384bc"}]}],"author":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-29 21:24:13.000000000","tz":-420},"committer":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-29 21:24:13.000000000","tz":-420},"subject":"Stop forcing creation of \"isolated\" credentials","message":"Stop forcing creation of \"isolated\" credentials\n\nA number of tests are creating an \"isolated\"\ntest credential to test cross project separation.\nThis isn\u0027t necessary, since test base classes\nalready request projects to run the tests. So we\ncan just use different project contexts where\nappropriate.\n\nIn the quota tests, we request tempest to allocate\nus fresh credentials, because these tests can be\ndestructive if they accidentally malfunction.\nWe do this by setting the class level directive\n\"force_tenant_isolation\u003dTrue\". Whenever we do\nthis, we will skip the tests if tempest is\nconfigured to not use dynamic credentials;\nbecause tempest will gladly ignore the explicit\nconfiguration to disallow dynamic credentials\nand give us new credentials, which might be\nfrustrating to test runners.\n\nIn this change, we\n- delete all the code to request \"isolated\"\n  test credentials from tempest.\n- delete code that was written to cleanup\n  test credentials allocated to test \"methods\"\n  and classes.\n- refactor quota tests to revert updates\n  when the test concludes (requesting fresh\n  credentials for these tests is just a fail-safe)\n\nCo-Authored-By: Lucio Seki \u003cluciomitsuru.seki@fit-tecnologia.org.br\u003e\nChange-Id: I7607019e47ce9e8c6584f69c2b2d3228873f24df\nCloses-Bug: #1883795\nSigned-off-by: Goutham Pacha Ravi \u003cgouthampravi@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/353cc6583e1b304a93ebbd77044588994c37147f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/353cc6583e1b304a93ebbd77044588994c37147f"}]},"branch":"refs/heads/master"},"cc1b230c1075083c4cf2c7e1d102ccf997fe3bb8":{"kind":"REWORK","_number":5,"created":"2020-06-29 21:37:57.000000000","uploader":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"ref":"refs/changes/95/736195/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/manila-tempest-plugin","ref":"refs/changes/95/736195/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/5"}}},"commit":{"parents":[{"commit":"8257d9e9c095a6d145217217f5c84edfa29384bc","subject":"Merge \"Add CephFS Native scenario tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/8257d9e9c095a6d145217217f5c84edfa29384bc"}]}],"author":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-29 21:24:13.000000000","tz":-420},"committer":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-29 21:37:50.000000000","tz":-420},"subject":"Stop forcing creation of \"isolated\" credentials","message":"Stop forcing creation of \"isolated\" credentials\n\nA number of tests are creating an \"isolated\"\ntest credential to test cross project separation.\nThis isn\u0027t necessary, since test base classes\nalready request projects to run the tests. So we\ncan just use different project contexts where\nappropriate.\n\nIn the quota tests, we request tempest to allocate\nus fresh credentials, because these tests can be\ndestructive if they accidentally malfunction.\nWe do this by setting the class level directive\n\"force_tenant_isolation\u003dTrue\". Whenever we do\nthis, we will skip the tests if tempest is\nconfigured to not use dynamic credentials;\nbecause tempest will gladly ignore the explicit\nconfiguration to disallow dynamic credentials\nand give us new credentials, which might be\nfrustrating to test runners.\n\nIn this change, we\n- delete all the code to request \"isolated\"\n  test credentials from tempest.\n- delete code that was written to cleanup\n  test credentials allocated to test \"methods\"\n  and classes.\n- refactor quota tests to revert updates\n  when the test concludes (requesting fresh\n  credentials for these tests is just a fail-safe)\n\nCo-Authored-By: Lucio Seki \u003cluciomitsuru.seki@fit-tecnologia.org.br\u003e\nChange-Id: I7607019e47ce9e8c6584f69c2b2d3228873f24df\nCloses-Bug: #1883795\nSigned-off-by: Goutham Pacha Ravi \u003cgouthampravi@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/cc1b230c1075083c4cf2c7e1d102ccf997fe3bb8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/cc1b230c1075083c4cf2c7e1d102ccf997fe3bb8"}]},"branch":"refs/heads/master"},"d5004bfeed1a4c10993cb69023a9797cffe0ad78":{"kind":"REWORK","_number":6,"created":"2020-06-29 23:07:58.000000000","uploader":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"ref":"refs/changes/95/736195/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/manila-tempest-plugin","ref":"refs/changes/95/736195/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/6"}}},"commit":{"parents":[{"commit":"8257d9e9c095a6d145217217f5c84edfa29384bc","subject":"Merge \"Add CephFS Native scenario tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/8257d9e9c095a6d145217217f5c84edfa29384bc"}]}],"author":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-29 21:24:13.000000000","tz":-420},"committer":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-29 23:07:30.000000000","tz":-420},"subject":"Stop forcing creation of \"isolated\" credentials","message":"Stop forcing creation of \"isolated\" credentials\n\nA number of tests are creating an \"isolated\"\ntest credential to test cross project separation.\nThis isn\u0027t necessary, since test base classes\nalready request projects to run the tests. So we\ncan just use different project contexts where\nappropriate.\n\nIn the quota tests, we request tempest to allocate\nus fresh credentials, because these tests can be\ndestructive if they accidentally malfunction.\nWe do this by setting the class level directive\n\"force_tenant_isolation\u003dTrue\". Whenever we do\nthis, we will skip the tests if tempest is\nconfigured to not use dynamic credentials;\nbecause tempest will gladly ignore the explicit\nconfiguration to disallow dynamic credentials\nand give us new credentials, which might be\nfrustrating to test runners.\n\nIn this change, we\n- delete all the code to request \"isolated\"\n  test credentials from tempest.\n- delete code that was written to cleanup\n  test credentials allocated to test \"methods\"\n  and classes.\n- refactor quota tests to revert updates\n  when the test concludes (requesting fresh\n  credentials for these tests is just a fail-safe)\n- Fix creation of share types per visibility, a change\n  that was accidentally removed in [1]\n- Fix \"SharesAdminQuotasUpdateTest.test_reset_share_type_quotas\"\n  by using a correct project user instead of a\n  random project\u0027s user to force enforce user quota\n  higher than the project.\n\n[1] I15880e400df30918762ebd7304244b4a27200168\nCo-Authored-By: Lucio Seki \u003cluciomitsuru.seki@fit-tecnologia.org.br\u003e\nChange-Id: I7607019e47ce9e8c6584f69c2b2d3228873f24df\nCloses-Bug: #1883795\nSigned-off-by: Goutham Pacha Ravi \u003cgouthampravi@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/d5004bfeed1a4c10993cb69023a9797cffe0ad78"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/d5004bfeed1a4c10993cb69023a9797cffe0ad78"}]},"branch":"refs/heads/master"},"af44826a3a6a9c3c59e36b7b3b881c2798af8d3f":{"kind":"REWORK","_number":7,"created":"2020-06-30 06:29:45.000000000","uploader":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"ref":"refs/changes/95/736195/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/manila-tempest-plugin","ref":"refs/changes/95/736195/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/manila-tempest-plugin refs/changes/95/736195/7"}}},"commit":{"parents":[{"commit":"8257d9e9c095a6d145217217f5c84edfa29384bc","subject":"Merge \"Add CephFS Native scenario tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/8257d9e9c095a6d145217217f5c84edfa29384bc"}]}],"author":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-29 21:24:13.000000000","tz":-420},"committer":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2020-06-30 06:29:33.000000000","tz":-420},"subject":"Stop forcing creation of \"isolated\" credentials","message":"Stop forcing creation of \"isolated\" credentials\n\nA number of tests are creating an \"isolated\"\ntest credential to test cross project separation.\nThis isn\u0027t necessary, since test base classes\nalready request projects to run the tests. So we\ncan just use different project contexts where\nappropriate.\n\nIn the quota tests, we request tempest to allocate\nus fresh credentials, because these tests can be\ndestructive if they accidentally malfunction.\nWe do this by setting the class level directive\n\"force_tenant_isolation\u003dTrue\". Whenever we do\nthis, we will skip the tests if tempest is\nconfigured to not use dynamic credentials;\nbecause tempest will gladly ignore the explicit\nconfiguration to disallow dynamic credentials\nand give us new credentials, which might be\nfrustrating to test runners.\n\nIn this change, we\n- delete all the code to request \"isolated\"\n  test credentials from tempest.\n- delete code that was written to cleanup\n  test credentials allocated to test \"methods\"\n  and classes.\n- refactor quota tests to revert updates\n  when the test concludes (requesting fresh\n  credentials for these tests is just a fail-safe)\n- Fix creation of share types per visibility, a change\n  that was accidentally removed in [1]\n- Fix \"SharesAdminQuotasUpdateTest.test_reset_share_type_quotas\"\n  by using a correct project user instead of a\n  random project\u0027s user to force enforce user quota\n  higher than the project.\n\n[1] I15880e400df30918762ebd7304244b4a27200168\nCo-Authored-By: Lucio Seki \u003cluciomitsuru.seki@fit-tecnologia.org.br\u003e\nChange-Id: I7607019e47ce9e8c6584f69c2b2d3228873f24df\nCloses-Bug: #1883795\nSigned-off-by: Goutham Pacha Ravi \u003cgouthampravi@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/af44826a3a6a9c3c59e36b7b3b881c2798af8d3f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/manila-tempest-plugin/commit/af44826a3a6a9c3c59e36b7b3b881c2798af8d3f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
