)]}'
{"id":"openstack%2Fpython-cinderclient~332480","triplet_id":"openstack%2Fpython-cinderclient~master~Ib427f7e76e8e28fee97b75413c3032fbee75af86","project":"openstack/python-cinderclient","branch":"master","topic":"(detached","hashtags":[],"change_id":"Ib427f7e76e8e28fee97b75413c3032fbee75af86","subject":"Showing the metadata readonly value as a separate field","status":"MERGED","created":"2016-06-21 21:20:22.000000000","updated":"2016-10-18 17:03:16.000000000","submitted":"2016-10-18 17:03:16.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":5,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"fb76a484625aa191b55a4874c31ddd2b1976a950","_number":332480,"virtual_id_number":332480,"owner":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"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":"2016-10-18 17:03:16.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},{"value":0,"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},{"value":0,"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},{"value":0,"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},{"value":0,"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"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":"2016-09-28 02:33:05.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":1,"date":"2016-10-18 02:58:01.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},{"value":0,"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},{"value":2,"date":"2016-10-18 16:03:32.000000000","_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":1,"date":"2016-09-27 21:38:38.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},{"value":0,"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},{"value":1,"date":"2016-10-07 10:36:01.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"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":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"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":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},{"value":0,"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},{"value":1,"date":"2016-10-18 16:03:32.000000000","_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"date":"2016-09-27 21:15:04.000000000","_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},{"value":0,"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},{"value":0,"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"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":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},{"value":0,"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},{"value":0,"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},{"value":0,"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},{"value":0,"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"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":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-07-06 14:30:23.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":"2016-07-25 16:59:32.000000000","updated_by":{"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},"reviewer":{"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},"state":"REVIEWER"},{"updated":"2016-08-29 12:46:10.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":"2016-09-27 21:15:04.000000000","updated_by":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"reviewer":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"state":"REVIEWER"},{"updated":"2016-09-27 21:38:38.000000000","updated_by":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"reviewer":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"state":"REVIEWER"},{"updated":"2016-09-28 02:33:05.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":"2016-10-07 10:36:01.000000000","updated_by":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"reviewer":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"state":"REVIEWER"},{"updated":"2016-10-18 02:58:01.000000000","updated_by":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"reviewer":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"state":"REVIEWER"},{"updated":"2016-10-18 16:03:32.000000000","updated_by":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"reviewer":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"state":"REVIEWER"},{"updated":"2016-10-18 17:03:16.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"5f998f2eed606fb62d85f6762e3dd1ac55b8982a","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-06-21 21:20:22.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e66bce547591ee04906379f30f8ec61ed5d12ed6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-21 22:25:14.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/80/332480/1/check/gate-python-cinderclient-docs/307838e//doc/build/html/ : SUCCESS in 2m 49s\n- gate-python-cinderclient-pep8 http://logs.openstack.org/80/332480/1/check/gate-python-cinderclient-pep8/8a2bb06/ : SUCCESS in 3m 08s\n- gate-python-cinderclient-python27 http://logs.openstack.org/80/332480/1/check/gate-python-cinderclient-python27/b849f75/ : SUCCESS in 6m 18s\n- gate-python-cinderclient-python34 http://logs.openstack.org/80/332480/1/check/gate-python-cinderclient-python34/3112fef/ : SUCCESS in 6m 24s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/80/332480/1/check/gate-tempest-dsvm-neutron-src-python-cinderclient/06f1b25/ : SUCCESS in 1h 03m 34s\n- gate-python-cinderclient-pypy http://logs.openstack.org/80/332480/1/check/gate-python-cinderclient-pypy/4c42059/ : FAILURE in 5m 48s (non-voting)\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/80/332480/1/check/gate-cinderclient-dsvm-functional/59f924d/ : SUCCESS in 23m 29s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/80/332480/1/check/gate-cinderclient-dsvm-functional-identity-v3-only-nv/f171074/ : FAILURE in 21m 14s (non-voting)\n- gate-python-cinderclient-pylint http://logs.openstack.org/80/332480/1/check/gate-python-cinderclient-pylint/bf41f03/ : SUCCESS in 9m 08s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"3340e69cf0fe4f513d0569e946c62b68e69d2d3b","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-06-29 18:43:22.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"dfb89f5b00d6a7dbadd77a01400e2d8fe1282d31","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-29 19:42:54.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/80/332480/2/check/gate-python-cinderclient-docs/9b5b5f6//doc/build/html/ : SUCCESS in 4m 03s\n- gate-python-cinderclient-pep8 http://logs.openstack.org/80/332480/2/check/gate-python-cinderclient-pep8/845e775/ : SUCCESS in 3m 21s\n- gate-python-cinderclient-python27 http://logs.openstack.org/80/332480/2/check/gate-python-cinderclient-python27/fed09af/ : SUCCESS in 3m 51s\n- gate-python-cinderclient-python34 http://logs.openstack.org/80/332480/2/check/gate-python-cinderclient-python34/8b5732c/ : SUCCESS in 3m 51s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/80/332480/2/check/gate-tempest-dsvm-neutron-src-python-cinderclient/12a39ee/ : SUCCESS in 56m 18s\n- gate-python-cinderclient-pypy http://logs.openstack.org/80/332480/2/check/gate-python-cinderclient-pypy/2305897/ : FAILURE in 4m 14s (non-voting)\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/80/332480/2/check/gate-cinderclient-dsvm-functional/db063a9/ : SUCCESS in 21m 53s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/80/332480/2/check/gate-cinderclient-dsvm-functional-identity-v3-only-nv/e9625b7/ : FAILURE in 23m 36s (non-voting)\n- gate-python-cinderclient-pylint http://logs.openstack.org/80/332480/2/check/gate-python-cinderclient-pylint/4d72328/ : SUCCESS in 3m 58s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"1a94e23fcf140485ef27f5ebe3c52487df5075ef","author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"date":"2016-06-30 20:35:47.000000000","message":"Patch Set 2: Code-Review-1\n\nI\u0027ve got a couple of comments here:\n\n1. It seems a little bit odd that readonly now shows up as a metadata entry and has its own entry, this might be fine but maybe we could consider not displaying it in metadata so it\u0027s not in two places\n\n2. I think readonly should also get its own line when the volume info is displayed after issuing a \u0027cinder create\u0027, it shows all of the same info as doing cinder show.","accounts_in_message":[],"_revision_number":2},{"id":"2249cd7522bd7559b69877553ab9ff6cbdf705ae","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-06-30 22:21:10.000000000","message":"Patch Set 2:\n\nHi Nate,\n\n1.) I think this might be confusing to see the readonly field separately and not in the volume metadata as expected at the same time. This patch i believe is more like a visual aid for output when metadata field is huge.\n\nApproach1:\nIf we agree to have it removed from metadata, is it sufficient to pop up it after it is received by the cinder client? My objection here is the data is inconsistent because cinder show api response contains readonly field  but the output would hide it.\n\nApproach2:\nElse do I need to make change to the api response?. I believe this requires a microversion bump\n\n2.)  I agree. readonly should show up in both cases in cinder create and cinder show. I will cater to that. I was skeptical initially as the bug was specific to cinder show. I will fix this.\n\nThanks for the review.  Let me know what you feel!!","accounts_in_message":[],"_revision_number":2},{"id":"f7bc74330c151f07084c0c9d41ab75dce54fcd45","author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"date":"2016-06-30 22:27:11.000000000","message":"Patch Set 2:\n\nYeah it could get complicated, if we really did make readonly its own field and removed it from the metadata there would have to be a lot of changes on the server side to accommodate that, just for a more convenient table view. I think leaving it is okay. Definitely add it to show though :). Thanks!","accounts_in_message":[],"_revision_number":2},{"id":"da9f246c04a7d2a401ad309282fb09fec3b70c94","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-07-01 00:29:48.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"a67f2c438dd0b0e0a52b2e77b93cf085d0cdc310","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-07-01 00:33:54.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"d41d7de9d0a64aa9730c9e6b14e0f2edcfa0e1c2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-01 01:40:58.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/80/332480/4/check/gate-python-cinderclient-docs/b7e4cd6//doc/build/html/ : SUCCESS in 2m 05s\n- gate-python-cinderclient-pep8 http://logs.openstack.org/80/332480/4/check/gate-python-cinderclient-pep8/802fcb3/ : SUCCESS in 3m 24s\n- gate-python-cinderclient-python27 http://logs.openstack.org/80/332480/4/check/gate-python-cinderclient-python27/faa1d0d/ : SUCCESS in 4m 34s\n- gate-python-cinderclient-python34 http://logs.openstack.org/80/332480/4/check/gate-python-cinderclient-python34/46c4d50/ : SUCCESS in 3m 22s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/80/332480/4/check/gate-tempest-dsvm-neutron-src-python-cinderclient/4925ccf/ : SUCCESS in 1h 04m 41s\n- gate-python-cinderclient-pypy http://logs.openstack.org/80/332480/4/check/gate-python-cinderclient-pypy/ca0e8f2/ : SUCCESS in 4m 08s (non-voting)\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/80/332480/4/check/gate-cinderclient-dsvm-functional/2d23dd7/ : SUCCESS in 22m 42s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/80/332480/4/check/gate-cinderclient-dsvm-functional-identity-v3-only-nv/4c751e4/ : FAILURE in 20m 57s (non-voting)\n- gate-python-cinderclient-pylint http://logs.openstack.org/80/332480/4/check/gate-python-cinderclient-pylint/c1e92dc/ : SUCCESS in 2m 45s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"0f7c1b6046cbebc328dba525a4f51726bb6737a4","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-07-11 18:41:14.000000000","message":"Topic changed from (detached to bug/1307098","accounts_in_message":[],"_revision_number":4},{"id":"b86b55270b6b14088ba505177dd8ff48302e8cf9","author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"date":"2016-07-25 16:16:24.000000000","message":"Patch Set 4: Code-Review+1\n\nThis still feels a little weird to me, like it\u0027s not the perfect solution. I guess it seems to me like readonly might deserve its own argument in creation rather than being inseted with --metadata readonly\u003dTrue if its going to get displayed like this (like --readonly True), but at this point it seems like it\u0027s just a matter of opinion. If others think this is a good solution it\u0027s cool with me, I\u0027d be interested in what the original bug reporter thinks too. I\u0027ll give a +1 for now though, since for what it is I think the code looks good.","accounts_in_message":[],"_revision_number":4},{"id":"189ac363bff023d8a8a349b4f97a98721b05fd97","author":{"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},"date":"2016-07-25 16:59:32.000000000","message":"Patch Set 4:\n\n(2 comments)\n\nJust a suggestion, otherwise LGTM","accounts_in_message":[],"_revision_number":4},{"id":"5ba2d1098d8ef1d14d2a0a0a82feb990baf6b1dc","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-07-26 02:10:29.000000000","message":"Patch Set 4:\n\n@Nate, Thanks for your review. I kind of agree it should be an optional parameter during create. Currently we need an additional step of creating \u0026 cinder readonly-mode-update for a volume.\n\nSince readonly wont be part of metadata, This will require restructuring the whole volume ovo and several methods like _from_db_object. For now at-least, it looks like a overkill. Lets see what the others have to say for this.","accounts_in_message":[],"_revision_number":4},{"id":"56e0d5210441f5579e78142ca854355dafd58ed1","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-07-26 02:17:04.000000000","message":"Patch Set 4:\n\n@Szymon: Thanks. Thats a valuable suggestion. I will make that change in the next patchset","accounts_in_message":[],"_revision_number":4},{"id":"938f823dd991683ea6a2d163ae54e06748c4343b","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-07-26 02:36:11.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"b092ba1d5d76ff0a09bcccdb4a3e414661e4f4f3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-26 05:34:35.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://logs.openstack.org/80/332480/5/check/gate-python-cinderclient-docs-ubuntu-xenial/ba42308/ : SUCCESS in 2m 38s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/80/332480/5/check/gate-python-cinderclient-pep8-ubuntu-xenial/44da762/ : SUCCESS in 2m 37s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/80/332480/5/check/gate-python-cinderclient-python27-ubuntu-xenial/2a7760e/ : SUCCESS in 5m 16s\n- gate-python-cinderclient-python34 http://logs.openstack.org/80/332480/5/check/gate-python-cinderclient-python34/3257865/ : SUCCESS in 3m 44s\n- gate-python-cinderclient-python35-nv http://logs.openstack.org/80/332480/5/check/gate-python-cinderclient-python35-nv/cb78748/ : SUCCESS in 5m 53s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/80/332480/5/check/gate-tempest-dsvm-neutron-src-python-cinderclient/43c5348/ : SUCCESS in 1h 15m 23s\n- gate-python-cinderclient-pypy http://logs.openstack.org/80/332480/5/check/gate-python-cinderclient-pypy/e8281fa/ : SUCCESS in 6m 03s (non-voting)\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/80/332480/5/check/gate-cinderclient-dsvm-functional/79ff338/ : SUCCESS in 32m 48s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/80/332480/5/check/gate-cinderclient-dsvm-functional-identity-v3-only-nv/f9c2c09/ : FAILURE in 27m 44s (non-voting)\n- gate-python-cinderclient-pylint http://logs.openstack.org/80/332480/5/check/gate-python-cinderclient-pylint/e5c7607/ : SUCCESS in 4m 12s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"19a7fc0cdae0e2620dc3f9c798763e43af0dc181","author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"date":"2016-08-01 23:16:57.000000000","message":"Patch Set 5: Code-Review+1\n\nI suppose this is probably the best solution now without making major changes to the metadata object and things, lgtm.","accounts_in_message":[],"_revision_number":5},{"id":"220e92e68edfb28e9478451ecb69d372b52cbcf8","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-08-23 04:38:17.000000000","message":"Patch Set 6: Patch Set 5 was rebased","accounts_in_message":[],"_revision_number":5},{"id":"dbee4603cf6e4ea67039eb99cf4864c39788f6f4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-23 05:55:27.000000000","message":"Patch Set 6: Verified-1\n\nBuild failed (check 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/80/332480/6/check/gate-python-cinderclient-docs-ubuntu-xenial/c70f94b//doc/build/html/ : SUCCESS in 3m 08s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/80/332480/6/check/gate-python-cinderclient-pep8-ubuntu-xenial/3ac6d96/ : SUCCESS in 3m 17s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/80/332480/6/check/gate-python-cinderclient-python27-ubuntu-xenial/97f259d/ : SUCCESS in 3m 24s\n- gate-python-cinderclient-python34 http://logs.openstack.org/80/332480/6/check/gate-python-cinderclient-python34/ca5e6df/ : SUCCESS in 4m 44s\n- gate-python-cinderclient-python35-nv http://logs.openstack.org/80/332480/6/check/gate-python-cinderclient-python35-nv/fad37c2/ : SUCCESS in 4m 13s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/80/332480/6/check/gate-tempest-dsvm-neutron-src-python-cinderclient/46853cd/ : FAILURE in 1h 12m 55s\n- gate-python-cinderclient-pypy http://logs.openstack.org/80/332480/6/check/gate-python-cinderclient-pypy/9968059/ : SUCCESS in 3m 48s (non-voting)\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/80/332480/6/check/gate-cinderclient-dsvm-functional/e699f6b/ : FAILURE in 25m 46s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/80/332480/6/check/gate-cinderclient-dsvm-functional-identity-v3-only-nv/ef63dde/ : SUCCESS in 37m 38s (non-voting)\n- gate-python-cinderclient-pylint http://logs.openstack.org/80/332480/6/check/gate-python-cinderclient-pylint/bf8f454/ : SUCCESS in 2m 42s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"fa7590a56afb5ea6effbc712b6ac5a735fb72760","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-08-27 01:04:26.000000000","message":"Patch Set 7: Patch Set 6 was rebased","accounts_in_message":[],"_revision_number":6},{"id":"a2a97b693979e815954cb8d017c0ddfaac363550","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-27 02:13:22.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/80/332480/7/check/gate-python-cinderclient-docs-ubuntu-xenial/7d2646b//doc/build/html/ : SUCCESS in 2m 20s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/80/332480/7/check/gate-python-cinderclient-pep8-ubuntu-xenial/658ae4c/ : SUCCESS in 2m 23s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/80/332480/7/check/gate-python-cinderclient-python27-ubuntu-xenial/6f54416/ : SUCCESS in 2m 40s\n- gate-python-cinderclient-python34 http://logs.openstack.org/80/332480/7/check/gate-python-cinderclient-python34/f2c5399/ : SUCCESS in 2m 43s\n- gate-python-cinderclient-python35-nv http://logs.openstack.org/80/332480/7/check/gate-python-cinderclient-python35-nv/6d63941/ : SUCCESS in 2m 45s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/80/332480/7/check/gate-tempest-dsvm-neutron-src-python-cinderclient/37dd3fa/ : SUCCESS in 1h 08m 38s\n- gate-python-cinderclient-pypy http://logs.openstack.org/80/332480/7/check/gate-python-cinderclient-pypy/6890801/ : SUCCESS in 2m 41s (non-voting)\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/80/332480/7/check/gate-cinderclient-dsvm-functional/7358ad0/ : SUCCESS in 34m 46s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/80/332480/7/check/gate-cinderclient-dsvm-functional-identity-v3-only-nv/e3233a8/ : SUCCESS in 31m 01s (non-voting)\n- gate-python-cinderclient-pylint http://logs.openstack.org/80/332480/7/check/gate-python-cinderclient-pylint/e04b8f1/ : SUCCESS in 2m 24s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"642d839436507af71b0788eaebf5dc1f3ce328e4","author":{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},"date":"2016-08-29 12:46:10.000000000","message":"Patch Set 7:\n\n(2 comments)","accounts_in_message":[],"_revision_number":7},{"id":"6cdf12aef246a7d6bde5b6b0db0dca0eb90864dc","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-09-12 04:57:49.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"c3814b882b0a47c9569beb43ebb67af1dbd01295","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-09-12 05:03:48.000000000","message":"Patch Set 7:\n\n(1 comment)","accounts_in_message":[],"_revision_number":7},{"id":"037dd44ca04a39e975efc8eb2dc040bfe72c65e6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-12 05:58:20.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/80/332480/8/check/gate-python-cinderclient-docs-ubuntu-xenial/32ffbc8//doc/build/html/ : SUCCESS in 3m 00s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/80/332480/8/check/gate-python-cinderclient-pep8-ubuntu-xenial/7f0fb9d/ : SUCCESS in 1m 57s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/80/332480/8/check/gate-python-cinderclient-python27-ubuntu-xenial/dcc9883/ : SUCCESS in 5m 39s\n- gate-python-cinderclient-python34 http://logs.openstack.org/80/332480/8/check/gate-python-cinderclient-python34/a20aec9/ : SUCCESS in 4m 08s\n- gate-python-cinderclient-python35-nv http://logs.openstack.org/80/332480/8/check/gate-python-cinderclient-python35-nv/810ee69/ : SUCCESS in 2m 41s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/80/332480/8/check/gate-tempest-dsvm-neutron-src-python-cinderclient/17eec17/ : SUCCESS in 1h 00m 19s\n- gate-python-cinderclient-pypy http://logs.openstack.org/80/332480/8/check/gate-python-cinderclient-pypy/d16f2dd/ : SUCCESS in 3m 05s (non-voting)\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/80/332480/8/check/gate-cinderclient-dsvm-functional/f3b9084/ : SUCCESS in 13m 12s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/80/332480/8/check/gate-cinderclient-dsvm-functional-identity-v3-only-nv/3ba3ecb/ : FAILURE in 19m 17s (non-voting)\n- gate-python-cinderclient-pylint http://logs.openstack.org/80/332480/8/check/gate-python-cinderclient-pylint/00c036e/ : SUCCESS in 2m 57s (non-voting)","accounts_in_message":[],"_revision_number":8},{"id":"cbea63ab0a526c8a2e03b4a3374da0f7b4bf84b3","author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"date":"2016-09-27 21:38:38.000000000","message":"Patch Set 8: Code-Review+1\n\nThis LGTM now","accounts_in_message":[],"_revision_number":8},{"id":"2f47f05ee27f5f27085484246bf3b92ad9e3ef7d","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2016-09-28 02:33:05.000000000","message":"Patch Set 8: Code-Review+2\n\nLooks good. I just wonder if we should pop it out of the metadata since it\u0027s being moved up a level.","accounts_in_message":[],"_revision_number":8},{"id":"ad5f2cfae42109f8d91cfb6502c68cb56f1b1c17","author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"date":"2016-10-07 05:14:13.000000000","message":"Patch Set 8:\n\nSean, It does seem like a good idea to me. Do we need to pop it out of the api response...before sending it to cinder client? this means a version bump.\n\nI think the easiest is now to remove it from response in the client i.e outside metadata as you said. Let me know what you think","accounts_in_message":[],"_revision_number":8},{"id":"47783130bf5b23bd02790940c583a288df0fc710","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2016-10-07 10:36:01.000000000","message":"Patch Set 8: Code-Review+1\n\nI feel that we should always present the same number of \"separate fields\" for a given user, so they should not depend on the volume we are showing.\n\nSo readonly should always be there:\n\n info[\u0027readonly\u0027] \u003d info[\u0027metadata\u0027].get(\u0027readonly\u0027, \u0027\u0027)","accounts_in_message":[],"_revision_number":8},{"id":"b756e53aa5d75e82afa558adbc2a61c2a2022e86","author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"date":"2016-10-18 02:58:01.000000000","message":"Patch Set 8: Code-Review+1\n\nLGTM","accounts_in_message":[],"_revision_number":8},{"id":"3ec9d3072ee8d6482f3fee227b3114bf467c81aa","author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"date":"2016-10-18 16:03:32.000000000","message":"Patch Set 8: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":8},{"id":"3699c856589677e07a2563979196bf427d276765","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-10-18 16:03:43.000000000","message":"Patch Set 8: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":8},{"id":"af9eae3e8545098bd803feab1dd99fb41f4f1ffa","date":"2016-10-18 17:03:16.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":8},{"id":"6ff4c401ba21562bc2f716a1df091429250e03e6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-10-18 17:03:16.000000000","message":"Patch Set 8: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/80/332480/8/gate/gate-python-cinderclient-docs-ubuntu-xenial/2efd382//doc/build/html/ : SUCCESS in 2m 52s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/80/332480/8/gate/gate-python-cinderclient-pep8-ubuntu-xenial/279de7c/ : SUCCESS in 2m 24s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/80/332480/8/gate/gate-python-cinderclient-python27-ubuntu-xenial/77595bd/ : SUCCESS in 2m 33s\n- gate-python-cinderclient-python34 http://logs.openstack.org/80/332480/8/gate/gate-python-cinderclient-python34/f62ffc3/ : SUCCESS in 5m 54s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/80/332480/8/gate/gate-tempest-dsvm-neutron-src-python-cinderclient/fbc5e9d/ : SUCCESS in 41m 36s","accounts_in_message":[],"_revision_number":8}],"current_revision_number":8,"current_revision":"23f7d19739c66e28efd425df6bd4314f9a0cc961","revisions":{"ab6f94d533ba09ad4b73e96043508ea29f443f17":{"kind":"REWORK","_number":1,"created":"2016-06-21 21:20:22.000000000","uploader":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"ref":"refs/changes/80/332480/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/80/332480/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/1"}}},"commit":{"parents":[{"commit":"835747d40fb64bbf78bb331cda0ecf5cdd16de84","subject":"Merge \"Fix argument order for assertEqual to (expected, observed)\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/835747d40fb64bbf78bb331cda0ecf5cdd16de84"}]}],"author":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-06-21 20:40:18.000000000","tz":0},"committer":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-06-21 21:19:52.000000000","tz":0},"subject":"[WIP] Showing the metadata readonly value as a separate field","message":"[WIP] Showing the metadata readonly value as a separate field\n\nNow, cinder show \u003cvolume_id\u003e, shows readonly as a separate field. This\nhelps the user to not have to scan all through the volume\u0027s metadata\nfield\n\nChange-Id: Ib427f7e76e8e28fee97b75413c3032fbee75af86\nCloses-bug: #1307098\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/ab6f94d533ba09ad4b73e96043508ea29f443f17"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/ab6f94d533ba09ad4b73e96043508ea29f443f17"}]},"branch":"refs/heads/master"},"4a50eb007a218a12b19015e5ee4d55294758ad20":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2016-06-29 18:43:22.000000000","uploader":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"ref":"refs/changes/80/332480/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/80/332480/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/2"}}},"commit":{"parents":[{"commit":"835747d40fb64bbf78bb331cda0ecf5cdd16de84","subject":"Merge \"Fix argument order for assertEqual to (expected, observed)\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/835747d40fb64bbf78bb331cda0ecf5cdd16de84"}]}],"author":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-06-21 20:40:18.000000000","tz":0},"committer":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-06-29 18:43:00.000000000","tz":0},"subject":"Showing the metadata readonly value as a separate field","message":"Showing the metadata readonly value as a separate field\n\nNow, cinder show \u003cvolume_id\u003e, shows readonly as a separate field. This\nhelps the user to not have to scan all through the volume\u0027s metadata\nfield\n\nChange-Id: Ib427f7e76e8e28fee97b75413c3032fbee75af86\nCloses-bug: #1307098\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/4a50eb007a218a12b19015e5ee4d55294758ad20"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/4a50eb007a218a12b19015e5ee4d55294758ad20"}]},"branch":"refs/heads/master"},"d733816a5d27d337541ffd15dbb8a0b893338a21":{"kind":"REWORK","_number":3,"created":"2016-07-01 00:29:48.000000000","uploader":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"ref":"refs/changes/80/332480/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/80/332480/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/3"}}},"commit":{"parents":[{"commit":"835747d40fb64bbf78bb331cda0ecf5cdd16de84","subject":"Merge \"Fix argument order for assertEqual to (expected, observed)\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/835747d40fb64bbf78bb331cda0ecf5cdd16de84"}]}],"author":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-06-21 20:40:18.000000000","tz":0},"committer":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-07-01 00:29:30.000000000","tz":0},"subject":"Showing the metadata readonly value as a separate field","message":"Showing the metadata readonly value as a separate field\n\nNow, cinder show \u003cvolume_id\u003e, shows readonly as a separate field. This\nhelps the user to not have to scan all through the volume\u0027s metadata\nfield\n\nChange-Id: Ib427f7e76e8e28fee97b75413c3032fbee75af86\nCloses-bug: #1307098\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/d733816a5d27d337541ffd15dbb8a0b893338a21"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/d733816a5d27d337541ffd15dbb8a0b893338a21"}]},"branch":"refs/heads/master"},"551e4c927f150f4817687714c15d76ec572c3ccf":{"kind":"REWORK","_number":4,"created":"2016-07-01 00:33:54.000000000","uploader":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"ref":"refs/changes/80/332480/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/80/332480/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/4"}}},"commit":{"parents":[{"commit":"835747d40fb64bbf78bb331cda0ecf5cdd16de84","subject":"Merge \"Fix argument order for assertEqual to (expected, observed)\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/835747d40fb64bbf78bb331cda0ecf5cdd16de84"}]}],"author":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-06-21 20:40:18.000000000","tz":0},"committer":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-07-01 00:33:43.000000000","tz":0},"subject":"Showing the metadata readonly value as a separate field","message":"Showing the metadata readonly value as a separate field\n\nNow, cinder show \u003cvolume_id\u003e, shows readonly as a separate field. This\nhelps the user to not have to scan all through the volume\u0027s metadata\nfield\n\nChange-Id: Ib427f7e76e8e28fee97b75413c3032fbee75af86\nCloses-bug: #1307098\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/551e4c927f150f4817687714c15d76ec572c3ccf"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/551e4c927f150f4817687714c15d76ec572c3ccf"}]},"branch":"refs/heads/master"},"ed006019e0d5b4425661abc07ba4bf2e99e18ebb":{"kind":"REWORK","_number":5,"created":"2016-07-26 02:36:11.000000000","uploader":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"ref":"refs/changes/80/332480/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/80/332480/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/5"}}},"commit":{"parents":[{"commit":"b61ec1e6a76907942298361f72334fc02a0adb23","subject":"Merge \"Fix string interpolation to delayed to be handled by the logging code\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b61ec1e6a76907942298361f72334fc02a0adb23"}]}],"author":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-06-21 20:40:18.000000000","tz":0},"committer":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-07-26 02:36:11.000000000","tz":0},"subject":"Showing the metadata readonly value as a separate field","message":"Showing the metadata readonly value as a separate field\n\nNow, cinder show \u003cvolume_id\u003e, shows readonly as a separate field. This\nhelps the user to not have to scan all through the volume\u0027s metadata\nfield\n\nChange-Id: Ib427f7e76e8e28fee97b75413c3032fbee75af86\nCloses-bug: #1307098\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/ed006019e0d5b4425661abc07ba4bf2e99e18ebb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/ed006019e0d5b4425661abc07ba4bf2e99e18ebb"}]},"branch":"refs/heads/master"},"8f07b2720946101f42e8250fe6f5707fa708d4b5":{"kind":"TRIVIAL_REBASE","_number":6,"created":"2016-08-23 04:38:17.000000000","uploader":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"ref":"refs/changes/80/332480/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/80/332480/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/6"}}},"commit":{"parents":[{"commit":"f802d1ae1add84a3cd590de596d7835c1fce79b3","subject":"Merge \"deprecate command `cinder endpoints`\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/f802d1ae1add84a3cd590de596d7835c1fce79b3"}]}],"author":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-06-21 20:40:18.000000000","tz":0},"committer":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-08-23 04:38:17.000000000","tz":0},"subject":"Showing the metadata readonly value as a separate field","message":"Showing the metadata readonly value as a separate field\n\nNow, cinder show \u003cvolume_id\u003e, shows readonly as a separate field. This\nhelps the user to not have to scan all through the volume\u0027s metadata\nfield\n\nChange-Id: Ib427f7e76e8e28fee97b75413c3032fbee75af86\nCloses-bug: #1307098\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8f07b2720946101f42e8250fe6f5707fa708d4b5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8f07b2720946101f42e8250fe6f5707fa708d4b5"}]},"branch":"refs/heads/master"},"66569fc70d842c09ad6e4f06a1199d721d9b31a7":{"kind":"TRIVIAL_REBASE","_number":7,"created":"2016-08-27 01:04:26.000000000","uploader":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"ref":"refs/changes/80/332480/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/80/332480/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/7"}}},"commit":{"parents":[{"commit":"e8d272a167b38656d5d7a6cedbb898cb32574794","subject":"Merge \"Changed backup-restore to accept backup name\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/e8d272a167b38656d5d7a6cedbb898cb32574794"}]}],"author":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-06-21 20:40:18.000000000","tz":0},"committer":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-08-27 01:04:26.000000000","tz":0},"subject":"Showing the metadata readonly value as a separate field","message":"Showing the metadata readonly value as a separate field\n\nNow, cinder show \u003cvolume_id\u003e, shows readonly as a separate field. This\nhelps the user to not have to scan all through the volume\u0027s metadata\nfield\n\nChange-Id: Ib427f7e76e8e28fee97b75413c3032fbee75af86\nCloses-bug: #1307098\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/66569fc70d842c09ad6e4f06a1199d721d9b31a7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/66569fc70d842c09ad6e4f06a1199d721d9b31a7"}]},"branch":"refs/heads/master"},"23f7d19739c66e28efd425df6bd4314f9a0cc961":{"kind":"REWORK","_number":8,"created":"2016-09-12 04:57:49.000000000","uploader":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"ref":"refs/changes/80/332480/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/80/332480/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/80/332480/8"}}},"commit":{"parents":[{"commit":"e8d272a167b38656d5d7a6cedbb898cb32574794","subject":"Merge \"Changed backup-restore to accept backup name\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/e8d272a167b38656d5d7a6cedbb898cb32574794"}]}],"author":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-06-21 20:40:18.000000000","tz":0},"committer":{"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","date":"2016-09-12 04:57:49.000000000","tz":0},"subject":"Showing the metadata readonly value as a separate field","message":"Showing the metadata readonly value as a separate field\n\nNow, cinder show \u003cvolume_id\u003e, shows readonly as a separate field. This\nhelps the user to not have to scan all through the volume\u0027s metadata\nfield\n\nChange-Id: Ib427f7e76e8e28fee97b75413c3032fbee75af86\nCloses-bug: #1307098\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/23f7d19739c66e28efd425df6bd4314f9a0cc961"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/23f7d19739c66e28efd425df6bd4314f9a0cc961"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}