)]}'
{"id":"openstack%2Fpython-ceilometerclient~366633","triplet_id":"openstack%2Fpython-ceilometerclient~master~Id8fb2f386e8101951716f30a6365c9aa15bd4b24","project":"openstack/python-ceilometerclient","branch":"master","topic":"bug/1620974","hashtags":[],"change_id":"Id8fb2f386e8101951716f30a6365c9aa15bd4b24","subject":"Fix from_response method to process response from requests","status":"MERGED","created":"2016-09-07 09:45:31.000000000","updated":"2016-09-07 12:01:46.000000000","submitted":"2016-09-07 12:01:46.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d200ee118245e7a8813dca348092baaa3285dc2b","_number":366633,"virtual_id_number":366633,"owner":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2016-09-07 12:01:46.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"}],"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":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2016-09-07 09:53:51.000000000","_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":2,"date":"2016-09-07 09:47:10.000000000","_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"}],"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":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2016-09-07 09:53:51.000000000","_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-09-07 09:47:10.000000000","updated_by":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"reviewer":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"state":"REVIEWER"},{"updated":"2016-09-07 09:53:51.000000000","updated_by":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"reviewer":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"state":"REVIEWER"},{"updated":"2016-09-07 12:01:46.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"2bb803763e318845e40b43ba9fd6e3263107b84d","author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"date":"2016-09-07 09:45:31.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"9119a277f8903438d7cff72f1d692bd66092e3f7","author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"date":"2016-09-07 09:47:10.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"cdb13d08b40847c5f1443812e07f7ce7657602c4","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2016-09-07 09:53:51.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1\n\nWhat a mess.","accounts_in_message":[],"_revision_number":1},{"id":"434fcc2970018c5cd89c895ae4a3869875b2e490","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-07 10:57:00.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-ceilometerclient-docs-ubuntu-xenial http://docs-draft.openstack.org/33/366633/1/check/gate-python-ceilometerclient-docs-ubuntu-xenial/870a346//doc/build/html/ : SUCCESS in 4m 31s\n- gate-python-ceilometerclient-pep8-ubuntu-xenial http://logs.openstack.org/33/366633/1/check/gate-python-ceilometerclient-pep8-ubuntu-xenial/44a5e14/ : SUCCESS in 2m 51s\n- gate-python-ceilometerclient-python27-ubuntu-xenial http://logs.openstack.org/33/366633/1/check/gate-python-ceilometerclient-python27-ubuntu-xenial/d36e96d/ : SUCCESS in 3m 58s\n- gate-python-ceilometerclient-python34 http://logs.openstack.org/33/366633/1/check/gate-python-ceilometerclient-python34/617ddbc/ : SUCCESS in 2m 18s\n- gate-python-ceilometerclient-python35 http://logs.openstack.org/33/366633/1/check/gate-python-ceilometerclient-python35/b42808b/ : SUCCESS in 3m 53s\n- gate-python-ceilometerclient-pypy http://logs.openstack.org/33/366633/1/check/gate-python-ceilometerclient-pypy/072578e/ : SUCCESS in 4m 07s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-ceilometerclient-plugin http://logs.openstack.org/33/366633/1/check/gate-tempest-dsvm-neutron-src-python-ceilometerclient-plugin/32c20cb/ : SUCCESS in 1h 08m 33s\n- gate-ceilometerclient-dsvm-functional http://logs.openstack.org/33/366633/1/check/gate-ceilometerclient-dsvm-functional/45a6153/ : SUCCESS in 25m 49s\n- gate-ceilometerclient-dsvm-functional-identity-v3-only http://logs.openstack.org/33/366633/1/check/gate-ceilometerclient-dsvm-functional-identity-v3-only/852d025/ : SUCCESS in 25m 48s\n- gate-telemetry-dsvm-integration-python-ceilometerclient http://logs.openstack.org/33/366633/1/check/gate-telemetry-dsvm-integration-python-ceilometerclient/41cff9b/ : SUCCESS in 29m 15s","accounts_in_message":[],"_revision_number":1},{"id":"01c066a1a1c1e92f242bc0eb44355933e2a6c974","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-07 10:57:16.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"d3f6efd63b81979a49f6119f045e7d80d22e6a34","date":"2016-09-07 12:01:46.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":1},{"id":"98ab8572c5a0185ad6c498385560dda483f05682","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-07 12:01:46.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-ceilometerclient-docs-ubuntu-xenial http://docs-draft.openstack.org/33/366633/1/gate/gate-python-ceilometerclient-docs-ubuntu-xenial/1aedf54//doc/build/html/ : SUCCESS in 3m 05s\n- gate-python-ceilometerclient-pep8-ubuntu-xenial http://logs.openstack.org/33/366633/1/gate/gate-python-ceilometerclient-pep8-ubuntu-xenial/a1079ee/ : SUCCESS in 3m 01s\n- gate-python-ceilometerclient-python27-ubuntu-xenial http://logs.openstack.org/33/366633/1/gate/gate-python-ceilometerclient-python27-ubuntu-xenial/e719de6/ : SUCCESS in 3m 15s\n- gate-python-ceilometerclient-python34 http://logs.openstack.org/33/366633/1/gate/gate-python-ceilometerclient-python34/9c2038b/ : SUCCESS in 4m 39s\n- gate-python-ceilometerclient-python35 http://logs.openstack.org/33/366633/1/gate/gate-python-ceilometerclient-python35/4da0c51/ : SUCCESS in 5m 02s\n- gate-tempest-dsvm-neutron-src-python-ceilometerclient-plugin http://logs.openstack.org/33/366633/1/gate/gate-tempest-dsvm-neutron-src-python-ceilometerclient-plugin/2c50a07/ : SUCCESS in 1h 04m 26s\n- gate-ceilometerclient-dsvm-functional http://logs.openstack.org/33/366633/1/gate/gate-ceilometerclient-dsvm-functional/47875d2/ : SUCCESS in 22m 08s\n- gate-ceilometerclient-dsvm-functional-identity-v3-only http://logs.openstack.org/33/366633/1/gate/gate-ceilometerclient-dsvm-functional-identity-v3-only/f7ee31e/ : SUCCESS in 24m 19s\n- gate-telemetry-dsvm-integration-python-ceilometerclient http://logs.openstack.org/33/366633/1/gate/gate-telemetry-dsvm-integration-python-ceilometerclient/e1c2026/ : SUCCESS in 37m 11s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"1b1917ab9b56fe92235ce336d3df3e7a8c1c81d2","revisions":{"1b1917ab9b56fe92235ce336d3df3e7a8c1c81d2":{"kind":"REWORK","_number":1,"created":"2016-09-07 09:45:31.000000000","uploader":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"ref":"refs/changes/33/366633/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-ceilometerclient","ref":"refs/changes/33/366633/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/33/366633/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/33/366633/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/33/366633/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-ceilometerclient refs/changes/33/366633/1"}}},"commit":{"parents":[{"commit":"7c6cc05ff0e9e95313384303deee69ef11929bc1","subject":"Merge \"Correct the parameters\u0027s position of assertEqual\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/7c6cc05ff0e9e95313384303deee69ef11929bc1"}]}],"author":{"name":"Andrey Kurilin","email":"akurilin@mirantis.com","date":"2016-09-07 09:39:12.000000000","tz":180},"committer":{"name":"Andrey Kurilin","email":"akurilin@mirantis.com","date":"2016-09-07 09:45:25.000000000","tz":180},"subject":"Fix from_response method to process response from requests","message":"Fix from_response method to process response from requests\n\nSessionClient uses requests library. It\u0027s response class doesn\u0027t have\n\"status\" property[1], so in case of any errors(with status code \u003e 400),\nfrom_response method is called and raises AttributeError.\n\nAlso, HTTPClient implementation uses requests lib by default[2](if \u0027http\u0027\nargument was not transmitted), so from_response method will raise\nAttributeError too.\n\n[1] - http://docs.python-requests.org/en/master/api/#requests.Response.status_code\n[2] - https://github.com/openstack/python-ceilometerclient/blob/2.6.0/ceilometerclient/openstack/common/apiclient/client.py#L99-L100\n\nChange-Id: Id8fb2f386e8101951716f30a6365c9aa15bd4b24\nCloses-Bug: #1620974\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/1b1917ab9b56fe92235ce336d3df3e7a8c1c81d2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/1b1917ab9b56fe92235ce336d3df3e7a8c1c81d2"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
