)]}'
{"id":"openstack%2Fpython-novaclient~295681","triplet_id":"openstack%2Fpython-novaclient~master~Ie98d340044a86049f6201b2980ad2c93518038dd","project":"openstack/python-novaclient","branch":"master","topic":"bug_1415319","hashtags":[],"change_id":"Ie98d340044a86049f6201b2980ad2c93518038dd","subject":"flavor postional argument in nova boot help","status":"ABANDONED","created":"2016-03-22 09:06:35.000000000","updated":"2016-12-09 21:04:38.000000000","total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"73dee9884073afe190f1f2c3964e581c4f774660","_number":295681,"virtual_id_number":295681,"owner":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"},{"value":-1,"date":"2016-03-28 10:26:38.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2016-03-30 16:29:53.000000000","_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"_account_id":10618,"name":"Pavel Kholkin","email":"p.v.holkin@mail.ru","username":"pkholkin"},{"date":"2016-03-30 05:19:51.000000000","_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},{"date":"2016-12-09 21:04:38.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":10618,"name":"Pavel Kholkin","email":"p.v.holkin@mail.ru","username":"pkholkin"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":-1,"date":"2016-09-07 09:32:17.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":10618,"name":"Pavel Kholkin","email":"p.v.holkin@mail.ru","username":"pkholkin"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":[{"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"_account_id":10618,"name":"Pavel Kholkin","email":"p.v.holkin@mail.ru","username":"pkholkin"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":1},"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"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":10618,"name":"Pavel Kholkin","email":"p.v.holkin@mail.ru","username":"pkholkin"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":10618,"name":"Pavel Kholkin","email":"p.v.holkin@mail.ru","username":"pkholkin"},{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},{"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-03-22 11:37:26.000000000","updated_by":{"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"},"reviewer":{"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"},"state":"REVIEWER"},{"updated":"2016-03-28 10:26:38.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2016-03-30 16:29:53.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":"2016-09-07 09:32:17.000000000","updated_by":{"_account_id":10618,"name":"Pavel Kholkin","email":"p.v.holkin@mail.ru","username":"pkholkin"},"reviewer":{"_account_id":10618,"name":"Pavel Kholkin","email":"p.v.holkin@mail.ru","username":"pkholkin"},"state":"REVIEWER"},{"updated":"2016-12-09 21:04: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"}],"messages":[{"id":"a124fb0f6432764c2b16ee1a29b1121282c698fd","author":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"date":"2016-03-22 09:06:35.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"2b0598dec7cedebea406f4376fd27ae09d2277a1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-03-22 10:43:57.000000000","message":"Patch Set 1: 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-novaclient-docs http://docs-draft.openstack.org/81/295681/1/check/gate-python-novaclient-docs/c87c820//doc/build/html/ : SUCCESS in 14m 45s\n- gate-python-novaclient-pep8 http://logs.openstack.org/81/295681/1/check/gate-python-novaclient-pep8/9d1a85f/ : SUCCESS in 2m 45s\n- gate-python-novaclient-python27 http://logs.openstack.org/81/295681/1/check/gate-python-novaclient-python27/aca01f7/ : FAILURE in 3m 08s\n- gate-python-novaclient-python34 http://logs.openstack.org/81/295681/1/check/gate-python-novaclient-python34/3fe946a/ : FAILURE in 4m 39s\n- gate-python-novaclient-pypy http://logs.openstack.org/81/295681/1/check/gate-python-novaclient-pypy/e212448/ : FAILURE in 4m 29s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-novaclient http://logs.openstack.org/81/295681/1/check/gate-tempest-dsvm-neutron-src-python-novaclient/4e0ba40/ : SUCCESS in 1h 28m 29s\n- gate-novaclient-dsvm-functional http://logs.openstack.org/81/295681/1/check/gate-novaclient-dsvm-functional/7a4c4a7/ : FAILURE in 45m 17s","accounts_in_message":[],"_revision_number":1},{"id":"5d63c3f9a7f9227a2b17e0a991b9e4a5c47258b2","author":{"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"},"date":"2016-03-22 11:37:26.000000000","message":"Patch Set 1: Code-Review+1\n\n(1 comment)\n\nyes it is true that flavor is not an optional arguments, but are you sure that two positional arguments can be added??\n\none comment inline with commit message","accounts_in_message":[],"_revision_number":1},{"id":"edb6765a65524c77f127f762caea7e621ccb4655","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2016-03-22 15:49:33.000000000","message":"Patch Set 1: Code-Review-1\n\nThis alters the UI and cannot be done this way because we need to maintain backwards compatibility.  This sort of change has been done before; what happened there was adding an _optional_ positional argument, deprecating the option, and then testing within the do_*() function to ensure that a flavor was specified.","accounts_in_message":[],"_revision_number":1},{"id":"7dbac74ba5e45be0a6a18d8c621f80a93601e888","author":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"date":"2016-03-23 03:39:30.000000000","message":"Patch Set 1:\n\n\u003e This alters the UI and cannot be done this way because we need to\n \u003e maintain backwards compatibility.  This sort of change has been\n \u003e done before; what happened there was adding an _optional_\n \u003e positional argument, deprecating the option, and then testing\n \u003e within the do_*() function to ensure that a flavor was specified.\n\nHi Kevin,\n\nIf you can elaborate a bit on what you mentioned in your comment as I couldn\u0027t understood it completely.\n\nHopefully I can work it that way and we can get this bug merged and rectified.\n\nThanks and Regards\nAbhishek Talwar","accounts_in_message":[],"_revision_number":1},{"id":"38ab32069f7dffc4dc93441dc1fc9425f6015fc7","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2016-03-23 16:33:18.000000000","message":"Patch Set 1:\n\nSo, what you should do is add a new positional argument with the same \u0027dest\u0027 as the existing option.  The new positional argument must be set to be optional, and you would put the documentation there.  On the original option, you use DeprecatedAction (see the rest of shell.py for examples) and set the help to argparse.SUPPRESS.  Finally, in the do_* method, you would add, at the top level, a test to verify that the value is set (since both positional and option are, well, optional), and if it isn\u0027t, raise an appropriate CommandError explaining that the value must be provided.\n\nDoes that help clarify what I mean?  See the do_server_group_create() method for an example…","accounts_in_message":[],"_revision_number":1},{"id":"0aa37b678bc014fef6817cf670c8dd3b91df6e63","author":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"date":"2016-03-28 06:43:38.000000000","message":"Patch Set 1:\n\n\u003e So, what you should do is add a new positional argument with the\n \u003e same \u0027dest\u0027 as the existing option.  The new positional argument\n \u003e must be set to be optional, and you would put the documentation\n \u003e there.  On the original option, you use DeprecatedAction (see the\n \u003e rest of shell.py for examples) and set the help to\n \u003e argparse.SUPPRESS.  Finally, in the do_* method, you would add, at\n \u003e the top level, a test to verify that the value is set (since both\n \u003e positional and option are, well, optional), and if it isn\u0027t, raise\n \u003e an appropriate CommandError explaining that the value must be\n \u003e provided.\n \u003e \n \u003e Does that help clarify what I mean?  See the do_server_group_create()\n \u003e method for an example…\n\nHi Kevin,\n\nThanks a lot for the help. I will have a look at it and will change the patch as required.\nWill submit the next patch as soon as possible.\n\nThanks and Regards\nAbhishek Talwar","accounts_in_message":[],"_revision_number":1},{"id":"55d18f93180c5bbd8058a993a938c0003954efbb","author":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"date":"2016-03-28 07:04:28.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"ba9f75df29cee1023a9dce446a5f86384d12d2e4","author":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"date":"2016-03-28 07:12:36.000000000","message":"Patch Set 2:\n\nHi Kevin,\n\nI have uploaded the updated code.\n\nThanks and Regards\nAbhishek Talwar","accounts_in_message":[],"_revision_number":2},{"id":"1f507c5c3327b0d35bc101c51089db61dcb243d6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-03-28 07:31:32.000000000","message":"Patch Set 2: 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-novaclient-docs http://docs-draft.openstack.org/81/295681/2/check/gate-python-novaclient-docs/1e57deb//doc/build/html/ : SUCCESS in 1m 37s\n- gate-python-novaclient-pep8 http://logs.openstack.org/81/295681/2/check/gate-python-novaclient-pep8/5b11b9d/ : SUCCESS in 1m 39s\n- gate-python-novaclient-python27 http://logs.openstack.org/81/295681/2/check/gate-python-novaclient-python27/abaa27a/ : FAILURE in 2m 15s\n- gate-python-novaclient-python34 http://logs.openstack.org/81/295681/2/check/gate-python-novaclient-python34/7696840/ : FAILURE in 5m 16s\n- gate-python-novaclient-pypy http://logs.openstack.org/81/295681/2/check/gate-python-novaclient-pypy/e7000c1/ : FAILURE in 3m 25s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-novaclient http://logs.openstack.org/81/295681/2/check/gate-tempest-dsvm-neutron-src-python-novaclient/61d15a0/ : FAILURE in 12m 30s\n- gate-novaclient-dsvm-functional http://logs.openstack.org/81/295681/2/check/gate-novaclient-dsvm-functional/096c1b6/ : FAILURE in 24m 41s","accounts_in_message":[],"_revision_number":2},{"id":"d69cf6b875edafa879a4724b43e3c6817e94513c","author":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"date":"2016-03-28 09:02:34.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"cdc6f82239dfbb758a8ba7ceed3decbee8294d20","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-03-28 10:26:38.000000000","message":"Patch Set 3: 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-novaclient-docs http://docs-draft.openstack.org/81/295681/3/check/gate-python-novaclient-docs/6710ef4//doc/build/html/ : SUCCESS in 4m 08s\n- gate-python-novaclient-pep8 http://logs.openstack.org/81/295681/3/check/gate-python-novaclient-pep8/b9ea899/ : SUCCESS in 3m 26s\n- gate-python-novaclient-python27 http://logs.openstack.org/81/295681/3/check/gate-python-novaclient-python27/2a33e08/ : FAILURE in 4m 25s\n- gate-python-novaclient-python34 http://logs.openstack.org/81/295681/3/check/gate-python-novaclient-python34/5e42c80/ : FAILURE in 3m 56s\n- gate-python-novaclient-pypy http://logs.openstack.org/81/295681/3/check/gate-python-novaclient-pypy/3fd9cf6/ : FAILURE in 4m 02s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-novaclient http://logs.openstack.org/81/295681/3/check/gate-tempest-dsvm-neutron-src-python-novaclient/bf808dc/ : SUCCESS in 1h 21m 01s\n- gate-novaclient-dsvm-functional http://logs.openstack.org/81/295681/3/check/gate-novaclient-dsvm-functional/f13658b/ : FAILURE in 56m 12s","accounts_in_message":[],"_revision_number":3},{"id":"e05ad0c6c05e1f1feb6fb3abe80d16a33ddf92ad","author":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"date":"2016-03-30 05:19:51.000000000","message":"Patch Set 3:\n\nHi Kevin,\n\nI have work around on the things but I think I have some mistakes in my code thats why there are a lot of tests failing. \nIf you can please help me on what should be the right approach.\n\nThanks and Regards\nAbhishek Talwar","accounts_in_message":[],"_revision_number":3},{"id":"b33525641c7e0bd0f3053688ba645400497af262","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2016-03-30 16:29:53.000000000","message":"Patch Set 3:\n\n(1 comment)\n\nA look at the tests indicates that you\u0027re trying to pass a list as the flavor ID being referenced:\n\n    2016-03-28 09:09:08.770 |     AssertionError: Called unknown API method: GET /flavors/[\u00271\u0027], expected fakes method name: get_flavors_[\u00271\u0027]\n\nI commented inline on the area of the code that causes this to be a list, and I\u0027ve also included a reference to the section of the \"argparse\" documentation describing the add_argument() method, which is called under the hood by the @utils.arg() decorator.  Hopefully that information is enough for you to figure out the problems on your own.\n\nI should mention that I will also want to see an update to the tests to test the new calling convention, without removing the tests on the existing calling convention.  (That is, I want to see tests that call the command with a positional parameter, as well as existing tests that call with the --flavor option.)  There should also be a test that confirms that leaving out both the positional parameter and the --flavor option will result in an appropriate error.","accounts_in_message":[],"_revision_number":3},{"id":"7e6c14c7c2cc382c7dd16fb64c2ace8eeea02cde","author":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"date":"2016-03-31 07:50:43.000000000","message":"Patch Set 3:\n\n(1 comment)\n\n\u003e (1 comment)\n \u003e \n \u003e A look at the tests indicates that you\u0027re trying to pass a list as\n \u003e the flavor ID being referenced:\n \u003e \n \u003e 2016-03-28 09:09:08.770 |     AssertionError: Called unknown API\n \u003e method: GET /flavors/[\u00271\u0027], expected fakes method name:\n \u003e get_flavors_[\u00271\u0027]\n \u003e \n \u003e I commented inline on the area of the code that causes this to be a\n \u003e list, and I\u0027ve also included a reference to the section of the\n \u003e \"argparse\" documentation describing the add_argument() method,\n \u003e which is called under the hood by the @utils.arg() decorator. \n \u003e Hopefully that information is enough for you to figure out the\n \u003e problems on your own.\n \u003e \n \u003e I should mention that I will also want to see an update to the\n \u003e tests to test the new calling convention, without removing the\n \u003e tests on the existing calling convention.  (That is, I want to see\n \u003e tests that call the command with a positional parameter, as well as\n \u003e existing tests that call with the --flavor option.)  There should\n \u003e also be a test that confirms that leaving out both the positional\n \u003e parameter and the --flavor option will result in an appropriate\n \u003e error.","accounts_in_message":[],"_revision_number":3},{"id":"f5a5350e9b5f38531b9e4eee884c3b44774bac42","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2016-03-31 18:48:56.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"32093f5d6d7d2c1c1b9fee18f660237f109ea2dc","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2016-06-10 16:11:26.000000000","message":"Patch Set 3:\n\nAre you still working on this change?  It hasn\u0027t been updated since March.","accounts_in_message":[],"_revision_number":3},{"id":"530fce415044835d08465f280bc29d8e3f4a6564","author":{"_account_id":10618,"name":"Pavel Kholkin","email":"p.v.holkin@mail.ru","username":"pkholkin"},"date":"2016-09-07 09:32:17.000000000","message":"Patch Set 3: Code-Review-1\n\nI think we should leave it as it is. With \u0027image\u0027 the situation is the same. I like that we can pass these arguments in any order, also this change will follow many changes in using \u0027boot\u0027 command which is our main operation. I understand the use case and that we got error, but still I think we should abandon this.","accounts_in_message":[],"_revision_number":3},{"id":"49d9c2883be1cf63f63f42cc818db58f6b5851c9","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2016-12-09 21:04:38.000000000","message":"Abandoned\n\nThis review is \u003e 6 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a \u0027recheck\u0027 comment to get fresh test results.","accounts_in_message":[],"_revision_number":3},{"id":"46d41dd1a6c587131791e1e82fb38343566ef27e","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2016-12-09 21:04:38.000000000","message":"Patch Set 3:\n\nThis review is \u003e 6 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a \u0027recheck\u0027 comment to get fresh test results.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"bbae08152e57a1b66e3460199378922ad585e0fc","revisions":{"c6f3c6d6356e379735bd6813ffb70905189964a9":{"kind":"REWORK","_number":1,"created":"2016-03-22 09:06:35.000000000","uploader":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"ref":"refs/changes/81/295681/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/81/295681/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/1"}}},"commit":{"parents":[{"commit":"abe6707d26859c08a031d53ca316d31aaff6bc3b","subject":"Merge \"Wrap interface_list by proper resource class\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/abe6707d26859c08a031d53ca316d31aaff6bc3b"}]}],"author":{"name":"abhishek.talwar","email":"abhishek.talwar@tcs.com","date":"2016-03-22 09:03:19.000000000","tz":330},"committer":{"name":"abhishek.talwar","email":"abhishek.talwar@tcs.com","date":"2016-03-22 09:06:28.000000000","tz":330},"subject":"flavor postional argument in nova boot help","message":"flavor postional argument in nova boot help\n\nThe help for boot specifies flavor as a\noptional parameter, where as running the\ncommand without a flavor results in an\nerror. So corrected the help to include\nflavor in the positional parameter list.\n\nChange-Id: Ie98d340044a86049f6201b2980ad2c93518038dd\nCloses-Bug: #1415319\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/c6f3c6d6356e379735bd6813ffb70905189964a9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/c6f3c6d6356e379735bd6813ffb70905189964a9"}]},"branch":"refs/heads/master"},"be725cd9bad7cca07086732f0bec5c4056cad3b3":{"kind":"REWORK","_number":2,"created":"2016-03-28 07:04:28.000000000","uploader":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"ref":"refs/changes/81/295681/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/81/295681/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/2"}}},"commit":{"parents":[{"commit":"abe6707d26859c08a031d53ca316d31aaff6bc3b","subject":"Merge \"Wrap interface_list by proper resource class\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/abe6707d26859c08a031d53ca316d31aaff6bc3b"}]}],"author":{"name":"abhishek.talwar","email":"abhishek.talwar@tcs.com","date":"2016-03-22 09:03:19.000000000","tz":330},"committer":{"name":"abhishek.talwar","email":"abhishek.talwar@tcs.com","date":"2016-03-28 07:00:45.000000000","tz":330},"subject":"flavor postional argument in nova boot help","message":"flavor postional argument in nova boot help\n\nThe help for boot specifies flavor as a\noptional parameter, where as running the\ncommand without a flavor results in an\nerror. So corrected the help to include\nflavor in the positional parameter list.\n\nChange-Id: Ie98d340044a86049f6201b2980ad2c93518038dd\nCloses-Bug: #1415319\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/be725cd9bad7cca07086732f0bec5c4056cad3b3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/be725cd9bad7cca07086732f0bec5c4056cad3b3"}]},"branch":"refs/heads/master"},"bbae08152e57a1b66e3460199378922ad585e0fc":{"kind":"REWORK","_number":3,"created":"2016-03-28 09:02:34.000000000","uploader":{"_account_id":12140,"name":"Abhishek Talwar","email":"abhishek.talwar@tcs.com","username":"abhishek.talwar"},"ref":"refs/changes/81/295681/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/81/295681/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/81/295681/3"}}},"commit":{"parents":[{"commit":"abe6707d26859c08a031d53ca316d31aaff6bc3b","subject":"Merge \"Wrap interface_list by proper resource class\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/abe6707d26859c08a031d53ca316d31aaff6bc3b"}]}],"author":{"name":"abhishek.talwar","email":"abhishek.talwar@tcs.com","date":"2016-03-22 09:03:19.000000000","tz":330},"committer":{"name":"abhishek.talwar","email":"abhishek.talwar@tcs.com","date":"2016-03-28 08:49:33.000000000","tz":330},"subject":"flavor postional argument in nova boot help","message":"flavor postional argument in nova boot help\n\nThe help for boot specifies flavor as a\noptional parameter, where as running the\ncommand without a flavor results in an\nerror. So corrected the help to include\nflavor in the positional parameter list.\n\nChange-Id: Ie98d340044a86049f6201b2980ad2c93518038dd\nCloses-Bug: #1415319\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/bbae08152e57a1b66e3460199378922ad585e0fc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/bbae08152e57a1b66e3460199378922ad585e0fc"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}