)]}'
{"id":"openstack%2Fpython-magnumclient~170334","triplet_id":"openstack%2Fpython-magnumclient~master~I97ad2cf1c3b3e7edc978920a54edd9bb3dc9d287","project":"openstack/python-magnumclient","branch":"master","topic":"bug/1439943","hashtags":[],"change_id":"I97ad2cf1c3b3e7edc978920a54edd9bb3dc9d287","subject":"Make json argument in container-create command positional","status":"ABANDONED","created":"2015-04-03 01:57:39.000000000","updated":"2015-04-03 05:48:33.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2241d9b1a7e383e9435cbb33de2e08c3bb397647","_number":170334,"virtual_id_number":170334,"owner":{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},{"value":1,"date":"2015-04-03 05:10:58.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"}],"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":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-1,"date":"2015-04-03 05:09:54.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"}],"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":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"}],"values":{"-2":"Do Not Backport","-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport","+2":"Should Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},{"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-04-03 02:13:57.000000000","updated_by":{"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},"reviewer":{"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},"state":"REVIEWER"},{"updated":"2015-04-03 05:09:54.000000000","updated_by":{"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},"reviewer":{"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},"state":"REVIEWER"},{"updated":"2015-04-03 05:10:58.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"38983043d3b165d82fc0e4bdb72bd15ff16b15ef","author":{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"},"date":"2015-04-03 01:57:39.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6e6e5e8295cca24b6a7c94ffa9b4d1ff0c67ff74","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-03 02:00:19.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-magnumclient-pep8 http://logs.openstack.org/34/170334/1/check/gate-python-magnumclient-pep8/267f488/ : SUCCESS in 43s\n- gate-python-magnumclient-docs http://docs-draft.openstack.org/34/170334/1/check/gate-python-magnumclient-docs/b269922//doc/build/html/ : SUCCESS in 58s\n- gate-python-magnumclient-python27 http://logs.openstack.org/34/170334/1/check/gate-python-magnumclient-python27/2745981/ : SUCCESS in 52s","accounts_in_message":[],"_revision_number":1},{"id":"a44d6bb16c1d577e43e28e5414dca2ccd396b129","author":{"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},"date":"2015-04-03 02:13:57.000000000","message":"Patch Set 1: Code-Review-1\n\nCould you add a unit test for this change?","accounts_in_message":[],"_revision_number":1},{"id":"03e9ee646f568911fae446f2b37cf54ce3966445","author":{"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},"date":"2015-04-03 02:27:13.000000000","message":"Patch Set 1: Code-Review-1\n\nif you fix bugs, pls open bug and add that in git message.\n\nAlso I am not sure how do you execute that command, and those needed fit in bug descrptions.\n\nThanks","accounts_in_message":[],"_revision_number":1},{"id":"7e7adfb1db0dfa1873f9d9aff73494bd32b1ec9e","author":{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"},"date":"2015-04-03 05:04:42.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"88be4b7c1ef6dee7796645b71742405f3a4593c1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-03 05:05:41.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-magnumclient-pep8 http://logs.openstack.org/34/170334/2/check/gate-python-magnumclient-pep8/7c5ccd2/ : FAILURE in 34s\n- gate-python-magnumclient-docs http://docs-draft.openstack.org/34/170334/2/check/gate-python-magnumclient-docs/9edf373//doc/build/html/ : SUCCESS in 55s\n- gate-python-magnumclient-python27 http://logs.openstack.org/34/170334/2/check/gate-python-magnumclient-python27/61965ce/ : SUCCESS in 57s","accounts_in_message":[],"_revision_number":2},{"id":"4834f01d17ac96db0f954f093395179b1e73f42c","author":{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"},"date":"2015-04-03 05:05:59.000000000","message":"Patch Set 1:\n\n@Yuanying, add the test case, thanks!\n@Kai Qiang, add the bug link in the commit message, thanks!","accounts_in_message":[],"_revision_number":1},{"id":"57fe2f168158efd6e1b497b1bfc1f53f1d1ab3dc","author":{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"},"date":"2015-04-03 05:07:18.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"f88861f94fb912eef22df721aafe46a0c702a9a0","author":{"_account_id":8580,"name":"Digambar","email":"digambarpatil15@yahoo.co.in","username":"digambar"},"date":"2015-04-03 05:08:52.000000000","message":"Patch Set 3: Code-Review-1","accounts_in_message":[],"_revision_number":3},{"id":"a858f09b79538dc7dccd2e5e964f0bb5eba3e75b","author":{"_account_id":8580,"name":"Digambar","email":"digambarpatil15@yahoo.co.in","username":"digambar"},"date":"2015-04-03 05:09:22.000000000","message":"Patch Set 3: -Code-Review","accounts_in_message":[],"_revision_number":3},{"id":"e3d43fe7b746f9f7a2e28ca95588887b4341d1f4","author":{"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},"date":"2015-04-03 05:09:54.000000000","message":"Patch Set 3: Code-Review-1\n\nCheck  my comment as :\nhttps://bugs.launchpad.net/magnum/+bug/1439943\n\nHi, I think it is optional, as you can use command as below:\n\necho \u0027{\"name\":\"mycontainer\", \"image_id\":\"cirros\"}\u0027 | magnum --debug container-create\n\nSo you not need --json format\n\n\n-------------------\nI would -1 about this change, but we can wait for others option if you think your change is necessary.\n\nThanks","accounts_in_message":[],"_revision_number":3},{"id":"fb8cc71e2b365dc110fc3a59162b18de92bd4d30","author":{"_account_id":8580,"name":"Digambar","email":"digambarpatil15@yahoo.co.in","username":"digambar"},"date":"2015-04-03 05:10:08.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"5d4bed7bdf63d1a702355d69f361edf07c15104b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-03 05:10:58.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-magnumclient-pep8 http://logs.openstack.org/34/170334/3/check/gate-python-magnumclient-pep8/7e18c5b/ : SUCCESS in 48s\n- gate-python-magnumclient-docs http://docs-draft.openstack.org/34/170334/3/check/gate-python-magnumclient-docs/f87e5ed//doc/build/html/ : SUCCESS in 50s\n- gate-python-magnumclient-python27 http://logs.openstack.org/34/170334/3/check/gate-python-magnumclient-python27/776f69b/ : SUCCESS in 57s","accounts_in_message":[],"_revision_number":3},{"id":"27f19fb280e8607a5a08f41d3a585765a9f3d21a","author":{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"},"date":"2015-04-03 05:48:33.000000000","message":"Abandoned\n\n@Kai Qiang, I think you are right, thanks for your clarification. Abandon this change.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"689adbb114fba97d1f70e7472f33776d64761f20","revisions":{"dc35e3b1e6bee3b3510bfabe2b27c6f44239e05b":{"kind":"REWORK","_number":1,"created":"2015-04-03 01:57:39.000000000","uploader":{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"},"ref":"refs/changes/34/170334/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-magnumclient","ref":"refs/changes/34/170334/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/1"}}},"commit":{"parents":[{"commit":"0665588735cfcabcde437065badc40063b2c2027","subject":"Merge \"Allow bay name when replication controller is created\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/0665588735cfcabcde437065badc40063b2c2027"}]}],"author":{"name":"lqslan","email":"lqslan@cn.ibm.com","date":"2015-04-03 01:50:03.000000000","tz":480},"committer":{"name":"lqslan","email":"lqslan@cn.ibm.com","date":"2015-04-03 01:58:38.000000000","tz":480},"subject":"Make json argument in container-create command positional","message":"Make json argument in container-create command positional\n\nCurrently, execute \"magnum container-create\" command will hangs on\nbecause the json argument is optional.\n\nI think the json should be a positional argument as this patch did.\n\nChange-Id: I97ad2cf1c3b3e7edc978920a54edd9bb3dc9d287\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/dc35e3b1e6bee3b3510bfabe2b27c6f44239e05b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/dc35e3b1e6bee3b3510bfabe2b27c6f44239e05b"}]},"branch":"refs/heads/master"},"d5af22053b00eda92657b0040504a31c9f8a05eb":{"kind":"REWORK","_number":2,"created":"2015-04-03 05:04:42.000000000","uploader":{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"},"ref":"refs/changes/34/170334/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-magnumclient","ref":"refs/changes/34/170334/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/2"}}},"commit":{"parents":[{"commit":"0665588735cfcabcde437065badc40063b2c2027","subject":"Merge \"Allow bay name when replication controller is created\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/0665588735cfcabcde437065badc40063b2c2027"}]}],"author":{"name":"lqslan","email":"lqslan@cn.ibm.com","date":"2015-04-03 01:50:03.000000000","tz":480},"committer":{"name":"lqslan","email":"lqslan@cn.ibm.com","date":"2015-04-03 05:05:19.000000000","tz":480},"subject":"Make json argument in container-create command positional","message":"Make json argument in container-create command positional\n\nCurrently, execute \"magnum container-create\" command will hangs on\nbecause the json argument is optional.\n\nThe json should be a positional argument as this patch did.\n\nChange-Id: I97ad2cf1c3b3e7edc978920a54edd9bb3dc9d287\nCloses-bug: #1439943\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/d5af22053b00eda92657b0040504a31c9f8a05eb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/d5af22053b00eda92657b0040504a31c9f8a05eb"}]},"branch":"refs/heads/master"},"689adbb114fba97d1f70e7472f33776d64761f20":{"kind":"REWORK","_number":3,"created":"2015-04-03 05:07:18.000000000","uploader":{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"},"ref":"refs/changes/34/170334/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-magnumclient","ref":"refs/changes/34/170334/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-magnumclient refs/changes/34/170334/3"}}},"commit":{"parents":[{"commit":"0665588735cfcabcde437065badc40063b2c2027","subject":"Merge \"Allow bay name when replication controller is created\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/0665588735cfcabcde437065badc40063b2c2027"}]}],"author":{"name":"lqslan","email":"lqslan@cn.ibm.com","date":"2015-04-03 01:50:03.000000000","tz":480},"committer":{"name":"lqslan","email":"lqslan@cn.ibm.com","date":"2015-04-03 05:08:10.000000000","tz":480},"subject":"Make json argument in container-create command positional","message":"Make json argument in container-create command positional\n\nCurrently, execute \"magnum container-create\" command will hangs on\nbecause the json argument is optional.\n\nThe json should be a positional argument as this patch did.\n\nChange-Id: I97ad2cf1c3b3e7edc978920a54edd9bb3dc9d287\nCloses-bug: #1439943\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/689adbb114fba97d1f70e7472f33776d64761f20"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/689adbb114fba97d1f70e7472f33776d64761f20"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}