)]}'
{"id":"openstack%2Fpython-cinderclient~244962","triplet_id":"openstack%2Fpython-cinderclient~master~Iad4bf76e87f84b6e57ec59d134b9413fcad16ce8","project":"openstack/python-cinderclient","branch":"master","topic":"bug/1502639","hashtags":[],"change_id":"Iad4bf76e87f84b6e57ec59d134b9413fcad16ce8","subject":"Add optional argument to list subcommand","status":"MERGED","created":"2015-11-13 02:27:27.000000000","updated":"2015-12-09 00:19:41.000000000","submitted":"2015-12-09 00:19:40.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f7864079d7caf7555c67034b5246da040689fb88","_number":244962,"virtual_id_number":244962,"owner":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"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":0,"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},{"value":2,"date":"2015-12-09 00:19:40.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2015-12-08 23:45:25.000000000","_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":9176,"name":"Tomoki Sekiyama","email":"tomoki.sekiyama@gmail.com","username":"tsekiyama"},{"value":0,"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"value":0,"date":"2015-12-08 23:39:59.000000000","_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"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":"2015-12-08 20:29:10.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":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":9176,"name":"Tomoki Sekiyama","email":"tomoki.sekiyama@gmail.com","username":"tsekiyama"},{"value":0,"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"value":0,"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},{"value":2,"date":"2015-12-08 22:49:02.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"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":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":9176,"name":"Tomoki Sekiyama","email":"tomoki.sekiyama@gmail.com","username":"tsekiyama"},{"value":0,"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"value":0,"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},{"value":1,"date":"2015-12-08 22:49:02.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"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":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":9176,"name":"Tomoki Sekiyama","email":"tomoki.sekiyama@gmail.com","username":"tsekiyama"},{"value":0,"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"value":0,"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"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":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":9176,"name":"Tomoki Sekiyama","email":"tomoki.sekiyama@gmail.com","username":"tsekiyama"},{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-11-13 03:25:56.000000000","updated_by":{"_account_id":9176,"name":"Tomoki Sekiyama","email":"tomoki.sekiyama@gmail.com","username":"tsekiyama"},"reviewer":{"_account_id":9176,"name":"Tomoki Sekiyama","email":"tomoki.sekiyama@gmail.com","username":"tsekiyama"},"state":"REVIEWER"},{"updated":"2015-11-13 03:26:14.000000000","updated_by":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"reviewer":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"state":"REVIEWER"},{"updated":"2015-11-16 11:42:29.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":"2015-12-04 10:04:18.000000000","updated_by":{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},"reviewer":{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},"state":"REVIEWER"},{"updated":"2015-12-08 20:29:10.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":"2015-12-08 22:49:02.000000000","updated_by":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"reviewer":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"state":"REVIEWER"},{"updated":"2015-12-08 23:39:59.000000000","updated_by":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"reviewer":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"state":"REVIEWER"},{"updated":"2015-12-09 00:19:40.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"3c757edd69fb5081872017ad8143ee03989c11ae","author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"date":"2015-11-13 02:27:27.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a5b016295175a0c43c2147141c09cafca9753797","author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"date":"2015-11-13 02:31:27.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"81d0e246f492e7e5bd3066ca42919db6ce136106","author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"date":"2015-11-13 02:33:25.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"458675aace0259904430056ec49ef3d4025b8efa","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-13 03:43:07.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-python26 http://logs.openstack.org/62/244962/3/check/gate-python-cinderclient-python26/df23dfe/ : SUCCESS in 1m 17s\n- gate-python-cinderclient-pep8 http://logs.openstack.org/62/244962/3/check/gate-python-cinderclient-pep8/e286f6d/ : SUCCESS in 58s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/62/244962/3/check/gate-python-cinderclient-docs/770904f//doc/build/html/ : SUCCESS in 59s\n- gate-python-cinderclient-python27 http://logs.openstack.org/62/244962/3/check/gate-python-cinderclient-python27/b375fed/ : SUCCESS in 1m 13s\n- gate-python-cinderclient-python34 http://logs.openstack.org/62/244962/3/check/gate-python-cinderclient-python34/df3c346/ : SUCCESS in 1m 19s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/62/244962/3/check/gate-tempest-dsvm-neutron-src-python-cinderclient/9299eeb/ : SUCCESS in 1h 09m 36s\n- gate-python-cinderclient-pypy http://logs.openstack.org/62/244962/3/check/gate-python-cinderclient-pypy/0f24100/ : SUCCESS in 1m 58s (non-voting)\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/62/244962/3/check/gate-cinderclient-dsvm-functional/d4d51d6/ : SUCCESS in 25m 14s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"bb452d1182491140a9f0e5189461e92a246407dd","author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"date":"2015-11-16 11:42:29.000000000","message":"Patch Set 3: Code-Review-1\n\nAdding \u0027--detail\u0027 param is good idea but I should use this param in the REST API call too.\n\nIMO, 2 views is enough: detailed and non-detaild","accounts_in_message":[],"_revision_number":3},{"id":"7ea4e9a0d8ad2764cd66f9adb28b988134a6abeb","author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"date":"2015-11-16 16:28:32.000000000","message":"Patch Set 3:\n\nHi Ivan,\n\nThank you for your comment.\n\nCurrently Cinder already has two types of volume list in the API, summary and detail. The summary view only returns volume ID and Name, but this implementation is same as Nova\u0027s one.\n\nAlso detail view return much information related to volume, and this also same implementation of Nova\u0027s one. I think changing API affects backward compatibility, so it is better to use API as is.\n\nNova list CLI command calls detail list API and then always masks unnecessary information to avoid user\u0027s confusion. So I\u0027d like to try same implementation to take into cinder client.\n\nIMO these are important fields and others should be masked as default view.(only show --detail is specified)\n\n[ID, Status, Name, Size, Volume Type, Bootable, Attached to]\n\nAlso --minimal and --fields are supported at Nova, I think --fields is worth to add cinder client when user want to customize their necessary columns.\n\nAny thought?","accounts_in_message":[],"_revision_number":3},{"id":"38d97d2ea8f646ae554b3f44289f81c82f0e3137","author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"date":"2015-11-16 16:40:03.000000000","message":"Patch Set 3:\n\nMitsuhiro, \n\nAs mentioned above, we\u0027ve already have \u0027summary\u0027 and \u0027detailed\u0027 view in the API. I\u0027m OK if we\u0027ll have a similar feature in the client.\n\nIntroducing more options in the client to show more or less columns makes client more complicated both from the developers and users point of view.\n\nIf we already pass something from API to client, why we can\u0027t just show it to users? \n\nAlso, I like an option to show only needed fields.","accounts_in_message":[],"_revision_number":3},{"id":"25f262d1da6dc9043d96cad575d5c8c8e3388f67","author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"date":"2015-11-16 17:02:34.000000000","message":"Patch Set 3:\n\nI understand that adding more options such as --detail or --minimal to increase confusion for users/admins.\n\nI\u0027d like to suggest that we change the default CLI list view only showing these eight fields same behaviour of Juno.\n\n[ID, Status, Name, Size, Volume Type, Bootable, Attached to]\n(Just remove Migration Status and Multiattach from default CLI list view.)\n\nAnd then add --fields option to show the list with more flexibility.","accounts_in_message":[],"_revision_number":3},{"id":"fdb01c26b18670e5f14d9f34ec671c120297b127","author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"date":"2015-11-17 22:25:55.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"b8edd7f1b3a61251333db0b7260305dfb4ff2be7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-17 23:43:33.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-python26 http://logs.openstack.org/62/244962/4/check/gate-python-cinderclient-python26/c73fdff/ : SUCCESS in 1m 40s\n- gate-python-cinderclient-pep8 http://logs.openstack.org/62/244962/4/check/gate-python-cinderclient-pep8/436789d/ : SUCCESS in 1m 00s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/62/244962/4/check/gate-python-cinderclient-docs/19daa04//doc/build/html/ : SUCCESS in 1m 05s\n- gate-python-cinderclient-python27 http://logs.openstack.org/62/244962/4/check/gate-python-cinderclient-python27/9416045/ : SUCCESS in 1m 05s\n- gate-python-cinderclient-python34 http://logs.openstack.org/62/244962/4/check/gate-python-cinderclient-python34/792f336/ : SUCCESS in 1m 11s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/62/244962/4/check/gate-tempest-dsvm-neutron-src-python-cinderclient/90a42bf/ : SUCCESS in 1h 08m 50s\n- gate-python-cinderclient-pypy http://logs.openstack.org/62/244962/4/check/gate-python-cinderclient-pypy/fc8f45e/ : SUCCESS in 1m 27s (non-voting)\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/62/244962/4/check/gate-cinderclient-dsvm-functional/4760a5d/ : SUCCESS in 25m 55s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"fba7711919bb0751b9c505a8f8e834bcbdc4a164","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2015-11-28 17:07:18.000000000","message":"Patch Set 4: Code-Review-1\n\n(1 comment)\n\nI am ok with this approach but am concerned that there is no checking of the fields requested.  Seems like we should do some value checking and provide an appropriate error if invalid input is provided.","accounts_in_message":[],"_revision_number":4},{"id":"15caf8e90ea4539f65a31d03173c7af83d9e85be","author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"date":"2015-11-29 01:20:44.000000000","message":"Patch Set 4:\n\nJay,\n\nThank you for the review. I will try to add validation for arguments.","accounts_in_message":[],"_revision_number":4},{"id":"a9c37e2dd5e41b26631c9efe494e01811f35bba7","author":{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},"date":"2015-12-04 10:04:18.000000000","message":"Patch Set 4: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"0cdc38b101de58cfcd78d71a919076394b49a500","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2015-12-04 16:13:21.000000000","message":"Patch Set 4:\n\nJust had discussion in IRC that the print_list function in utils.py will just ignore fields that don\u0027t exist.  So, there is validation there.  Mituhiro suggested that we update the help text to indicate the fact that invalid fields are just ignored.  I am good with that approach.","accounts_in_message":[],"_revision_number":4},{"id":"2fd86f13234ebbcfe5a6ca56a7c0a8c3a9f1be38","author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"date":"2015-12-04 16:44:33.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"941e9e275b8c23b908e804f55886a5820b89a0a0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-04 18:32:45.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/62/244962/5/check/gate-python-cinderclient-pep8/1627112/ : SUCCESS in 53s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/62/244962/5/check/gate-python-cinderclient-docs/e11ea4f//doc/build/html/ : SUCCESS in 57s\n- gate-python-cinderclient-python27 http://logs.openstack.org/62/244962/5/check/gate-python-cinderclient-python27/43b55cb/ : SUCCESS in 1m 12s\n- gate-python-cinderclient-python34 http://logs.openstack.org/62/244962/5/check/gate-python-cinderclient-python34/da543e6/ : SUCCESS in 1m 21s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/62/244962/5/check/gate-tempest-dsvm-neutron-src-python-cinderclient/93a51c5/ : SUCCESS in 1h 09m 18s\n- gate-python-cinderclient-pypy http://logs.openstack.org/62/244962/5/check/gate-python-cinderclient-pypy/04e72f2/ : SUCCESS in 1m 50s (non-voting)\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/62/244962/5/check/gate-cinderclient-dsvm-functional/5ad5a2d/ : SUCCESS in 21m 58s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"1edfd54b33c6f41a58c9f893e18283180df0de1d","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2015-12-08 20:29:10.000000000","message":"Patch Set 5: Code-Review+2\n\nLooks fine to me, thanks!","accounts_in_message":[],"_revision_number":5},{"id":"c587ba2e40357d910584a95fb67d134d7c0fbcc7","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2015-12-08 22:49:02.000000000","message":"Patch Set 5: Code-Review+2 Workflow+1\n\nLooks good Mitsuhiro.  Thank you!","accounts_in_message":[],"_revision_number":5},{"id":"447546e683f3af0f9d32be7087d36f1116c7f908","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-08 22:49:08.000000000","message":"Patch Set 5: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":5},{"id":"77992286b55d242b7a597d8ce3ebdea80b35ddf8","author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"date":"2015-12-08 23:39:59.000000000","message":"Patch Set 5:\n\nJay beat me to it.  I tested this out and it\u0027s fine.  Only thing that I would object to is that there is no way to find out the valid --fields entries.  since you get no feedback from \u0027bad\u0027 fields items as they are ignored.","accounts_in_message":[],"_revision_number":5},{"id":"51e1669e660e48e2e0a25bcd94da5d586704e86e","author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"date":"2015-12-08 23:45:25.000000000","message":"Patch Set 5:\n\nHi Walter,\n\nUser can find valid entries by using cinder show. It was mentioned in the help message and this is same spec of Nova\u0027s \"nova list --fields\" command.\n\n\"Use the show command to see which fields are available.\"","accounts_in_message":[],"_revision_number":5},{"id":"54cdcb7aa7fea338d04d7ebbd95c2419d26fe99f","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2015-12-08 23:46:35.000000000","message":"Patch Set 5:\n\nI had the same concern too until I noticed the help text directed users to the show command to see what fields are available. That\u0027s probably the safest option since if we called out the fields explicitly here, they would get out if sync sooner or later.","accounts_in_message":[],"_revision_number":5},{"id":"26e19ca6b994a3b992119ab33f407b40dec71fd3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-09 00:19:40.000000000","message":"Patch Set 5: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/62/244962/5/gate/gate-python-cinderclient-docs/9845566//doc/build/html/ : SUCCESS in 1m 36s\n- gate-python-cinderclient-pep8 http://logs.openstack.org/62/244962/5/gate/gate-python-cinderclient-pep8/084fdca/ : SUCCESS in 1m 33s\n- gate-python-cinderclient-python27 http://logs.openstack.org/62/244962/5/gate/gate-python-cinderclient-python27/fc1aa6c/ : SUCCESS in 1m 42s\n- gate-python-cinderclient-python34 http://logs.openstack.org/62/244962/5/gate/gate-python-cinderclient-python34/9e55844/ : SUCCESS in 1m 41s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/62/244962/5/gate/gate-tempest-dsvm-neutron-src-python-cinderclient/28660f4/ : SUCCESS in 1h 13m 51s","accounts_in_message":[],"_revision_number":5},{"id":"23cea5552ea4c7ca9c1dec46c30ba6b8881ff430","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-09 00:19:41.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":5}],"current_revision_number":5,"current_revision":"c38b35cb61baff1f9efbd61508890cdf632e188f","revisions":{"5ddd38c0e8c030ac1142f6d95a6762a0acca81a4":{"kind":"REWORK","_number":1,"created":"2015-11-13 02:27:27.000000000","uploader":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"ref":"refs/changes/62/244962/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/62/244962/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/1"}}},"commit":{"parents":[{"commit":"2ca2b96723aa41e81da37628386f01727ecbb2cb","subject":"Merge \"Implement cinder type-show\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/2ca2b96723aa41e81da37628386f01727ecbb2cb"}]}],"author":{"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@hds.com","date":"2015-11-13 01:57:40.000000000","tz":-300},"committer":{"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@hds.com","date":"2015-11-13 02:22:01.000000000","tz":-300},"subject":"Add optional arguments to list subcommand","message":"Add optional arguments to list subcommand\n\nCinder list command was intended to show summary of volumes,\nbut currently the command shows many entries than we expected.\n\nThis fix adds three options to show volume list more efficiently.\n- minimal : Only volume id and name are shown.\n- field   : This option allows users to show specified entries.\n- detail  : Additional entries such as multiattach and migration\n            status are shown.\n\nChange-Id: Iad4bf76e87f84b6e57ec59d134b9413fcad16ce8\nCloses-Bug: #1502639\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/5ddd38c0e8c030ac1142f6d95a6762a0acca81a4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/5ddd38c0e8c030ac1142f6d95a6762a0acca81a4"}]},"branch":"refs/heads/master"},"130d644f9696072d2f919bb01d0a214986966aea":{"kind":"REWORK","_number":2,"created":"2015-11-13 02:31:27.000000000","uploader":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"ref":"refs/changes/62/244962/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/62/244962/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/2"}}},"commit":{"parents":[{"commit":"2ca2b96723aa41e81da37628386f01727ecbb2cb","subject":"Merge \"Implement cinder type-show\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/2ca2b96723aa41e81da37628386f01727ecbb2cb"}]}],"author":{"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@hds.com","date":"2015-11-13 01:57:40.000000000","tz":-300},"committer":{"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@hds.com","date":"2015-11-13 02:31:03.000000000","tz":-300},"subject":"Add optional arguments to list subcommand","message":"Add optional arguments to list subcommand\n\nCinder list command was intended to show summary of volumes,\nbut currently the command shows many entries than we expected.\n\nThis fix adds three options to show volume list more efficiently.\n- minimal : Only volume id and name are shown.\n- field   : This option allows users to show specified entries.\n- detail  : Additional entries such as multiattach and migration\n            status are shown.\n\nChange-Id: Iad4bf76e87f84b6e57ec59d134b9413fcad16ce8\nCloses-Bug: #1502639\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/130d644f9696072d2f919bb01d0a214986966aea"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/130d644f9696072d2f919bb01d0a214986966aea"}]},"branch":"refs/heads/master"},"6b382f7c533c3e010a452978b45006b23e4fa2c1":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2015-11-13 02:33:25.000000000","uploader":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"ref":"refs/changes/62/244962/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/62/244962/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/3"}}},"commit":{"parents":[{"commit":"2ca2b96723aa41e81da37628386f01727ecbb2cb","subject":"Merge \"Implement cinder type-show\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/2ca2b96723aa41e81da37628386f01727ecbb2cb"}]}],"author":{"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@hds.com","date":"2015-11-13 01:57:40.000000000","tz":-300},"committer":{"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@hds.com","date":"2015-11-13 02:33:14.000000000","tz":-300},"subject":"Add optional arguments to list subcommand","message":"Add optional arguments to list subcommand\n\nCinder list command was intended to show summary of volumes,\nbut currently the command shows many entries than we expected.\n\nThis fix adds three options to show volume list more efficiently.\n- minimal : Only volume id and name are shown.\n- field   : This option allows users to show specified entries.\n- detail  : Additional entries such as multiattach and migration\n            status are shown.\n\nDocImpact\nChange-Id: Iad4bf76e87f84b6e57ec59d134b9413fcad16ce8\nCloses-Bug: #1502639\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/6b382f7c533c3e010a452978b45006b23e4fa2c1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/6b382f7c533c3e010a452978b45006b23e4fa2c1"}]},"branch":"refs/heads/master"},"92891afc069f524588040dd3887f94c83fc00f4b":{"kind":"REWORK","_number":4,"created":"2015-11-17 22:25:55.000000000","uploader":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"ref":"refs/changes/62/244962/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/62/244962/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/4"}}},"commit":{"parents":[{"commit":"2ca2b96723aa41e81da37628386f01727ecbb2cb","subject":"Merge \"Implement cinder type-show\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/2ca2b96723aa41e81da37628386f01727ecbb2cb"}]}],"author":{"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@hds.com","date":"2015-11-13 01:57:40.000000000","tz":-300},"committer":{"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@hds.com","date":"2015-11-17 22:23:56.000000000","tz":-300},"subject":"Add optional argument to list subcommand","message":"Add optional argument to list subcommand\n\nCinder list command was intended to show summary of\nvolumes, but currently the command shows many entries\nthan we expected.\n\nThis patch adds these two fixes to show volume list\nmore simply.\n\n- Remove \u0027migration status\u0027 and \u0027multiattach\u0027 from\n  default list view.\n\n- Add \u0027--field\u0027 optional argument to \u0027list\u0027 subcommand.\n  This argument allows users to show only specified\n  entries.\n\nDocImpact\nChange-Id: Iad4bf76e87f84b6e57ec59d134b9413fcad16ce8\nCloses-Bug: #1502639\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/92891afc069f524588040dd3887f94c83fc00f4b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/92891afc069f524588040dd3887f94c83fc00f4b"}]},"branch":"refs/heads/master"},"c38b35cb61baff1f9efbd61508890cdf632e188f":{"kind":"REWORK","_number":5,"created":"2015-12-04 16:44:33.000000000","uploader":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"ref":"refs/changes/62/244962/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/62/244962/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/62/244962/5"}}},"commit":{"parents":[{"commit":"2ca2b96723aa41e81da37628386f01727ecbb2cb","subject":"Merge \"Implement cinder type-show\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/2ca2b96723aa41e81da37628386f01727ecbb2cb"}]}],"author":{"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@hds.com","date":"2015-11-13 01:57:40.000000000","tz":-300},"committer":{"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@hds.com","date":"2015-12-04 16:43:06.000000000","tz":-300},"subject":"Add optional argument to list subcommand","message":"Add optional argument to list subcommand\n\nCinder list command was intended to show summary of\nvolumes, but currently the command shows many entries\nthan we expected.\n\nThis patch adds these two fixes to show volume list\nmore simply.\n\n- Remove \u0027migration status\u0027 and \u0027multiattach\u0027 from\n  default list view.\n\n- Add \u0027--field\u0027 optional argument to \u0027list\u0027 subcommand.\n  This argument allows users to show only specified\n  entries. Unavailable fields will be ignored from\n  the list result.\n\nDocImpact\nChange-Id: Iad4bf76e87f84b6e57ec59d134b9413fcad16ce8\nCloses-Bug: #1502639\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/c38b35cb61baff1f9efbd61508890cdf632e188f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/c38b35cb61baff1f9efbd61508890cdf632e188f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}