)]}'
{"id":"openstack%2Fpython-cinderclient~413156","triplet_id":"openstack%2Fpython-cinderclient~master~I592259a2bc5a59804e21b16a8b9568587e3e0214","project":"openstack/python-cinderclient","branch":"master","hashtags":[],"change_id":"I592259a2bc5a59804e21b16a8b9568587e3e0214","subject":"Use \u0027reset-state\u0027 for objects other than volume","status":"ABANDONED","created":"2016-12-20 16:07:16.000000000","updated":"2017-06-23 14:16:20.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"8187c5f6b558952b226136e138b92d6aa9ea3353","_number":413156,"virtual_id_number":413156,"owner":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":-1,"date":"2017-02-15 20:50:42.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"}],"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":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},{"value":-1,"date":"2017-06-23 02:24:15.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"}],"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":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"}],"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":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-12-20 16:08:12.000000000","updated_by":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"reviewer":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"state":"REVIEWER"},{"updated":"2017-02-15 20:50:42.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2017-06-23 02:24:15.000000000","updated_by":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"reviewer":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"state":"REVIEWER"}],"messages":[{"id":"9848633d5c1155ef561c1269cd01a1c6f6fbbfb1","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2016-12-20 16:07:16.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"8204f32658edc632e5eb3fc57eafe37879ee6e0d","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2016-12-20 16:08:15.000000000","message":"Patch Set 1: Workflow-1","accounts_in_message":[],"_revision_number":1},{"id":"77f8167767afda35a798cb57f82b47ec276367b7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-20 17:28:25.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/56/413156/1/check/gate-python-cinderclient-docs-ubuntu-xenial/50d6ab1//doc/build/html/ : SUCCESS in 4m 31s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/56/413156/1/check/gate-python-cinderclient-pep8-ubuntu-xenial/ffadc93/ : SUCCESS in 4m 11s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/56/413156/1/check/gate-python-cinderclient-python27-ubuntu-xenial/858663e/ : SUCCESS in 4m 40s\n- gate-python-cinderclient-python35 http://logs.openstack.org/56/413156/1/check/gate-python-cinderclient-python35/8232853/ : SUCCESS in 4m 31s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/56/413156/1/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/38ce320/ : SUCCESS in 1h 19m 39s\n- gate-python-cinderclient-pypy http://logs.openstack.org/56/413156/1/check/gate-python-cinderclient-pypy/7dc484f/ : SUCCESS in 2m 45s (non-voting)\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/56/413156/1/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/a66f94d/ : SUCCESS in 19m 52s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/56/413156/1/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/83e777f/ : SUCCESS in 20m 18s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/56/413156/1/check/gate-python-cinderclient-pylint-ubuntu-xenial/2c20ee5/ : SUCCESS in 3m 49s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"f8e5894cfb44a6759faf8207bd3f70b232b8e0d4","author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"date":"2016-12-20 19:42:02.000000000","message":"Patch Set 1: Code-Review-1\n\nThis is definitely better than what we have today in cinder-client, the option that should have adopted at the beginning. But you are changing a behavior that is already solidified by users, and we will still have to keep the old behavior for 1/2 releases (I don\u0027t know how long its required). At that point OSC should be at pair with cinderclient (they ~90% for v1 and v2, v3 its almost just a wrapper).\n\nSo, I don\u0027t think the effort to change it is worth. And OSC is great:\n\nvolume set --state \u0027error\u0027\nvolume snapshot set --state \u0027error\u0027","accounts_in_message":[],"_revision_number":1},{"id":"addbcf1e0f393bfc0c9038ad9a0c36d98b71029e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-21 22:25:14.000000000","message":"Patch Set 1: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"77a542c75fb40a38f7093471d90cfe994dac0029","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-02-15 20:50:30.000000000","message":"Patch Set 2: Published edit on patch set 1","accounts_in_message":[],"_revision_number":2},{"id":"009d06007f65aada1d574c7c2922c1b38b84dac4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-02-15 20:50:42.000000000","message":"Patch Set 2: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":2},{"id":"adbc467e31995e18c09b3644bc30195b3e2e4235","author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"date":"2017-06-23 02:24:15.000000000","message":"Patch Set 2: Code-Review-1\n\nIt has been addressed here: https://review.openstack.org/#/c/446356/","accounts_in_message":[],"_revision_number":2},{"id":"8187c5f6b558952b226136e138b92d6aa9ea3353","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-06-23 14:16:20.000000000","message":"Abandoned\n\nhttps://review.openstack.org/#/c/446356/","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"8ba787dbf178781fc8ef2e1be6dbd35a69443039","revisions":{"a8c3f5e82f3b0719c0aaffe6bb12c866c2c15756":{"kind":"REWORK","_number":1,"created":"2016-12-20 16:07:16.000000000","uploader":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"ref":"refs/changes/56/413156/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/56/413156/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/56/413156/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/56/413156/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/56/413156/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/56/413156/1"}}},"commit":{"parents":[{"commit":"1da9f792dd450d683b32f0e7d3d66c3b3f9a2dfc","subject":"Merge \"Update hacking version\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/1da9f792dd450d683b32f0e7d3d66c3b3f9a2dfc"}]}],"author":{"name":"Eric Harney","email":"eharney@redhat.com","date":"2016-12-20 16:02:01.000000000","tz":-300},"committer":{"name":"Eric Harney","email":"eharney@redhat.com","date":"2016-12-20 16:05:23.000000000","tz":-300},"subject":"Use \u0027reset-state\u0027 for objects other than volume","message":"Use \u0027reset-state\u0027 for objects other than volume\n\nThis is mostly a POC to demonstrate how I think\nwe should clean up the various reset-X-state interfaces\nand provide a single command that can reset any\nof them.\n\nThis leaves \"reset-state\" still working as before\nfor volumes, but adds a \"--type\" argument so that\none can run\n\n$ cinder reset-state --type snapshot abcd-1234 --status error\n\nThis is still rough and needs to be polished, but\nI\u0027m interested in what people think in general of\nthe idea.\n\nChange-Id: I592259a2bc5a59804e21b16a8b9568587e3e0214\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/a8c3f5e82f3b0719c0aaffe6bb12c866c2c15756"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/a8c3f5e82f3b0719c0aaffe6bb12c866c2c15756"}]},"branch":"refs/heads/master"},"8ba787dbf178781fc8ef2e1be6dbd35a69443039":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2017-02-15 20:50:30.000000000","uploader":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"ref":"refs/changes/56/413156/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/56/413156/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/56/413156/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/56/413156/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/56/413156/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/56/413156/2"}}},"commit":{"parents":[{"commit":"1da9f792dd450d683b32f0e7d3d66c3b3f9a2dfc","subject":"Merge \"Update hacking version\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/1da9f792dd450d683b32f0e7d3d66c3b3f9a2dfc"}]}],"author":{"name":"Eric Harney","email":"eharney@redhat.com","date":"2016-12-20 16:02:01.000000000","tz":-300},"committer":{"name":"Eric Harney","email":"eharney@redhat.com","date":"2017-02-15 20:50:27.000000000","tz":0},"subject":"Use \u0027reset-state\u0027 for objects other than volume","message":"Use \u0027reset-state\u0027 for objects other than volume\n\nThis is mostly a POC to demonstrate how I think\nwe should clean up the various reset-X-state interfaces\nand provide a single command that can reset any\nof them.\n\nThis leaves \"reset-state\" still working as before\nfor volumes, but adds a \"--type\" argument so that\none can run\n\n$ cinder reset-state --type snapshot abcd-1234 --status error\n\nThis is still rough and needs to be polished, but\nI\u0027m interested in what people think in general of\nthe idea.\n\nImplements: blueprint client-reset-state-improvements\n\nChange-Id: I592259a2bc5a59804e21b16a8b9568587e3e0214\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8ba787dbf178781fc8ef2e1be6dbd35a69443039"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8ba787dbf178781fc8ef2e1be6dbd35a69443039"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}