)]}'
{"id":"openstack%2Fcinder-tempest-plugin~801603","triplet_id":"openstack%2Fcinder-tempest-plugin~master~I64722613acfe7620d3ae0fb29c1d3f85582bfe38","project":"openstack/cinder-tempest-plugin","branch":"master","hashtags":[],"change_id":"I64722613acfe7620d3ae0fb29c1d3f85582bfe38","subject":"Rename cinder plugin config option variable","status":"NEW","created":"2021-07-21 08:50:09.000000000","updated":"2021-08-11 19:48:21.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":false,"submittable":false,"total_comment_count":4,"unresolved_comment_count":1,"has_review_started":true,"meta_rev_id":"dcbb71156195c221b1badda5e0caf1b394e934c2","_number":801603,"virtual_id_number":801603,"owner":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},{"tag":"autogenerated:zuul:check","value":1,"date":"2021-08-11 19:48:05.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":25243,"name":"NetApp CI","username":"NetApp-ci","inactive":true,"tags":["SERVICE_USER"]}],"REVIEWER":[{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-07-21 08:52:46.000000000","updated_by":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"reviewer":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"state":"REVIEWER"},{"updated":"2021-07-21 08:52:46.000000000","updated_by":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"reviewer":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"state":"REVIEWER"},{"updated":"2021-07-21 09:40:33.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2021-08-11 19:48:21.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":"CC"}],"messages":[{"id":"d8f87867ed575b85b6397083a1ab8644661b341f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"date":"2021-07-21 08:50:09.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"fcf14cdbb51f98ae9de6e4cfbd89c08f86d1a12c","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-21 09:40:33.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/a7674abe8d2a4817bb3a05a4c646749f : SUCCESS in 3m 41s\n- cinder-tempest-plugin-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/396b6079a6ab4dde8759b34afd3282e3 : FAILURE in 35m 48s\n- cinder-tempest-plugin-lvm-lio-barbican-centos-8-stream https://zuul.opendev.org/t/openstack/build/af70a999b24f4a3fb69514c22a3af98a : FAILURE in 35m 32s (non-voting)\n- cinder-tempest-plugin-lvm-tgt-barbican https://zuul.opendev.org/t/openstack/build/3920429c7b8a46d0ae65ef5126e0408c : FAILURE in 41m 40s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/6f536b5dce934479b6823e3a2f1b9349 : FAILURE in 38m 58s (non-voting)\n- cinder-tempest-plugin-cbak-ceph https://zuul.opendev.org/t/openstack/build/fe93b99cd456498bb81e2f43e6dc4929 : FAILURE in 21m 48s\n- cinder-tempest-plugin-cbak-s3 https://zuul.opendev.org/t/openstack/build/42f34e0c5e9744b288d9050417370ea7 : FAILURE in 27m 50s\n- cinder-tempest-plugin-basic-wallaby https://zuul.opendev.org/t/openstack/build/13113b454a7b461bb766c9e2228121fa : FAILURE in 35m 58s\n- cinder-tempest-plugin-basic-victoria https://zuul.opendev.org/t/openstack/build/0baf490a43eb499b8ac25db85202f8a3 : FAILURE in 27m 45s\n- cinder-tempest-plugin-basic-ussuri https://zuul.opendev.org/t/openstack/build/471a76772c1e4a99927dfa45b3ffa752 : FAILURE in 33m 16s\n- cinder-tempest-plugin-basic-train https://zuul.opendev.org/t/openstack/build/6d9654bf56b44411a079119b1d3dfb7d : FAILURE in 27m 32s\n- cinder-tempest-plugin-protection-functional https://zuul.opendev.org/t/openstack/build/f2f3b34bb42a4e66b2905a102f9f60fb : FAILURE in 44m 14s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"78329f9603b18ccc419933e1035106725233dbf7","author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"date":"2021-07-21 10:23:16.000000000","message":"Patch Set 1: Code-Review-1\n\nI think the removal of .name is incorrect. The rest is more cosmetic and I\u0027m not sure how much is really required. I\u0027ll defer that to QA cores.","accounts_in_message":[],"_revision_number":1},{"id":"2f0801d77ca024d03a80b8a0520827cc735817fd","author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"date":"2021-07-21 10:32:50.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1: Code-Review-1\n\u003e \n\u003e I think the removal of .name is incorrect. The rest is more cosmetic and I\u0027m not sure how much is really required. I\u0027ll defer that to QA cores.\n\nYou dont need the .name becasue there is __str__() function and it returns the name","accounts_in_message":[],"_revision_number":1},{"id":"d05f6288e722b6ad73a11627d019782c30f1e98d","author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"date":"2021-07-21 10:38:11.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1:\n\u003e \n\u003e \u003e Patch Set 1: Code-Review-1\n\u003e \u003e \n\u003e \u003e I think the removal of .name is incorrect. The rest is more cosmetic and I\u0027m not sure how much is really required. I\u0027ll defer that to QA cores.\n\u003e \n\u003e You dont need the .name becasue there is __str__() function and it returns the name\n\nI think its needed becasue there are two sections under cinder:\nvolume_group and VolumeGroup  (setting params, i would like to add volume encryption params and not use them as hardcoded in the code )\n\nvolume_feature_group and VolumeFeaturesGroup  -\u003e this one we are using now and the naming should be fixed ...","accounts_in_message":[],"_revision_number":1},{"id":"4f396336a05413a141a4695c4239596b291824a1","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"date":"2021-07-21 10:48:56.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"457531c4afe48370cd4b8c2cbc1f54854dc4d6b1","author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"date":"2021-07-21 10:51:10.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1: Code-Review-1\n\u003e \n\u003e I think the removal of .name is incorrect. The rest is more cosmetic and I\u0027m not sure how much is really required. I\u0027ll defer that to QA cores.\n\nSorry , yes you are right , its in a tuple ... my mistake.","accounts_in_message":[],"_revision_number":1},{"id":"4ec3bc52906370a8fc33ad82532f87ba6ced90c5","author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"date":"2021-07-21 12:21:29.000000000","message":"Patch Set 2:\n\nJust a note that if you want to also define options inside [volume] in addition to [volume-feature-enabled] you don\u0027t need to rename those internal variables. Just add whatever is needed to define use [volume] as well. So if you have other changes pending, you can propose them *independently* of this, while its usefulness is evaluated.\n\nAlso, which options do you want to add?","accounts_in_message":[],"_revision_number":2},{"id":"6edc4a4cbf247529477a6366a31a7066e3e4f954","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-21 12:34:26.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/a430e411307a48b59e52005699777427 : FAILURE in 3m 44s\n- cinder-tempest-plugin-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/e2f455e12f0847b19b1fcc400af54cde : SUCCESS in 1h 22m 14s\n- cinder-tempest-plugin-lvm-lio-barbican-centos-8-stream https://zuul.opendev.org/t/openstack/build/3d2141dbd7424ef2945f934f87204786 : SUCCESS in 1h 36m 35s (non-voting)\n- cinder-tempest-plugin-lvm-tgt-barbican https://zuul.opendev.org/t/openstack/build/fb4745d33b7d48eba5c7386fb0c266c4 : SUCCESS in 1h 26m 26s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/768b9f35fcd74d2086e729728840eefa : FAILURE in 28m 03s (non-voting)\n- cinder-tempest-plugin-cbak-ceph https://zuul.opendev.org/t/openstack/build/9f04f16b786f4a8d894034dcef890a1a : FAILURE in 29m 40s\n- cinder-tempest-plugin-cbak-s3 https://zuul.opendev.org/t/openstack/build/ca8935379105488bbc74ef7101070baf : SUCCESS in 39m 23s\n- cinder-tempest-plugin-basic-wallaby https://zuul.opendev.org/t/openstack/build/408ff56ad6fd403a8dbaa674e70d7107 : SUCCESS in 44m 42s\n- cinder-tempest-plugin-basic-victoria https://zuul.opendev.org/t/openstack/build/c5dbc410f96f40adbec9a6fb90f70178 : SUCCESS in 36m 44s\n- cinder-tempest-plugin-basic-ussuri https://zuul.opendev.org/t/openstack/build/54c97587821e45fd8acb6942f9257d53 : SUCCESS in 33m 12s\n- cinder-tempest-plugin-basic-train https://zuul.opendev.org/t/openstack/build/b31e65bba29742bea228537d94df744d : SUCCESS in 36m 03s\n- cinder-tempest-plugin-protection-functional https://zuul.opendev.org/t/openstack/build/f196d7aaff3849b6bbdfb5723067cc28 : SUCCESS in 32m 31s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"951f34367be382a5943b5fa016498cc4571f97b1","author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"date":"2021-07-21 13:26:56.000000000","message":"Patch Set 2:\n\n\u003e Patch Set 2:\n\u003e \n\u003e Just a note that if you want to also define options inside [volume] in addition to [volume-feature-enabled] you don\u0027t need to rename those internal variables. Just add whatever is needed to define use [volume] as well. So if you have other changes pending, you can propose them *independently* of this, while its usefulness is evaluated.\n\u003e \n\u003e Also, which options do you want to add?\n\nI want to add encryption params to config file , we should not use hardcoded encryption cipher and options.\n\nThis patch is the first one i am planning , afterward i will create another section for volume and add encryption options .","accounts_in_message":[],"_revision_number":2},{"id":"6d6b328324dc0c261ff6affaab4ffeaf614a2f3f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"date":"2021-07-21 13:29:12.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"3bf9df846dab8d87c0823d9e22b6941ba4949032","author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"date":"2021-07-21 13:38:17.000000000","message":"Patch Set 3:\n\n\u003e Patch Set 2:\n\u003e \n\u003e \u003e Patch Set 2:\n\u003e \u003e \n\u003e \u003e Just a note that if you want to also define options inside [volume] in addition to [volume-feature-enabled] you don\u0027t need to rename those internal variables. Just add whatever is needed to define use [volume] as well. So if you have other changes pending, you can propose them *independently* of this, while its usefulness is evaluated.\n\u003e \u003e \n\u003e \u003e Also, which options do you want to add?\n\u003e \n\u003e I want to add encryption params to config file , we should not use hardcoded encryption cipher and options.\n\u003e \n\u003e This patch is the first one i am planning , afterward i will create another section for volume and add encryption options .\n\nI guess we all agree that parameters are better, that\u0027s not the problem\n\nAs I said, I don\u0027t think the rename is needed to other parameters. We may end up renaming those variables, but I don\u0027t think you need to wait for this patch to be approved to add more parameters. Did you try to add other parameters and didn\u0027t work without renaming this variable?","accounts_in_message":[],"_revision_number":3},{"id":"25ff47428a5388420e52b3b8b7ea76d8d32638e8","author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"date":"2021-07-21 13:43:38.000000000","message":"Patch Set 3:\n\n\u003e Patch Set 3:\n\u003e \n\u003e \u003e Patch Set 2:\n\u003e \u003e \n\u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \n\u003e \u003e \u003e Just a note that if you want to also define options inside [volume] in addition to [volume-feature-enabled] you don\u0027t need to rename those internal variables. Just add whatever is needed to define use [volume] as well. So if you have other changes pending, you can propose them *independently* of this, while its usefulness is evaluated.\n\u003e \u003e \u003e \n\u003e \u003e \u003e Also, which options do you want to add?\n\u003e \u003e \n\u003e \u003e I want to add encryption params to config file , we should not use hardcoded encryption cipher and options.\n\u003e \u003e \n\u003e \u003e This patch is the first one i am planning , afterward i will create another section for volume and add encryption options .\n\u003e \n\u003e I guess we all agree that parameters are better, that\u0027s not the problem\n\u003e \n\u003e As I said, I don\u0027t think the rename is needed to other parameters. We may end up renaming those variables, but I don\u0027t think you need to wait for this patch to be approved to add more parameters. Did you try to add other parameters and didn\u0027t work without renaming this variable?\n\nNo , we can add another section without this change but i prefer that we will do it same as in tempest... and not naming like cinder_options because its confusing","accounts_in_message":[],"_revision_number":3},{"id":"d519cb37bf8bbc041e6cba203017418744f9b0ed","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-21 15:25:08.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/7932ac8aae5844989fe6317091c2b64c : SUCCESS in 5m 05s\n- cinder-tempest-plugin-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/55adf3ef7cc94f1d8f6c107d03201a28 : SUCCESS in 1h 18m 26s\n- cinder-tempest-plugin-lvm-lio-barbican-centos-8-stream https://zuul.opendev.org/t/openstack/build/4fd79ba6b66e4e77962026658602efda : FAILURE in 1h 52m 07s (non-voting)\n- cinder-tempest-plugin-lvm-tgt-barbican https://zuul.opendev.org/t/openstack/build/d9c066f0c9a54f8ba863d75380bf32c8 : SUCCESS in 1h 25m 22s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/164eba2182cb4566add7361134a201ca : FAILURE in 29m 07s (non-voting)\n- cinder-tempest-plugin-cbak-ceph https://zuul.opendev.org/t/openstack/build/05f31fc3a4c74bb0a0084a018dc6ad0c : FAILURE in 29m 34s\n- cinder-tempest-plugin-cbak-s3 https://zuul.opendev.org/t/openstack/build/1336842b18ef4e7dbc604b948d98ddf4 : SUCCESS in 38m 37s\n- cinder-tempest-plugin-basic-wallaby https://zuul.opendev.org/t/openstack/build/da3175979d064385b1518d1d6a79351c : SUCCESS in 43m 47s\n- cinder-tempest-plugin-basic-victoria https://zuul.opendev.org/t/openstack/build/8c034de668c34c10ac7f810a9948acc7 : SUCCESS in 42m 19s\n- cinder-tempest-plugin-basic-ussuri https://zuul.opendev.org/t/openstack/build/271fee06f4414f878ebbe36371f3547e : SUCCESS in 44m 55s\n- cinder-tempest-plugin-basic-train https://zuul.opendev.org/t/openstack/build/616a3fe739e44be4a54c0145a7847a60 : SUCCESS in 43m 30s\n- cinder-tempest-plugin-protection-functional https://zuul.opendev.org/t/openstack/build/2f263ce77d3a4abbb74c361a92629858 : SUCCESS in 31m 57s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"3cbc69737bd22a0faf8a520cdb738c4f046c3040","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-07-21 15:53:18.000000000","message":"Patch Set 3:\n\n(2 comments)","accounts_in_message":[],"_revision_number":3},{"id":"9dcb0bf561a4737f6129d288b17345ebb35daac4","author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"date":"2021-07-21 15:57:54.000000000","message":"Patch Set 3:\n\n(2 comments)","accounts_in_message":[],"_revision_number":3},{"id":"8275ca1603bea35b4825ade50fc87f954764a8e8","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"date":"2021-07-21 15:58:54.000000000","message":"Uploaded patch set 4: Commit message was updated.","accounts_in_message":[],"_revision_number":4},{"id":"35f772f3eae2a8fe3ed90fc590f953ce29319f5b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-21 18:15:41.000000000","message":"Patch Set 4: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/fe4294b8ad0f4bb49a036db1032ba657 : SUCCESS in 3m 56s\n- cinder-tempest-plugin-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/fbbaa8224e264e429ff405c5289b4f62 : SUCCESS in 1h 14m 23s\n- cinder-tempest-plugin-lvm-lio-barbican-centos-8-stream https://zuul.opendev.org/t/openstack/build/9ad61f6dca084e4cb230757d2f60f1d1 : FAILURE in 1h 30m 25s (non-voting)\n- cinder-tempest-plugin-lvm-tgt-barbican https://zuul.opendev.org/t/openstack/build/4ef60e49915542e1adb28c7cceb37f91 : SUCCESS in 1h 09m 44s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/33908f63bfbb4ed28ef9511d5c71647b : FAILURE in 26m 57s (non-voting)\n- cinder-tempest-plugin-cbak-ceph https://zuul.opendev.org/t/openstack/build/64ce867eeacd487c826a118fbb9fb418 : FAILURE in 27m 39s\n- cinder-tempest-plugin-cbak-s3 https://zuul.opendev.org/t/openstack/build/e610020de49c4ca29e9ced3f9d930be8 : SUCCESS in 33m 04s\n- cinder-tempest-plugin-basic-wallaby https://zuul.opendev.org/t/openstack/build/e71db81f10da474ca8b5249a8d9d825c : SUCCESS in 40m 04s\n- cinder-tempest-plugin-basic-victoria https://zuul.opendev.org/t/openstack/build/831b064cb06049c7bfe3ad21d91a2dcd : SUCCESS in 38m 08s\n- cinder-tempest-plugin-basic-ussuri https://zuul.opendev.org/t/openstack/build/78117850e0e04b26a5729ebf558a4560 : SUCCESS in 35m 41s\n- cinder-tempest-plugin-basic-train https://zuul.opendev.org/t/openstack/build/7420a089b4234966a392bf6d090fe56c : SUCCESS in 35m 35s\n- cinder-tempest-plugin-protection-functional https://zuul.opendev.org/t/openstack/build/67923eef2d6d4400b332482ae8788919 : SUCCESS in 29m 46s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"a4e56c17de31b6d8597adc15e9a5c86d5023420d","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"date":"2021-08-11 18:06:38.000000000","message":"Patch Set 5: Commit message was updated.","accounts_in_message":[],"_revision_number":5},{"id":"b68c71a5116c70f4b3b53e38f868f48ffbee3f2c","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-11 19:48:05.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e2e4579cee3c41b2baa6d8b65db9ca71 : SUCCESS in 4m 11s\n- cinder-tempest-plugin-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/6717f7130f0f4d498ecc360a30b26626 : SUCCESS in 1h 29m 01s\n- cinder-tempest-plugin-lvm-lio-barbican-centos-8-stream https://zuul.opendev.org/t/openstack/build/b8278a6eb99f41b7a316d295ea34ac96 : FAILURE in 1h 31m 13s (non-voting)\n- cinder-tempest-plugin-lvm-tgt-barbican https://zuul.opendev.org/t/openstack/build/5a8ca66683334469b2f5aaa50494e826 : SUCCESS in 1h 29m 31s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/0d6b0c0c24c24084b45c9c1f0813e1b0 : SUCCESS in 1h 32m 06s (non-voting)\n- cinder-tempest-plugin-cbak-ceph https://zuul.opendev.org/t/openstack/build/9dc1c5c8d1a54544ad7ec29abf7c4d5b : SUCCESS in 1h 08m 33s\n- cinder-tempest-plugin-cbak-s3 https://zuul.opendev.org/t/openstack/build/abacdd929555465aaf290cc8599da20a : SUCCESS in 40m 49s\n- cinder-tempest-plugin-basic-wallaby https://zuul.opendev.org/t/openstack/build/ee6d91c965ae4530830a21701dc59b26 : SUCCESS in 36m 26s\n- cinder-tempest-plugin-basic-victoria https://zuul.opendev.org/t/openstack/build/fe6ead2cf54940e1b50ac4f57dca58ad : SUCCESS in 38m 12s\n- cinder-tempest-plugin-basic-ussuri https://zuul.opendev.org/t/openstack/build/b313bc1eb5a7451c88db7a8d92d22b77 : SUCCESS in 40m 54s\n- cinder-tempest-plugin-basic-train https://zuul.opendev.org/t/openstack/build/25c0028d6ec3418eaec6a6beb09ff5ab : SUCCESS in 39m 46s\n- cinder-tempest-plugin-protection-functional https://zuul.opendev.org/t/openstack/build/112671d56cff49fcab529ec0a585b651 : SUCCESS in 31m 03s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"dcbb71156195c221b1badda5e0caf1b394e934c2","author":{"_account_id":25243,"name":"NetApp CI","username":"NetApp-ci","inactive":true,"tags":["SERVICE_USER"]},"date":"2021-08-11 19:48:21.000000000","message":"Patch Set 5:\n\nBuild failed.\n\n- cinder-cDOT-iSCSI-cinder-tempest-plugin https://logs.openstack.netapp.com/logs/03/801603/5/upstream-check/cinder-cDOT-iSCSI-cinder-tempest-plugin/689f59b/ : NOT_REGISTERED\n- cinder-cDOT-NFS-cinder-tempest-plugin https://logs.openstack.netapp.com/logs/03/801603/5/upstream-check/cinder-cDOT-NFS-cinder-tempest-plugin/4a336e0/ : NOT_REGISTERED\n- cinder-cDOT-FCP-cinder-tempest-plugin https://logs.openstack.netapp.com/logs/03/801603/5/upstream-check/cinder-cDOT-FCP-cinder-tempest-plugin/45418fb/ : NOT_REGISTERED\n- cinder-solidfire-iSCSI-cinder-tempest-plugin https://logs.openstack.netapp.com/logs/03/801603/5/upstream-check/cinder-solidfire-iSCSI-cinder-tempest-plugin/d9fd55c/ : NOT_REGISTERED\n\nComment \u0027run-NetApp CI\u0027 to rerun these tests. https://wiki.openstack.org/wiki/ThirdPartySystems/NetApp_CI","accounts_in_message":[],"_revision_number":5}],"current_revision_number":5,"current_revision":"189f7ff9b21f1d6ece9b867d95e1633408ee5860","revisions":{"1a068fa6e8e0370077158c8459d1d81300c501c6":{"kind":"REWORK","_number":1,"created":"2021-07-21 08:50:09.000000000","uploader":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"ref":"refs/changes/03/801603/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder-tempest-plugin","ref":"refs/changes/03/801603/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/1"}}},"commit":{"parents":[{"commit":"a2a23ca6d10c90d19cda68220d102409c75390e5","subject":"Merge \"Change minversion to 3.18.0 in tox\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/a2a23ca6d10c90d19cda68220d102409c75390e5"}]}],"author":{"name":"Benny Kopilov","email":"bkopilov@redhat.com","date":"2021-07-21 08:43:57.000000000","tz":180},"committer":{"name":"Benny Kopilov","email":"bkopilov@redhat.com","date":"2021-07-21 08:50:07.000000000","tz":180},"subject":"Fix cinder plugin config registration","message":"Fix cinder plugin config registration\n\nCurrent naming in tempest based on type group and group info\nfor exeample : volume_feature_group and VolumeFeaturesGroup\n\nIn order to maintain more options for registarion we should be aligned\nwith tempest style .\n\nwe will have to extend registration to :volume_group and VolumeGroup\n\nChange-Id: I64722613acfe7620d3ae0fb29c1d3f85582bfe38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/1a068fa6e8e0370077158c8459d1d81300c501c6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/1a068fa6e8e0370077158c8459d1d81300c501c6"}]},"branch":"refs/heads/master"},"6f11fbb3f4eedae70a331ffba9e9f34be3023f98":{"kind":"REWORK","_number":2,"created":"2021-07-21 10:48:56.000000000","uploader":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"ref":"refs/changes/03/801603/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder-tempest-plugin","ref":"refs/changes/03/801603/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/2"}}},"commit":{"parents":[{"commit":"a2a23ca6d10c90d19cda68220d102409c75390e5","subject":"Merge \"Change minversion to 3.18.0 in tox\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/a2a23ca6d10c90d19cda68220d102409c75390e5"}]}],"author":{"name":"Benny Kopilov","email":"bkopilov@redhat.com","date":"2021-07-21 08:43:57.000000000","tz":180},"committer":{"name":"Benny Kopilov","email":"bkopilov@redhat.com","date":"2021-07-21 10:48:48.000000000","tz":180},"subject":"Fix cinder plugin config registration","message":"Fix cinder plugin config registration\n\nCurrent naming in tempest based on type group and group info\nfor exeample : volume_feature_group and VolumeFeaturesGroup\n\nIn order to maintain more options for registarion we should be aligned\nwith tempest style .\n\nwe will have to extend registration to :volume_group and VolumeGroup\n\nChange-Id: I64722613acfe7620d3ae0fb29c1d3f85582bfe38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/6f11fbb3f4eedae70a331ffba9e9f34be3023f98"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/6f11fbb3f4eedae70a331ffba9e9f34be3023f98"}]},"branch":"refs/heads/master"},"4cff2a74e42c498f5128dd3d3bfe0c1539ac940b":{"kind":"REWORK","_number":3,"created":"2021-07-21 13:29:12.000000000","uploader":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"ref":"refs/changes/03/801603/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder-tempest-plugin","ref":"refs/changes/03/801603/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/3"}}},"commit":{"parents":[{"commit":"a2a23ca6d10c90d19cda68220d102409c75390e5","subject":"Merge \"Change minversion to 3.18.0 in tox\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/a2a23ca6d10c90d19cda68220d102409c75390e5"}]}],"author":{"name":"Benny Kopilov","email":"bkopilov@redhat.com","date":"2021-07-21 08:43:57.000000000","tz":180},"committer":{"name":"Benny Kopilov","email":"bkopilov@redhat.com","date":"2021-07-21 13:29:02.000000000","tz":180},"subject":"Fix cinder plugin config registration","message":"Fix cinder plugin config registration\n\nCurrent naming in tempest based on type group and group info\nfor exeample : volume_feature_group and VolumeFeaturesGroup\n\nIn order to maintain more options for registarion we should be aligned\nwith tempest style .\n\nwe will have to extend registration to :volume_group and VolumeGroup\n\nChange-Id: I64722613acfe7620d3ae0fb29c1d3f85582bfe38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/4cff2a74e42c498f5128dd3d3bfe0c1539ac940b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/4cff2a74e42c498f5128dd3d3bfe0c1539ac940b"}]},"branch":"refs/heads/master"},"d1caf8cebf33213ac99959ce178e14d9bbe333ea":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2021-07-21 15:58:54.000000000","uploader":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"ref":"refs/changes/03/801603/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder-tempest-plugin","ref":"refs/changes/03/801603/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/4"}}},"commit":{"parents":[{"commit":"a2a23ca6d10c90d19cda68220d102409c75390e5","subject":"Merge \"Change minversion to 3.18.0 in tox\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/a2a23ca6d10c90d19cda68220d102409c75390e5"}]}],"author":{"name":"Benny Kopilov","email":"bkopilov@redhat.com","date":"2021-07-21 08:43:57.000000000","tz":180},"committer":{"name":"Benny Kopilov","email":"bkopilov@redhat.com","date":"2021-07-21 15:58:26.000000000","tz":180},"subject":"Rename cinder plugin config option variable","message":"Rename cinder plugin config option variable\n\nCurrent naming in tempest based on type group and group info\nfor exeample : volume_feature_group and VolumeFeaturesGroup\n\nIn order to maintain more options for registarion we should be aligned\nwith tempest style .\n\nwe will have to extend registration to :volume_group and VolumeGroup\n\nChange-Id: I64722613acfe7620d3ae0fb29c1d3f85582bfe38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/d1caf8cebf33213ac99959ce178e14d9bbe333ea"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/d1caf8cebf33213ac99959ce178e14d9bbe333ea"}]},"branch":"refs/heads/master"},"189f7ff9b21f1d6ece9b867d95e1633408ee5860":{"kind":"NO_CODE_CHANGE","_number":5,"created":"2021-08-11 18:06:38.000000000","uploader":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"ref":"refs/changes/03/801603/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder-tempest-plugin","ref":"refs/changes/03/801603/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder-tempest-plugin refs/changes/03/801603/5"}}},"commit":{"parents":[{"commit":"a2a23ca6d10c90d19cda68220d102409c75390e5","subject":"Merge \"Change minversion to 3.18.0 in tox\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/a2a23ca6d10c90d19cda68220d102409c75390e5"}]}],"author":{"name":"Benny Kopilov","email":"bkopilov@redhat.com","date":"2021-07-21 08:43:57.000000000","tz":180},"committer":{"name":"Benny Kopilov","email":"bkopilov@redhat.com","date":"2021-08-11 18:06:38.000000000","tz":0},"subject":"Rename cinder plugin config option variable","message":"Rename cinder plugin config option variable\n\nCurrent naming in tempest based on type group and group info\nfor exeample : volume_feature_group and VolumeFeaturesGroup\n\nIn order to maintain more options for registarion we should be aligned\nwith tempest style .\n\nwe will have to extend registration to :volume_group and VolumeGroup\nsame as in tempest\n\nChange-Id: I64722613acfe7620d3ae0fb29c1d3f85582bfe38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/189f7ff9b21f1d6ece9b867d95e1633408ee5860"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder-tempest-plugin/commit/189f7ff9b21f1d6ece9b867d95e1633408ee5860"}]},"branch":"refs/heads/master","description":"Edit commit message"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY"},{"label":"Workflow","status":"MAY"},{"label":"Review-Priority","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Review-Priority","description":"Review priority","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"-label:Review-Priority\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":[],"failing_atoms":["label:Review-Priority\u003dMIN"],"atom_explanations":{"label:Review-Priority\u003dMIN":""}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}