)]}'
{"id":"openstack%2Ftempest~779814","triplet_id":"openstack%2Ftempest~master~I96fdfdcff9e86a8088420fe9ce0f299f2b5683b4","project":"openstack/tempest","branch":"master","topic":"bug/1916926","hashtags":["glance-wallaby"],"change_id":"I96fdfdcff9e86a8088420fe9ce0f299f2b5683b4","subject":"Make image metadef tests optional","status":"ABANDONED","created":"2021-03-10 15:33:42.000000000","updated":"2021-03-12 21:42:40.000000000","total_comment_count":1,"unresolved_comment_count":1,"has_review_started":true,"meta_rev_id":"48e58fcae208823048c9601dc67223bf3d968c6f","_number":779814,"virtual_id_number":779814,"owner":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"tag":"autogenerated:zuul:check","value":1,"date":"2021-03-11 02:48:25.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":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"value":-1,"date":"2021-03-11 20:15:22.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_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":{"all":[{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"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"},{"_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},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Release Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-03-10 19:26:35.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-03-11 19:25:23.000000000","updated_by":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"reviewer":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"state":"REVIEWER"},{"updated":"2021-03-11 20:15:22.000000000","updated_by":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"reviewer":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"state":"REVIEWER"}],"messages":[{"id":"8e5657b43e8ad5ec289dc2e73a1d13208883b6ae","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2021-03-10 15:33:42.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b2ebbfc53fa7291f93e750a0e18a84f5dece7094","tag":"autogenerated:gerrit:setHashtag","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2021-03-10 15:38:38.000000000","message":"Hashtag added: glance-wallaby","accounts_in_message":[],"_revision_number":1},{"id":"79f3d4be28bf035364bdf541ace05f63894425b7","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2021-03-10 17:19:59.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"3f9a708b3dbbc72dea41e29cc37cf809c3c9347e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-10 19:26:35.000000000","message":"Patch Set 2: Verified-1\n\nThis change depends on a change that failed to merge.","accounts_in_message":[],"_revision_number":2},{"id":"2ff0c362ae8ea1fa03f55abe9842d58dffb59e0f","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2021-03-10 19:27:15.000000000","message":"Patch Set 2:\n\nrecheck devstack changed","accounts_in_message":[],"_revision_number":2},{"id":"14e37805da9a654603898a355e8116b7903040a2","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-11 02:48:25.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/5a45456449fe4d449a9abdd0c54b83a8 : SUCCESS in 1h 05m 02s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/6d1253af9a3d4c839fa88193cdd76aca : SUCCESS in 1h 28m 36s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/c3dc244245a0441d8c969387fb95733b : SUCCESS in 8m 19s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/38241c3b13c343eb894eaaacfeb8b584 : SUCCESS in 6m 19s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/b7737591ea894895896f758f58d11a34 : SUCCESS in 7m 11s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/5e05d00095d2400d9a037d59eae76bf1 : SUCCESS in 5m 36s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ceb93cd23c8e4ae98fd4db4b1d77842f : SUCCESS in 13m 41s\n- tempest-full-parallel https://zuul.opendev.org/t/openstack/build/9c9dee9e9f1e4f569d19de569bc65a9f : SUCCESS in 1h 37m 52s (non-voting)\n- tempest-full-py3-ipv6 https://zuul.opendev.org/t/openstack/build/97b92322f1e14fc78607093fd2dba5a5 : SUCCESS in 1h 16m 31s (non-voting)\n- glance-multistore-cinder-import https://zuul.opendev.org/t/openstack/build/c2aa715a315e4bd9a7654d153956c666 : SUCCESS in 1h 35m 19s (non-voting)\n- tempest-full-victoria-py3 https://zuul.opendev.org/t/openstack/build/f3ef39d7204e4f2881268bd96036a593 : SUCCESS in 1h 29m 40s\n- tempest-full-ussuri-py3 https://zuul.opendev.org/t/openstack/build/7173670c21024b1e87e81111e6b719c4 : SUCCESS in 1h 19m 19s\n- tempest-full-train-py3 https://zuul.opendev.org/t/openstack/build/03542d2de3fc43deb88693ef219970fd : SUCCESS in 1h 20m 17s\n- tempest-multinode-full-py3 https://zuul.opendev.org/t/openstack/build/6aee54a095a54d07a7e41501f15dfe55 : SUCCESS in 1h 53m 20s\n- tempest-tox-plugin-sanity-check https://zuul.opendev.org/t/openstack/build/56ec0c5ee734481497759ad00b2e46a6 : SUCCESS in 45m 50s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/cb2b7411afe14582a2b087652cce1b49 : SUCCESS in 59m 50s\n- tempest-slow-py3 https://zuul.opendev.org/t/openstack/build/503b582dd56749adaa257d4809cbf8bd : SUCCESS in 2h 17m 23s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/95240349926b4674a1b145434e26e25a : SUCCESS in 53m 20s (non-voting)\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/29049605d0244fe8bfc405094aca9378 : SUCCESS in 1h 29m 18s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/85467d78597f4c7dbbad385b4e0f22b1 : SUCCESS in 1h 03m 50s\n- neutron-tempest-dvr https://zuul.opendev.org/t/openstack/build/d952f68c0acc431fa6012b69c80dfe9d : SUCCESS in 1h 04m 08s (non-voting)\n- interop-tempest-consistency https://zuul.opendev.org/t/openstack/build/df30d9fed5cd4c2c9b892912998f80ee : SUCCESS in 3m 35s\n- tempest-full-test-account-py3 https://zuul.opendev.org/t/openstack/build/cd44c8d62bae4f7392d1351a1590854c : FAILURE in 1h 30m 52s (non-voting)\n- tempest-full-test-account-no-admin-py3 https://zuul.opendev.org/t/openstack/build/1ad64f0c732d431287dccd21c554fd67 : FAILURE in 1h 00m 46s (non-voting)\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/b7c4b878fc4a41739dbc272eb620a7ae : SUCCESS in 3m 45s","accounts_in_message":[],"_revision_number":2},{"id":"2bd11dcb8de4f20816303856a41d95d0cd8c7f8b","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2021-03-11 19:25:23.000000000","message":"Patch Set 2:\n\nSo, based on the conversation on the list, one way to change this is to make tempest use the admin client for the metadef stuff such that it will continue to work with admin-only creation policy, if that\u0027s what we decide to do.","accounts_in_message":[],"_revision_number":2},{"id":"28cdaaa17611ef6cad59dfc73327a2ee8fc77c00","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-03-11 20:15:22.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)\n\nyeah as this is only policy permission change we do not need to test both old and new policy in Tempest. Tempest can just test the latest default working policies. To do that you can do:\n\n- all these tests can be inherited from BaseV2ImageAdminTest\n- Move those test file under admin dir https://github.com/openstack/tempest/tree/master/tempest/api/image/v2/admin\n- defined all client in BaseV2ImageAdminTest what all these test needed, like -https://github.com/openstack/tempest/blob/143abec5c5ab30ed94109f870f93e781306f3c1b/tempest/api/image/base.py#L206","accounts_in_message":[],"_revision_number":2},{"id":"178808873e98ffecca0cce5debb7319c3de9920d","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-03-11 20:19:02.000000000","message":"Patch Set 2:\n\nalso these API(tests) are not used in interop (https://opendev.org/osf/interop/src/branch/master/next.json) so we are good here to move the test permission to admin.","accounts_in_message":[],"_revision_number":2},{"id":"f7c1ae96438aab0be3b8ea4cd855820b55483dec","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2021-03-11 20:20:59.000000000","message":"Patch Set 2:\n\n\u003e - all these tests can be inherited from BaseV2ImageAdminTest\n\u003e - Move those test file under admin dir https://github.com/openstack/tempest/tree/master/tempest/api/image/v2/admin\n\u003e - defined all client in BaseV2ImageAdminTest what all these test needed, like -https://github.com/openstack/tempest/blob/143abec5c5ab30ed94109f870f93e781306f3c1b/tempest/api/image/base.py#L206\n\nAck, I will try to get something to do that stacked on top here to show the option. Thanks Gmann!","accounts_in_message":[],"_revision_number":2},{"id":"48e58fcae208823048c9601dc67223bf3d968c6f","tag":"autogenerated:gerrit:abandon","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2021-03-12 21:42:40.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"18edecf433adf049c20e407dd4563d6ffd3ac0ef","revisions":{"4c16e09ba0c63f033226dea2da4d9a4549e7cefd":{"kind":"REWORK","_number":1,"created":"2021-03-10 15:33:42.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/14/779814/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tempest","ref":"refs/changes/14/779814/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/779814/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/779814/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/779814/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tempest refs/changes/14/779814/1"}}},"commit":{"parents":[{"commit":"371a96b7025dc59b15d71663401ecf98abf2f288","subject":"Merge \"Create tmd dir with specific prefix in check-uuid unit tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/371a96b7025dc59b15d71663401ecf98abf2f288"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2021-03-10 15:29:51.000000000","tz":-480},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2021-03-10 15:31:52.000000000","tz":-480},"subject":"Make image metadef tests optional","message":"Make image metadef tests optional\n\nStarting in Wallaby, Glance has disabled these APIs by default in\npolicy, and requires operators to opt-in to them if they want them\nenabled and accept the risk of doing so. This adds a boolean config\ntoggle to tell tempest whether or not it should try to test those\nAPIs.\n\nDepends-On: https://review.opendev.org/c/openstack/devstack/+/779808\nChange-Id: I96fdfdcff9e86a8088420fe9ce0f299f2b5683b4\nRelated-Bug: #1916926\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/4c16e09ba0c63f033226dea2da4d9a4549e7cefd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/4c16e09ba0c63f033226dea2da4d9a4549e7cefd"}]},"branch":"refs/heads/master"},"18edecf433adf049c20e407dd4563d6ffd3ac0ef":{"kind":"REWORK","_number":2,"created":"2021-03-10 17:19:59.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/14/779814/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tempest","ref":"refs/changes/14/779814/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/779814/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/779814/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/779814/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tempest refs/changes/14/779814/2"}}},"commit":{"parents":[{"commit":"371a96b7025dc59b15d71663401ecf98abf2f288","subject":"Merge \"Create tmd dir with specific prefix in check-uuid unit tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/371a96b7025dc59b15d71663401ecf98abf2f288"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2021-03-10 15:29:51.000000000","tz":-480},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2021-03-10 17:19:58.000000000","tz":-480},"subject":"Make image metadef tests optional","message":"Make image metadef tests optional\n\nStarting in Wallaby, Glance has disabled these APIs by default in\npolicy, and requires operators to opt-in to them if they want them\nenabled and accept the risk of doing so. This adds a boolean config\ntoggle to tell tempest whether or not it should try to test those\nAPIs.\n\nDepends-On: https://review.opendev.org/c/openstack/devstack/+/779808\nChange-Id: I96fdfdcff9e86a8088420fe9ce0f299f2b5683b4\nRelated-Bug: #1916926\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/18edecf433adf049c20e407dd4563d6ffd3ac0ef"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/18edecf433adf049c20e407dd4563d6ffd3ac0ef"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}