)]}'
{"id":"openstack%2Fpython-novaclient~190111","triplet_id":"openstack%2Fpython-novaclient~master~I848de9a72757a77e9fe5dc29396e3a19d7ceddd0","project":"openstack/python-novaclient","branch":"master","topic":"bp/is","hashtags":[],"change_id":"I848de9a72757a77e9fe5dc29396e3a19d7ceddd0","subject":"Enhance CLI commands for more usability","status":"ABANDONED","created":"2015-06-10 11:00:03.000000000","updated":"2016-06-10 15:44:26.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d852a75b713f7b1bb74d34f2e4eeb4103b6f001e","_number":190111,"virtual_id_number":190111,"owner":{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},{"value":-1,"date":"2016-03-07 11:56:23.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":17108,"name":"Shengming Yang","email":"rory.yang@elits.com","username":"YangShengMing"},{"date":"2015-06-10 16:11:36.000000000","_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"date":"2015-10-08 21:44:37.000000000","_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":13419,"name":"Sudheer Kalla","email":"sudheer.kalla@tcs.com","username":"SudheerKall"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"date":"2015-06-10 16:35:01.000000000","_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"}],"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":{"rejected":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"recommended":{"_account_id":13419,"name":"Sudheer Kalla","email":"sudheer.kalla@tcs.com","username":"SudheerKall"},"disliked":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":-2,"date":"2015-07-27 15:36:52.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2015-10-29 09:59:11.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":17108,"name":"Shengming Yang","email":"rory.yang@elits.com","username":"YangShengMing"},{"value":2,"date":"2015-10-23 16:31:02.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":1,"date":"2015-06-18 12:32:33.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":13419,"name":"Sudheer Kalla","email":"sudheer.kalla@tcs.com","username":"SudheerKall"},{"value":-1,"date":"2015-06-17 13:37:38.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":-1,"date":"2015-10-29 08:48:07.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"date":"2016-02-02 23:03:00.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},{"date":"2015-06-18 12:30:25.000000000","_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":17108,"name":"Shengming Yang","email":"rory.yang@elits.com","username":"YangShengMing"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":13419,"name":"Sudheer Kalla","email":"sudheer.kalla@tcs.com","username":"SudheerKall"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"date":"2015-06-24 12:10:55.000000000","_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":17108,"name":"Shengming Yang","email":"rory.yang@elits.com","username":"YangShengMing"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":13419,"name":"Sudheer Kalla","email":"sudheer.kalla@tcs.com","username":"SudheerKall"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},{"_account_id":13419,"name":"Sudheer Kalla","email":"sudheer.kalla@tcs.com","username":"SudheerKall"},{"_account_id":17108,"name":"Shengming Yang","email":"rory.yang@elits.com","username":"YangShengMing"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-06-10 16:11:36.000000000","updated_by":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"reviewer":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"state":"REVIEWER"},{"updated":"2015-06-17 13:37:38.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2015-06-18 12:30:25.000000000","updated_by":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"reviewer":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"state":"REVIEWER"},{"updated":"2015-06-18 12:32:33.000000000","updated_by":{"_account_id":13419,"name":"Sudheer Kalla","email":"sudheer.kalla@tcs.com","username":"SudheerKall"},"reviewer":{"_account_id":13419,"name":"Sudheer Kalla","email":"sudheer.kalla@tcs.com","username":"SudheerKall"},"state":"REVIEWER"},{"updated":"2015-06-24 12:10:55.000000000","updated_by":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"reviewer":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"state":"REVIEWER"},{"updated":"2015-07-27 15:36:52.000000000","updated_by":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"reviewer":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"state":"REVIEWER"},{"updated":"2015-10-08 21:44:37.000000000","updated_by":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"reviewer":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"state":"REVIEWER"},{"updated":"2015-10-29 09:59:11.000000000","updated_by":{"_account_id":17108,"name":"Shengming Yang","email":"rory.yang@elits.com","username":"YangShengMing"},"reviewer":{"_account_id":17108,"name":"Shengming Yang","email":"rory.yang@elits.com","username":"YangShengMing"},"state":"REVIEWER"},{"updated":"2016-02-02 23:03:00.000000000","updated_by":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"reviewer":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"state":"REVIEWER"},{"updated":"2016-03-07 11:56:23.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"cc530bc4e0b5a0dfee870538e272282418c81d9f","author":{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},"date":"2015-06-10 11:00:03.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"2a2943f81ccef16756d4d779ebcabdba6d54ca2e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-06-10 12:09:40.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-novaclient-python26 http://logs.openstack.org/11/190111/1/check/gate-python-novaclient-python26/3ade144/ : SUCCESS in 1m 53s\n- gate-python-novaclient-pep8 http://logs.openstack.org/11/190111/1/check/gate-python-novaclient-pep8/256daca/ : SUCCESS in 1m 10s\n- gate-python-novaclient-docs http://docs-draft.openstack.org/11/190111/1/check/gate-python-novaclient-docs/dc56541//doc/build/html/ : SUCCESS in 1m 19s\n- gate-python-novaclient-python27 http://logs.openstack.org/11/190111/1/check/gate-python-novaclient-python27/71291fc/ : SUCCESS in 1m 51s\n- gate-python-novaclient-python34 http://logs.openstack.org/11/190111/1/check/gate-python-novaclient-python34/60846ad/ : SUCCESS in 2m 19s\n- gate-python-novaclient-pypy http://logs.openstack.org/11/190111/1/check/gate-python-novaclient-pypy/14fc528/ : SUCCESS in 2m 38s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-novaclient http://logs.openstack.org/11/190111/1/check/gate-tempest-dsvm-neutron-src-python-novaclient/68a1c1d/ : SUCCESS in 59m 37s\n- check-novaclient-dsvm-functional http://logs.openstack.org/11/190111/1/check/check-novaclient-dsvm-functional/30212cf/ : SUCCESS in 19m 33s","accounts_in_message":[],"_revision_number":1},{"id":"45058db92ca97b9e4c64941e290412164f3c4fb6","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2015-06-10 16:11:36.000000000","message":"Patch Set 1:\n\n(1 comment)\n\nCode quality wise, this looks good, and it passes tests.  I can\u0027t find the blueprint you reference, though…","accounts_in_message":[],"_revision_number":1},{"id":"6953b218e62a855e82105515a29f9b9d92bf7c36","author":{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},"date":"2015-06-10 16:35:01.000000000","message":"Patch Set 1:\n\nhttps://blueprints.launchpad.net/python-novaclient/+spec/enhance-commands-for-usability","accounts_in_message":[],"_revision_number":1},{"id":"812c4f6e9744ed17658590d4ef052b41a74a02f4","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2015-06-10 17:05:03.000000000","message":"Patch Set 1: Code-Review+2\n\nInteresting that it didn\u0027t show up when I searched earlier, especially given that it shows up now.  And yes, I see that it was created 12 hours ago.  I think this must just be Launchpad sucking…","accounts_in_message":[],"_revision_number":1},{"id":"b7f743969132e3c41421a25ecb0e231bee19a3f8","author":{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},"date":"2015-06-11 03:45:12.000000000","message":"Patch Set 1:\n\nHi Kevin,\n\nI got to know why it has not been visible to you earlier, now it is visible to you. \n\nIn blueprint information of a blueprint in launchpad, has a section called Implementation, where you can give the status of implementation. In that section, earlier I have selected Implemented as status, which changes the overall status to complete. That\u0027s the reason, you are to able to see. \nWhen you have said you can\u0027t find the blueprint reference, at that time, it was also not visible to me. \n\nLater I have changed the implementation status to Code review.","accounts_in_message":[],"_revision_number":1},{"id":"e1b1753e24dc46acfd7cdae1914f5449114bdcca","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2015-06-11 17:20:20.000000000","message":"Patch Set 1:\n\nAh, interesting.  Yeah, that would probably explain why I couldn\u0027t find it :)","accounts_in_message":[],"_revision_number":1},{"id":"f017f608c3be22f2bfc6155ece04c09ff07e7ec8","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2015-06-17 13:37:38.000000000","message":"Patch Set 1: Code-Review-1\n\nI\u0027m actually a little concerned with adding additional output by default. Accepting more than one server seems fine, but I feel like we need to be careful sticking output out there for it. Maybe only do it during --verbose or something. What is the expected error return by the cli under these circumstances?\n\nI also really think we need functional tests on this.","accounts_in_message":[],"_revision_number":1},{"id":"967c33ff4b6f2b0ab1c23b4045361133d859c205","author":{"_account_id":13419,"name":"Sudheer Kalla","email":"sudheer.kalla@tcs.com","username":"SudheerKall"},"date":"2015-06-18 12:32:33.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"b15d4d97e8f14598b2cf2e1c5344b7e1e54ed186","author":{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},"date":"2015-06-18 12:40:19.000000000","message":"Patch Set 1:\n\nIt is not the additional output. In contrast,  when you delete an instance you will get the same output message. This patch also works fine when you get the erroneous response. For example, in case of error state it will look like this. \n\n$ nova suspend test\nCannot \u0027suspend\u0027 instance ab7fb30e-7fb0-4591-a7be-594f369e8f78 while it is in vm_state suspended (HTTP 409) (Request-ID: req-ad25e588-7cb4-4b74-800e-36b55ae0a1d9)\nERROR (CommandError): Unable to suspend the specified server(s).\n\nAnd this has no impacts on functional changes, test cases will just become a redundant. This patch provides a for loop for existing commands.  As suspend, pause have already test cases available. \n\nIf I miss something, please tell me.","accounts_in_message":[],"_revision_number":1},{"id":"ca3672a56b87e2b5fdfa874212582ca12bbb9738","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2015-07-24 21:30:41.000000000","message":"Patch Set 1:\n\nSean: *poke* could you respond to Ritesh\u0027s rebuttal of your -1?","accounts_in_message":[],"_revision_number":1},{"id":"595934e096941329a02bb0968a852b2c98e0ee2b","author":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"date":"2015-07-27 15:36:52.000000000","message":"Patch Set 1: Code-Review-2\n\nI hate to be that guy, but the blueprint is marked as pending approval, not approved. You\u0027ll want to reach out to Bryant and possibly Garbutt about getting it approved.","accounts_in_message":[],"_revision_number":1},{"id":"caaee78ef893be2ac30c0d74f1203785b538cb85","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2015-09-11 18:04:29.000000000","message":"Patch Set 1:\n\nThis change hasn\u0027t been updated since the end of July; are you still working on it?  If not, you may want to consider abandoning it…","accounts_in_message":[],"_revision_number":1},{"id":"07ccd4f00ed3edd0f855484e139cd185f414d7e4","author":{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},"date":"2015-09-14 04:32:47.000000000","message":"Patch Set 1:\n\nI am waiting for approval, if some one approve or give comments  that would be good.","accounts_in_message":[],"_revision_number":1},{"id":"e0dcc3301afa4e40a28b1590bb179f0423b3c04e","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2015-09-14 15:57:51.000000000","message":"Patch Set 1:\n\nApproval for the blueprint?","accounts_in_message":[],"_revision_number":1},{"id":"22909c280798bd1901a0c91f4cefb9c77837a6b8","author":{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},"date":"2015-09-15 07:51:48.000000000","message":"Patch Set 1:\n\nYes, kevin approval for the blueprint and this patch. \n\n\nhttps://blueprints.launchpad.net/python-novaclient/+spec/enhance-commands-for-usability","accounts_in_message":[],"_revision_number":1},{"id":"e8a5843355fe656fe2806e74c67c3d64ea729eea","author":{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},"date":"2015-09-22 04:43:37.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"66c224af42c0b781e957628d88a57a1693600203","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2015-10-02 15:56:59.000000000","message":"Patch Set 1: -Code-Review\n\nWhat\u0027s the current status on this change?","accounts_in_message":[],"_revision_number":1},{"id":"a8b27ea235bd47bb5b83270036cdae7468bfc4ad","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2015-10-02 23:59:10.000000000","message":"Patch Set 1:\n\nIt seems like this should be okay based on the fact that we\u0027re already doing it for \u0027nova delete\u0027 that uses the utils.do_action_on_many.\n\nTo be safe, I\u0027d probably want to bring it up at the nova meeting and/or ML to make sure there isn\u0027t some reason we shouldn\u0027t do this, and to also get approval for the blueprint for bookkeeping.","accounts_in_message":[],"_revision_number":1},{"id":"ee9aafaa649df2a75f1d75b4dde81dc896a4b6f7","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2015-10-03 00:25:34.000000000","message":"Patch Set 1:\n\nFor completeness, I checked and we\u0027re also already doing it for: reboot, stop, and start.","accounts_in_message":[],"_revision_number":1},{"id":"f06c52d4c657a1ccd1867c5355616f9d84908c7f","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2015-10-08 21:44:37.000000000","message":"Patch Set 1:\n\nmikal will be approving the blueprint since we talked about it in today\u0027s nova meeting.","accounts_in_message":[],"_revision_number":1},{"id":"04cd682a9bcf7b231412f0ec902392d84b69693e","author":{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},"date":"2015-10-09 08:15:52.000000000","message":"Patch Set 1:\n\nThank you . Please let me know any further comments.","accounts_in_message":[],"_revision_number":1},{"id":"9e1998b87c7073fd05a323938500a6c494a0c846","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2015-10-23 16:31:02.000000000","message":"Patch Set 1: Code-Review+2\n\nSo, the BP has been approved, answering Matt Dietz\u0027s objection.  I don\u0027t think the output change is a particular problem, which is Sean\u0027s objection.  Thus, I think this is ready for +2.  Anyone else have any other objections?","accounts_in_message":[],"_revision_number":1},{"id":"7fe5ef63171cd8c799666696918478dbcda1a278","author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"date":"2015-10-29 08:48:07.000000000","message":"Patch Set 1: Code-Review-1\n\nI think the output change is possibly a thing. What do we think about hiding that behind a --verbose flag?","accounts_in_message":[],"_revision_number":1},{"id":"3e5ad3e3f60fb3f6cfa38170cfbab3f0d73eb661","author":{"_account_id":17108,"name":"Shengming Yang","email":"rory.yang@elits.com","username":"YangShengMing"},"date":"2015-10-29 09:59:11.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"e7443fd6c08da765dd7a2c5672ac0c3deb3a63dd","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2015-10-29 10:18:52.000000000","message":"Patch Set 1:\n\nI could see the desire to have to opt-in to the verbose output, however, there are already commands in novaclient that output similar messages without any flag for controlling it. So, at some point there wasn\u0027t concern about additional output back when those patches merged.\n\nIf we add a --verbose flag, I think we should add it to the old commands too.\n\nGiven how long delete, reboot, start, and stop have had the additional output without a flag, I didn\u0027t think this would be an issue.","accounts_in_message":[],"_revision_number":1},{"id":"1d542479016cfb5368b680462da72cce395de467","author":{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},"date":"2016-02-01 10:00:44.000000000","message":"Patch Set 1:\n\nCould you please confirm on this. \nPending since long time.","accounts_in_message":[],"_revision_number":1},{"id":"a001a4d322d0ca476c401bccdffa1a5a36580b2e","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2016-02-02 23:03:00.000000000","message":"Patch Set 1:\n\nHi Ritesh, I think there were enough concerns around adding more output to the commands that it\u0027s desired to put the output behind a --verbose flag.\n\nIf you would be willing to add a verbose option to do_action_on_many and the commands that call do_action_on_many and only print the output if --verbose was specified, I think we can accept this patch.","accounts_in_message":[],"_revision_number":1},{"id":"427b254bb7729bd54806fc46ff1ad57adc626068","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-03-07 11:56:23.000000000","message":"Patch Set 1: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"101fe26fa533c1833d5325faf88a4895cf7f8964","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2016-06-10 15:44:26.000000000","message":"Abandoned\n\nHas not been updated since February; no response to review comments.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"79963c676d36ea622d45d229e77f220e109ef099","revisions":{"79963c676d36ea622d45d229e77f220e109ef099":{"kind":"REWORK","_number":1,"created":"2015-06-10 11:00:03.000000000","uploader":{"_account_id":11503,"name":"Ritesh Paiboina","email":"rsritesh@rediff.com","username":"rsritesh"},"ref":"refs/changes/11/190111/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/11/190111/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/11/190111/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/11/190111/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/11/190111/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/11/190111/1"}}},"commit":{"parents":[{"commit":"73957fc5ffaf2ccb1c8aa43b23b8df4f9bd24b43","subject":"Merge \"Adds support to set admin password from the cli\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/73957fc5ffaf2ccb1c8aa43b23b8df4f9bd24b43"}]}],"author":{"name":"rsritesh","email":"rsritesh@rediff.com","date":"2015-06-10 11:08:12.000000000","tz":120},"committer":{"name":"rsritesh","email":"rsritesh@rediff.com","date":"2015-06-10 11:10:50.000000000","tz":120},"subject":"Enhance CLI commands for more usability","message":"Enhance CLI commands for more usability\n\nCurrent nova commands such as suspend, pause, resume,\npause take a single server output and does not show\nany acceptance of command. This blueprint is to allow\nacceptance of more than one server and allow to show\noutput message when the request from the command has been accepted.\n\nFor example,\n\nCurrent Scenario :\n\n1) No acceptance output\n\nritesh@Cloud58:~$ nova suspend test2\n\n2) Accepting more than one server\n\nritesh@Cloud58:~$ nova suspend test test2\n\nExpected Scenario :\n\n1) ritesh@Cloud58:~$ nova suspend test\n    Request to suspend server test has been accepted.\n\n2) ritesh@Cloud58:~$ nova suspend test test2\n    Request to suspend server test has been accepted.\n    Request to suspend server test2 has been accepted.\n\nFollowing commands will have impact\n\npause\nsuspend\nresume\nshelve\nshelve-offload\nunshelve\nunpause\n\nChange-Id: I848de9a72757a77e9fe5dc29396e3a19d7ceddd0\nImplements: blueprint enhance-commands-for-usability\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/79963c676d36ea622d45d229e77f220e109ef099"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/79963c676d36ea622d45d229e77f220e109ef099"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
