)]}'
{"id":"openstack%2Fpython-ironicclient~57285","triplet_id":"openstack%2Fpython-ironicclient~master~I65dc71c6fb15483fb88ef4c6a423147fe0bb54cb","project":"openstack/python-ironicclient","branch":"master","topic":"bug/1251712","hashtags":[],"change_id":"I65dc71c6fb15483fb88ef4c6a423147fe0bb54cb","subject":"Remove in-place try/except blocks from shell commands","status":"MERGED","created":"2013-11-19 17:10:11.000000000","updated":"2013-11-20 11:24:44.000000000","submitted":"2013-11-20 11:24:44.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"b55d6ec0831668b9af07e10557f2c20926a83a31","_number":57285,"virtual_id_number":57285,"owner":{"_account_id":8968,"name":"Max Lobur","email":"max_lobur@outlook.com","username":"max_lobur"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-11-20 11:24:44.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},{"value":0,"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"value":0,"_account_id":1726,"name":"Ghe Rivero","email":"ghe.rivero@gmail.com","username":"ghe.rivero"}],"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":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-11-20 09:53:18.000000000","_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":1,"date":"2013-11-20 06:00:58.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},{"value":2,"date":"2013-11-20 02:26:25.000000000","_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"value":1,"date":"2013-11-20 09:40:58.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":1726,"name":"Ghe Rivero","email":"ghe.rivero@gmail.com","username":"ghe.rivero"}],"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":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-11-20 09:53:18.000000000","_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"date":"2013-11-20 06:00:42.000000000","_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},{"value":0,"date":"2013-11-20 02:26:25.000000000","_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"value":0,"_account_id":1726,"name":"Ghe Rivero","email":"ghe.rivero@gmail.com","username":"ghe.rivero"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},{"value":0,"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"value":0,"_account_id":1726,"name":"Ghe Rivero","email":"ghe.rivero@gmail.com","username":"ghe.rivero"}],"values":{"-1":"Do Not Backport"," 0":"Backport Review Needed","+1":"Should Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1726,"name":"Ghe Rivero","email":"ghe.rivero@gmail.com","username":"ghe.rivero"},{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-11-20 02:26:25.000000000","updated_by":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"reviewer":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"state":"REVIEWER"},{"updated":"2013-11-20 06:00:42.000000000","updated_by":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"reviewer":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"state":"REVIEWER"},{"updated":"2013-11-20 09:40:58.000000000","updated_by":{"_account_id":1726,"name":"Ghe Rivero","email":"ghe.rivero@gmail.com","username":"ghe.rivero"},"reviewer":{"_account_id":1726,"name":"Ghe Rivero","email":"ghe.rivero@gmail.com","username":"ghe.rivero"},"state":"REVIEWER"},{"updated":"2013-11-20 09:53:18.000000000","updated_by":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"reviewer":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"state":"REVIEWER"},{"updated":"2013-11-20 11:24:44.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"9897d8c82efbaa3b532c6ea271c9ba5ab2a4e642","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-19 18:20:26.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-python-ironicclient-pep8 http://logs.openstack.org/85/57285/1/check/gate-python-ironicclient-pep8/4195d08 : SUCCESS in 2m 14s\n- gate-python-ironicclient-python26 http://logs.openstack.org/85/57285/1/check/gate-python-ironicclient-python26/639ad9c : SUCCESS in 53s\n- gate-python-ironicclient-python27 http://logs.openstack.org/85/57285/1/check/gate-python-ironicclient-python27/0c63c33 : SUCCESS in 2m 05s\n- gate-python-ironicclient-python33 http://logs.openstack.org/85/57285/1/check/gate-python-ironicclient-python33/282780f : FAILURE in 5m 28s (non-voting)\n- gate-python-ironicclient-pypy http://logs.openstack.org/85/57285/1/check/gate-python-ironicclient-pypy/428d978 : SUCCESS in 1m 50s\n","accounts_in_message":[],"_revision_number":1},{"id":"8d058829d1003f4bacef790e6aade610ccfa2e31","author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"date":"2013-11-20 02:26:25.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"cf827cab55dd079b86a5aba3a68a932626ba867b","author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"date":"2013-11-20 06:00:58.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":1},{"id":"d7d332fb3b7ec13ddb2ba1ae04247720808fa450","author":{"_account_id":1726,"name":"Ghe Rivero","email":"ghe.rivero@gmail.com","username":"ghe.rivero"},"date":"2013-11-20 09:40:58.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":1},{"id":"92a906ccae477a8326c9772dc2c826ed83562c80","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2013-11-20 09:53:18.000000000","message":"Patch Set 1: Looks good to me (core reviewer); Approved\n\nJust gave it a whirl, thanks max for the fix! LGTM","accounts_in_message":[],"_revision_number":1},{"id":"600bb23ae94b2c4bd8a303d0b63562e03da5b6c5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-20 10:35:53.000000000","message":"Patch Set 1: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"dd7bf794e7274e2b0b534f2c7a2202a363e8795b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-20 11:24:44.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1},{"id":"888b238c6b0315d08fa7761c3bbc6568e1f1df85","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-20 11:24:44.000000000","message":"Patch Set 1: Verified\n\nBuild succeeded.\n\n- gate-python-ironicclient-pep8 http://logs.openstack.org/85/57285/1/gate/gate-python-ironicclient-pep8/b81b873 : SUCCESS in 2m 00s\n- gate-python-ironicclient-python26 http://logs.openstack.org/85/57285/1/gate/gate-python-ironicclient-python26/6339b3a : SUCCESS in 57s\n- gate-python-ironicclient-python27 http://logs.openstack.org/85/57285/1/gate/gate-python-ironicclient-python27/046f29c : SUCCESS in 2m 09s\n- gate-python-ironicclient-python33 http://logs.openstack.org/85/57285/1/gate/gate-python-ironicclient-python33/9702a00 : FAILURE in 5m 23s (non-voting)\n- gate-python-ironicclient-pypy http://logs.openstack.org/85/57285/1/gate/gate-python-ironicclient-pypy/f13b854 : SUCCESS in 1m 44s\n","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"b55b848f1428516a1083c45f806e00635bc10c5f","revisions":{"b55b848f1428516a1083c45f806e00635bc10c5f":{"kind":"REWORK","_number":1,"created":"2013-11-19 17:10:11.000000000","uploader":{"_account_id":8968,"name":"Max Lobur","email":"max_lobur@outlook.com","username":"max_lobur"},"ref":"refs/changes/85/57285/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-ironicclient","ref":"refs/changes/85/57285/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-ironicclient refs/changes/85/57285/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-ironicclient refs/changes/85/57285/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-ironicclient refs/changes/85/57285/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-ironicclient refs/changes/85/57285/1"}}},"commit":{"parents":[{"commit":"8c394d441b5ae4300376c2698673f051954defb6","subject":"Merge \"Comply with new hacking release\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ironicclient/commit/8c394d441b5ae4300376c2698673f051954defb6"}]}],"author":{"name":"max_lobur","email":"mlobur@mirantis.com","date":"2013-11-19 16:35:06.000000000","tz":120},"committer":{"name":"max_lobur","email":"mlobur@mirantis.com","date":"2013-11-19 17:09:07.000000000","tz":120},"subject":"Remove in-place try/except blocks from shell commands","message":"Remove in-place try/except blocks from shell commands\n\nCurrently we have try/except blocks for each shell command which\nmay cause an exception, and often we unaware of all possible\nexceptions. Therefore our in-place messages may not correspond\nto real server exceptions. This patch removes in-place handlers\nso shell will rely on single try/except based in\n/ironicclient/shell.py: main, which simply prints server\u0027s\nexception message.\n\nChange-Id: I65dc71c6fb15483fb88ef4c6a423147fe0bb54cb\nCloses-Bug: #1251712\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ironicclient/commit/b55b848f1428516a1083c45f806e00635bc10c5f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ironicclient/commit/b55b848f1428516a1083c45f806e00635bc10c5f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
