)]}'
{"id":"openstack%2Fpython-cinderclient~476264","triplet_id":"openstack%2Fpython-cinderclient~master~I545b3f28e545b380842ea003dc38ad70cb413aee","project":"openstack/python-cinderclient","branch":"master","topic":"bug/1696556","hashtags":[],"change_id":"I545b3f28e545b380842ea003dc38ad70cb413aee","subject":"UnboundLocalError on message-list","status":"MERGED","created":"2017-06-21 20:38:10.000000000","updated":"2017-07-07 19:42:05.000000000","submitted":"2017-07-07 19:42:05.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"0ec007f3171496607dacd66be509d898a8dc8df9","_number":476264,"virtual_id_number":476264,"owner":{"_account_id":26155,"name":"Chaynika Saikia","email":"csaikia@asu.edu","username":"csaikia"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":2,"date":"2017-07-07 19:42:05.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},{"value":0,"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"value":0,"_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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":2,"date":"2017-07-07 15:38:19.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},{"value":2,"date":"2017-06-29 07:31:20.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"value":0,"_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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":1,"date":"2017-07-07 15:38:19.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},{"value":0,"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"value":0,"date":"2017-07-05 20:58:46.000000000","permitted_voting_range":{"min":0,"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":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},{"value":0,"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"value":0,"_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":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-06-21 20:43:47.000000000","updated_by":{"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},"reviewer":{"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},"state":"REVIEWER"},{"updated":"2017-06-29 07:31:20.000000000","updated_by":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"reviewer":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"state":"REVIEWER"},{"updated":"2017-07-05 20:58:46.000000000","updated_by":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"reviewer":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"state":"REVIEWER"},{"updated":"2017-07-07 15:38:19.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2017-07-07 19:42:05.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"0dc46ba3839b955dac35c2199104a5dc5b063492","author":{"_account_id":26155,"name":"Chaynika Saikia","email":"csaikia@asu.edu","username":"csaikia"},"date":"2017-06-21 20:38:10.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"f7253c86ca48d1306c9256277e425afc20bd66df","author":{"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},"date":"2017-06-21 20:43:47.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"7db22571a29ffdf41fd9e8e7a8450e2a15f41246","author":{"_account_id":26155,"name":"Chaynika Saikia","email":"csaikia@asu.edu","username":"csaikia"},"date":"2017-06-21 21:09:47.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"16d91215c93a4ab127757d21da4559ec5c5bed0b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-06-21 22:28:29.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/64/476264/2/check/gate-python-cinderclient-docs-ubuntu-xenial/b85a170//doc/build/html/ : SUCCESS in 4m 34s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/64/476264/2/check/gate-python-cinderclient-pep8-ubuntu-xenial/d3ffd48/ : SUCCESS in 3m 51s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/64/476264/2/check/gate-python-cinderclient-python27-ubuntu-xenial/178a36a/ : SUCCESS in 3m 24s\n- gate-python-cinderclient-python35 http://logs.openstack.org/64/476264/2/check/gate-python-cinderclient-python35/e78e745/ : SUCCESS in 3m 36s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/64/476264/2/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/e9699be/ : SUCCESS in 1h 17m 08s\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/64/476264/2/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/b0e9d04/ : SUCCESS in 15m 29s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/64/476264/2/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/89f0c03/ : SUCCESS in 15m 02s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/64/476264/2/check/gate-python-cinderclient-pylint-ubuntu-xenial/0594fd3/ : SUCCESS in 2m 33s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"045f5a555364090d64eca1b7253ef4e458d35cb4","author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"date":"2017-06-29 07:31:20.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"1cf187dd35a678ef61dda53dc754cec79b15aa47","author":{"_account_id":26155,"name":"Chaynika Saikia","email":"csaikia@asu.edu","username":"csaikia"},"date":"2017-07-07 14:02:07.000000000","message":"Patch Set 2:\n\n\u003e Uploaded patch set 2: Commit message was updated.\n\nHi Eric Harney and jessegler,\n\nI request you to review my changes for this patch.\n\nThanks,\nChaynika","accounts_in_message":[],"_revision_number":2},{"id":"512ff5fa1b445a24873cd4f7f6c56eec240e9d54","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2017-07-07 15:38:19.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"15b1102edfb51b2c2bc02715a60947722546b0e3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-07-07 15:38:31.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"db5915acd50224f3411e5a47749f61a09e85d426","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-07-07 16:42:56.000000000","message":"Patch Set 2: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/64/476264/2/gate/gate-python-cinderclient-docs-ubuntu-xenial/0ba798c//doc/build/html/ : SUCCESS in 4m 02s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/64/476264/2/gate/gate-python-cinderclient-pep8-ubuntu-xenial/35fa6e1/ : SUCCESS in 5m 10s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/64/476264/2/gate/gate-python-cinderclient-python27-ubuntu-xenial/72f2241/ : SUCCESS in 4m 22s\n- gate-python-cinderclient-python35 http://logs.openstack.org/64/476264/2/gate/gate-python-cinderclient-python35/808bb5d/ : SUCCESS in 4m 41s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/64/476264/2/gate/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/9f200ad/ : FAILURE in 1h 03m 38s","accounts_in_message":[],"_revision_number":2},{"id":"1b44f59f91b0109a03a93df83024c0336bef2f50","author":{"_account_id":26155,"name":"Chaynika Saikia","email":"csaikia@asu.edu","username":"csaikia"},"date":"2017-07-07 17:11:24.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"82315ecb63457dc57642641d46a9b2ceec20aa8c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-07-07 18:24:18.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/64/476264/2/check/gate-python-cinderclient-docs-ubuntu-xenial/a7eef8c//doc/build/html/ : SUCCESS in 2m 28s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/64/476264/2/check/gate-python-cinderclient-pep8-ubuntu-xenial/051259e/ : SUCCESS in 5m 25s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/64/476264/2/check/gate-python-cinderclient-python27-ubuntu-xenial/66f3438/ : SUCCESS in 2m 44s\n- gate-python-cinderclient-python35 http://logs.openstack.org/64/476264/2/check/gate-python-cinderclient-python35/c9bc30b/ : SUCCESS in 5m 23s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/64/476264/2/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/e8a4198/ : SUCCESS in 1h 06m 17s\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/64/476264/2/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/bc16441/ : SUCCESS in 26m 48s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/64/476264/2/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/3f465e1/ : SUCCESS in 26m 18s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/64/476264/2/check/gate-python-cinderclient-pylint-ubuntu-xenial/e08aa57/ : SUCCESS in 5m 23s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"89a818cb824788785cd116d62a7e271009e78e08","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-07-07 18:24:28.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"5a91fb735a61114609a4ccbd3880165622111c39","date":"2017-07-07 19:42:05.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2},{"id":"e23d06418ba5319cfafa015b97a3731bf55208ab","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-07-07 19:42:05.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/64/476264/2/gate/gate-python-cinderclient-docs-ubuntu-xenial/e06a71a//doc/build/html/ : SUCCESS in 7m 23s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/64/476264/2/gate/gate-python-cinderclient-pep8-ubuntu-xenial/c826011/ : SUCCESS in 7m 21s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/64/476264/2/gate/gate-python-cinderclient-python27-ubuntu-xenial/028bcb4/ : SUCCESS in 3m 47s\n- gate-python-cinderclient-python35 http://logs.openstack.org/64/476264/2/gate/gate-python-cinderclient-python35/948d43c/ : SUCCESS in 2m 20s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/64/476264/2/gate/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/b3806d9/ : SUCCESS in 1h 15m 50s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"f38c8c5df83b50fd3ef38f3db2456b54294ee0aa","revisions":{"f9984fc426324ee025a746bfff4aeb42dd3930e6":{"kind":"REWORK","_number":1,"created":"2017-06-21 20:38:10.000000000","uploader":{"_account_id":26155,"name":"Chaynika Saikia","email":"csaikia@asu.edu","username":"csaikia"},"ref":"refs/changes/64/476264/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/64/476264/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/64/476264/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/64/476264/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/64/476264/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/64/476264/1"}}},"commit":{"parents":[{"commit":"71dfec67434ab9108a5faeee094d0225a5970e0a","subject":"Merge \"Fix cmd options for updating a quota class\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/71dfec67434ab9108a5faeee094d0225a5970e0a"}]}],"author":{"name":"Chaynika Saikia","email":"csaikia@asu.edu","date":"2017-06-21 20:33:49.000000000","tz":-240},"committer":{"name":"Chaynika Saikia","email":"csaikia@asu.edu","date":"2017-06-21 20:37:17.000000000","tz":-240},"subject":"Closes-Bug: 1696556","message":"Closes-Bug: 1696556\n\n\u0027UnboundLocalError on message-list\u0027 fixed such that now\nif the user gives wrong filters, then the command properly\nprints WARNING message and ignores that filter. Previously,\nfor wrong filters, it was giving UnboundLocalError.\n\nChange-Id: I545b3f28e545b380842ea003dc38ad70cb413aee\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/f9984fc426324ee025a746bfff4aeb42dd3930e6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/f9984fc426324ee025a746bfff4aeb42dd3930e6"}]},"branch":"refs/heads/master"},"f38c8c5df83b50fd3ef38f3db2456b54294ee0aa":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2017-06-21 21:09:47.000000000","uploader":{"_account_id":26155,"name":"Chaynika Saikia","email":"csaikia@asu.edu","username":"csaikia"},"ref":"refs/changes/64/476264/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/64/476264/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/64/476264/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/64/476264/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/64/476264/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/64/476264/2"}}},"commit":{"parents":[{"commit":"71dfec67434ab9108a5faeee094d0225a5970e0a","subject":"Merge \"Fix cmd options for updating a quota class\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/71dfec67434ab9108a5faeee094d0225a5970e0a"}]}],"author":{"name":"Chaynika Saikia","email":"csaikia@asu.edu","date":"2017-06-21 20:33:49.000000000","tz":-240},"committer":{"name":"Chaynika Saikia","email":"csaikia@asu.edu","date":"2017-06-21 20:59:28.000000000","tz":-240},"subject":"UnboundLocalError on message-list","message":"UnboundLocalError on message-list\n\nThis bug for \u0027cinder message-list\u0027 has been fixed such that now\nif the user gives wrong filters, then the command properly\nprints WARNING message saying it is ignoring that filter. Previously,\nfor wrong filters which were not passed in the correct usage format,\nthe \u0027cinder message-list --filters\u0027 command was giving UnboundLocalError\nexception.\n\nThe extract_filters() method was previously assuming that all the\nmessage filters passed were in the format \"\u003cfilter_name\u003e\u003d\u003cvalue\u003e\",\nbecause of which if the user passed something like:\n\t\u0027cinder message-list --filters event_id:VOLUME_000002\u0027\nan UnboundLocalError was thrown.\n\nThe changes are made to the method such that if the user passes a\nfilter in invalid format, the cinder CLI ignores that filter while\nproviding the output.\n\nCloses-Bug: #1696556\n\nChange-Id: I545b3f28e545b380842ea003dc38ad70cb413aee\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/f38c8c5df83b50fd3ef38f3db2456b54294ee0aa"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/f38c8c5df83b50fd3ef38f3db2456b54294ee0aa"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
