)]}'
{"id":"openstack%2Fglance~774721","triplet_id":"openstack%2Fglance~master~Id613cfd61760c383c7c3cc6aea3f37eecb5ed4d9","project":"openstack/glance","branch":"master","topic":"bug/1914826","hashtags":[],"change_id":"Id613cfd61760c383c7c3cc6aea3f37eecb5ed4d9","subject":"Make action wrapper support arbitrary properties","status":"MERGED","created":"2021-02-09 20:14:25.000000000","updated":"2021-03-06 17:34:06.000000000","submitted":"2021-03-06 17:34:06.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"774721-bug/1914826","meta_rev_id":"26331c4d75a3103ff1b7a4f9601596a3c889da4b","_number":774721,"virtual_id_number":774721,"owner":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-03-06 17:34:05.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":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"all":[{"value":2,"date":"2021-02-10 05:38:16.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"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":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"all":[{"value":1,"date":"2021-03-05 18:25:01.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"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},"Review-Priority":{"recommended":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"all":[{"value":1,"date":"2021-02-10 05:38:16.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Top Priority / Holds Gate"},"description":"","value":1,"default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]}],"REVIEWER":[{"_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-02-09 20:16:02.000000000","updated_by":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-02-10 00:17:12.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-02-10 05:38:16.000000000","updated_by":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"reviewer":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"state":"REVIEWER"}],"messages":[{"id":"5b97069d095fdc52b5a3ef1c5081d868a56537f7","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2021-02-09 20:14:25.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"848db3a49ba367006db55b5748f2167180fb6fce","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-02-09 20:16:02.000000000","message":"Patch Set 1:\n\nBuild failed\n\n- ext-glance-zuul : NOT_REGISTERED","accounts_in_message":[],"_revision_number":1},{"id":"c92f685a57d42382b62131f500bb788669eb1ed1","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-10 00:17:12.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/ef9cbaa38c084047b536122371e1e368 : SUCCESS in 47m 00s\n- tempest-integrated-storage https://zuul.opendev.org/t/openstack/build/8d7eac8f3f2e4d10a60815b6609582ec : SUCCESS in 1h 13m 33s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/7730e829fe0146b88151dea8284c7ca5 : SUCCESS in 7m 20s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/fa86c7aceefe47bfaafbf10207ea9d63 : SUCCESS in 4m 28s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/bb7e53e56e14441bbb0df771f814f4ac : SUCCESS in 5m 05s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/0ed5b2868add49b9837d82b210aaab85 : SUCCESS in 6m 05s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/a76099f60ed24a5081d6e1ff1a45f272 : SUCCESS in 5m 55s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/8a83bccdef1c49db930b53e207a41024 : SUCCESS in 6m 09s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/bafe62db73134a43899cd6c669da1476 : SUCCESS in 9m 11s\n- openstack-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/333e255e8f3445768d9cbe2a5d84f96b : SUCCESS in 8m 39s\n- glance-code-constants-check https://zuul.opendev.org/t/openstack/build/436db9f7cabb49edb14b8966339db3cf : SUCCESS in 5m 03s\n- glance-ceph-thin-provisioning https://zuul.opendev.org/t/openstack/build/fa60aab8aaa446d79371ebdf9af3baa0 : SUCCESS in 1h 05m 58s (non-voting)\n- tempest-integrated-storage-import https://zuul.opendev.org/t/openstack/build/094a3d2e0f1b4490bf8c6778a66142ab : SUCCESS in 1h 21m 29s\n- tempest-integrated-storage-import-standalone https://zuul.opendev.org/t/openstack/build/c360f1458efc4566a2b769770c633439 : SUCCESS in 1h 12m 32s\n- glance-multistore-cinder-import https://zuul.opendev.org/t/openstack/build/245509d1bc6548ecad7d916ecba0be33 : SUCCESS in 1h 06m 22s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/d0e5652e9a2b4dd8990a91da14548e50 : SUCCESS in 59m 02s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/77ba66c6b5fc467d9f640631f2a655ba : SUCCESS in 1h 05m 32s","accounts_in_message":[],"_revision_number":1},{"id":"bc910cd64b899009b2bc0e515b261ed71b6f5340","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2021-02-10 05:38:16.000000000","message":"Patch Set 1: Code-Review+2 Review-Priority+1","accounts_in_message":[],"_revision_number":1},{"id":"eac739a35ea0370f001f79df6a6cc093eea8a31a","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2021-03-05 18:25:01.000000000","message":"Patch Set 1: Workflow+1\n\nNinja approving, sitting open for long, and will be good improvement for import workflow and plugins.\nThank you.","accounts_in_message":[],"_revision_number":1},{"id":"420cdc8489221e64df9f2f8d196ae3a357bc60a3","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-06 01:55:37.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"c41f94b517bd4ae3feaed55562f1ff235df9731b","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-06 11:41:39.000000000","message":"Patch Set 1: Verified-2\n\nThis change depends on a change that failed to merge.","accounts_in_message":[],"_revision_number":1},{"id":"ef2431fcf523e10fb28b6e73cb03d1ba81d85dca","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2021-03-06 11:47:51.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"3583a9f890ace7292f7a0778671f2f066de800b4","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-06 13:23:09.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/e315fa4ff5bb4c559b3d3f1e24dbc1d6 : SUCCESS in 59m 26s\n- tempest-integrated-storage https://zuul.opendev.org/t/openstack/build/d3aa8a75e4864e96aaf0448ec7284dfc : SUCCESS in 1h 21m 56s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/3a24afc02fce4ef2bc560d65b9519a5a : SUCCESS in 6m 27s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/34eecef716d249ca8381648ec5bc65fc : SUCCESS in 5m 30s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/c1a4bf03f0f34cd6a9d0213f56be93a0 : SUCCESS in 7m 39s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/c99bdf6cc8ce4a2b9ebdc0214e47660e : SUCCESS in 6m 27s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/8f9c6f21ad50426eb66eac337e54df19 : SUCCESS in 7m 37s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e76399a113e64992b78492829c057322 : SUCCESS in 6m 33s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/0bbe5449dca74b37b185882b273887a0 : SUCCESS in 10m 35s\n- openstack-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/e4d8ac17c2cd4aaf86446c67dfd5f546 : SUCCESS in 10m 13s\n- glance-code-constants-check https://zuul.opendev.org/t/openstack/build/d30918fe621e45bf9b1a01f4cf916f03 : SUCCESS in 6m 21s\n- glance-ceph-thin-provisioning https://zuul.opendev.org/t/openstack/build/b78a7b28dc62447aab395600c1a9ac3a : SUCCESS in 1h 29m 03s (non-voting)\n- tempest-integrated-storage-import https://zuul.opendev.org/t/openstack/build/9cf0de457edb4723a5f3404acbc3225a : SUCCESS in 1h 33m 21s\n- tempest-integrated-storage-import-standalone https://zuul.opendev.org/t/openstack/build/87000578dfa9484784d277bc4c483282 : SUCCESS in 1h 12m 05s\n- glance-multistore-cinder-import https://zuul.opendev.org/t/openstack/build/a40e27a19ac2416882906aabd5154c73 : SUCCESS in 1h 17m 00s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/c79de3e53c26414396016d14b8dd203a : SUCCESS in 1h 21m 19s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/566e20a8560044be84c49af26b76a39c : SUCCESS in 1h 12m 28s","accounts_in_message":[],"_revision_number":1},{"id":"d5c174d6057a5596fbc19089042f42af60dd6220","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-06 13:23:52.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"c5ff369f4465e2ae9e05a012d802e53271b0d33c","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-06 17:34:05.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/908b585a99dc4c86ad773b1c9b3e918d : SUCCESS in 54m 09s\n- tempest-integrated-storage https://zuul.opendev.org/t/openstack/build/696540e4d2ad4643be5f115ae1e771f8 : SUCCESS in 1h 14m 53s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/38c203ac8b6c49d6bee366dad3457ac5 : SUCCESS in 6m 43s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b29167dfdb8941a48ae41c618ed0f06a : SUCCESS in 5m 53s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/73d44e3bc8b6471192b19d4e904d76d1 : SUCCESS in 6m 15s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/be3d34887629435899920c53a3b22bbc : SUCCESS in 6m 20s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6ad3d934517d4445a181a0f1b45c3dbc : SUCCESS in 7m 16s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/b221cb519f2f4044af5f65b90474adf9 : SUCCESS in 8m 24s\n- openstack-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/3c3123722dbf465c86de61894f3110b7 : SUCCESS in 10m 33s\n- tempest-integrated-storage-import https://zuul.opendev.org/t/openstack/build/116fdf1aeb3c4dc091bcf966f0ac0792 : SUCCESS in 1h 22m 05s\n- tempest-integrated-storage-import-standalone https://zuul.opendev.org/t/openstack/build/f9abdd61f3324c5da3a03c12a9a9096e : SUCCESS in 1h 07m 13s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/d79eb62f71ca45bfaae5f6064e5a196a : SUCCESS in 55m 30s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/16f0d9713a4f4fd1b6e7fa9cd12ff333 : SUCCESS in 1h 14m 01s","accounts_in_message":[],"_revision_number":1},{"id":"26331c4d75a3103ff1b7a4f9601596a3c889da4b","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-06 17:34:06.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"5331f1e0badf0820a93264e6d7063927b2afe51b","revisions":{"5331f1e0badf0820a93264e6d7063927b2afe51b":{"kind":"REWORK","_number":1,"created":"2021-02-09 20:14:25.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/21/774721/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/21/774721/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/21/774721/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/21/774721/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/21/774721/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/21/774721/1"}}},"commit":{"parents":[{"commit":"cbb2af6e329c1bf6c1c39cb59f95aaad2c0d7a07","subject":"Make web-download revert all stores on fail","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/cbb2af6e329c1bf6c1c39cb59f95aaad2c0d7a07"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2021-02-09 18:10:31.000000000","tz":-480},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2021-02-09 20:33:05.000000000","tz":-480},"subject":"Make action wrapper support arbitrary properties","message":"Make action wrapper support arbitrary properties\n\nPreviously, the only attribute of the image that an import plugin\ncould change was the status. In order to convert the rest of the\nplugins to use the wrapper, we need to define additional properties\nthat they can mutate. This removes set_image_status() and replaces\nit with set_image_attribute() which allows a subset of those\nproperties to be set.\n\nThis also ends up passing the action wrapper to all plugin init\nmethods by way of just putting it straight into the kwargs that we\ngive to everything. That is added here as setup for the patches to\nfollow which depend on it.\n\nChange-Id: Id613cfd61760c383c7c3cc6aea3f37eecb5ed4d9\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/5331f1e0badf0820a93264e6d7063927b2afe51b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/5331f1e0badf0820a93264e6d7063927b2afe51b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"}},{"label":"Review-Priority","status":"MAY","applied_by":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}