)]}'
{"id":"openstack%2Fpython-cinderclient~311926","triplet_id":"openstack%2Fpython-cinderclient~master~Ic1918fcd6b59a44820a4bb6d48f384ed80aa8632","project":"openstack/python-cinderclient","branch":"master","topic":"bp/wrap-long-table-values","hashtags":[],"change_id":"Ic1918fcd6b59a44820a4bb6d48f384ed80aa8632","subject":"Wrap long values in CLI tables","status":"ABANDONED","created":"2016-05-03 03:25:39.000000000","updated":"2017-01-20 19:27:51.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"8dce5df8b5cb86a42a2b9775cebed4e8f0443c83","_number":311926,"virtual_id_number":311926,"owner":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"date":"2016-05-03 03:26:52.000000000","_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"value":1,"date":"2016-05-03 04:25:27.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"}],"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":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-1,"date":"2016-05-03 17:59:31.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"}],"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":{"rejected":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"all":[{"value":-1,"date":"2016-05-12 13:41:54.000000000","permitted_voting_range":{"min":-1,"max":0},"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"value":0,"date":"2016-05-10 19:50:18.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"}],"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":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-05-03 04:25:27.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2016-05-03 17:59:31.000000000","updated_by":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"reviewer":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"state":"REVIEWER"},{"updated":"2016-05-10 19:50:18.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"}],"messages":[{"id":"b84c3da8901cb00d7f9c77520789d72aa219123d","author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"date":"2016-05-03 03:25:39.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"dfd7689b5efdcff763400eef6991bfbba934c038","author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"date":"2016-05-03 03:26:52.000000000","message":"Patch Set 1:\n\nNot sure if we need to be extra cautious about people scraping CLI output, but I added a release note. If this is something we need to be concerned about, adding an option to wrap or not is trivial.","accounts_in_message":[],"_revision_number":1},{"id":"68fe929647b1728b73cf0e1e6fe4b0607edf2279","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-03 04:25:27.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/26/311926/1/check/gate-python-cinderclient-docs/afda8c8//doc/build/html/ : SUCCESS in 1m 56s\n- gate-python-cinderclient-pep8 http://logs.openstack.org/26/311926/1/check/gate-python-cinderclient-pep8/229b51c/ : SUCCESS in 1m 53s\n- gate-python-cinderclient-python27 http://logs.openstack.org/26/311926/1/check/gate-python-cinderclient-python27/0886322/ : SUCCESS in 1m 49s\n- gate-python-cinderclient-python34 http://logs.openstack.org/26/311926/1/check/gate-python-cinderclient-python34/bdf46b6/ : SUCCESS in 2m 02s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/26/311926/1/check/gate-tempest-dsvm-neutron-src-python-cinderclient/d46eddf/ : SUCCESS in 59m 02s\n- gate-python-cinderclient-pypy http://logs.openstack.org/26/311926/1/check/gate-python-cinderclient-pypy/846add4/ : SUCCESS in 2m 54s (non-voting)\n- gate-python-cinderclient-releasenotes http://docs-draft.openstack.org/26/311926/1/check/gate-python-cinderclient-releasenotes/ad150a9//releasenotes/build/html/ : SUCCESS in 1m 58s\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/26/311926/1/check/gate-cinderclient-dsvm-functional/eaa9518/ : SUCCESS in 19m 41s (non-voting)\n- gate-python-cinderclient-pylint http://logs.openstack.org/26/311926/1/check/gate-python-cinderclient-pylint/c6d02e4/ : SUCCESS in 2m 10s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"fbab2657a1f093ad373f065f26fcd86697e99a62","author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"date":"2016-05-03 17:59:31.000000000","message":"Patch Set 1: Code-Review-1\n\nI don\u0027t think we can wrap by default, since there are plenty of scripts that do grep and awk on this output and will break in weird and wonderful ways. Unfortunately a release note doesn\u0027t cover it for the client, since most of the people affect probably won\u0027t even notice it has been upgraded until stuff breaks and won\u0027t know where to find the release notes (distributor packaged stuff)","accounts_in_message":[],"_revision_number":1},{"id":"82ac3568280876b67d3dc580a19456c3f5f25d55","author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"date":"2016-05-04 14:36:44.000000000","message":"Patch Set 1:\n\nunfortunately I think you are right. I will see about adding an option with an environment variable that defaults to previous behavior. Perhaps devstack openrc can default to wrapping.","accounts_in_message":[],"_revision_number":1},{"id":"49daab48938769a92c469bd8f29e9f7dea0e1cf5","author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"date":"2016-05-08 15:13:39.000000000","message":"Patch Set 1:\n\nThanks, that sounds like a pragmatic solution. Might be worth seeing if the unified client can do any better from day one?","accounts_in_message":[],"_revision_number":1},{"id":"02794170a28f222ef27fc136ff0d9175936da27b","author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"date":"2016-05-12 13:41:54.000000000","message":"Patch Set 1: Workflow-1","accounts_in_message":[],"_revision_number":1},{"id":"5df461320be844aa52cb75e58ad5f20340532176","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2017-01-20 19:27:51.000000000","message":"Abandoned\n\nThis review is \u003e 4 weeks with unaddressed issues. Feel free to reactivate the review by pressing the restore button and addressing the issue on this review if it is still needed.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"202c79a6daabf0c3347e06675986abdfd5a2ef1e","revisions":{"202c79a6daabf0c3347e06675986abdfd5a2ef1e":{"kind":"REWORK","_number":1,"created":"2016-05-03 03:25:39.000000000","uploader":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"ref":"refs/changes/26/311926/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/26/311926/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/26/311926/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/26/311926/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/26/311926/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/26/311926/1"}}},"commit":{"parents":[{"commit":"8d79acda0f5ff0e33ece7c84e65f5168b9b465bf","subject":"Merge \"Use six.moves.urllib.parse urlencode\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8d79acda0f5ff0e33ece7c84e65f5168b9b465bf"}]}],"author":{"name":"Alex Meade","email":"mr.alex.meade@gmail.com","date":"2016-05-03 03:01:25.000000000","tz":-240},"committer":{"name":"Alex Meade","email":"mr.alex.meade@gmail.com","date":"2016-05-03 03:25:25.000000000","tz":-240},"subject":"Wrap long values in CLI tables","message":"Wrap long values in CLI tables\n\nThis change wraps values that appear in CLI tables to\na 45 character max. This is true for both print_list\nand print_dict.\n\nImplements Blueprint: wrap-long-table-values\n\nChange-Id: Ic1918fcd6b59a44820a4bb6d48f384ed80aa8632\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/202c79a6daabf0c3347e06675986abdfd5a2ef1e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/202c79a6daabf0c3347e06675986abdfd5a2ef1e"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}