)]}'
{"id":"openstack%2Fnova~13639","triplet_id":"openstack%2Fnova~master~I797dbb0f6f0c5012859c02689907aa286c218808","project":"openstack/nova","branch":"master","topic":"log-exception","hashtags":[],"change_id":"I797dbb0f6f0c5012859c02689907aa286c218808","subject":"LOG.exception() should only be used in exception handler","status":"MERGED","created":"2012-09-25 16:31:38.000000000","updated":"2012-09-25 17:22:14.000000000","submitted":"2012-09-25 17:22:13.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"18e9f20d77a8f77af6d3d4ceb9d216b6b3821c81","_number":13639,"virtual_id_number":13639,"owner":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},{"value":2,"date":"2012-09-25 17:22:13.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},{"value":1,"date":"2012-09-25 17:00:22.000000000","_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"}],"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":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"all":[{"value":2,"date":"2012-09-25 16:44:55.000000000","_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2012-09-25 16:42:16.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"}],"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":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"all":[{"value":1,"date":"2012-09-25 16:44:55.000000000","_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2012-09-25 16:42:16.000000000","_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"}],"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":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2012-09-25 16:39:13.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":"2012-09-25 16:44:55.000000000","updated_by":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"reviewer":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"state":"REVIEWER"},{"updated":"2012-09-25 17:00:22.000000000","updated_by":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"reviewer":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2012-09-25 17:22:13.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"3ec37e963c0e954bf5077c999ee25d2badaa3a14","author":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"date":"2012-09-25 16:34:33.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"4bb9577d7a5ea3cea319f9269565e2e964dbe6f5","author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"date":"2012-09-25 16:35:28.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\n","accounts_in_message":[],"_revision_number":2},{"id":"bcf718ade90902b25ccea23a4f6bd9a8c4c59a97","author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"date":"2012-09-25 16:35:59.000000000","message":"Patch Set 2: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"1fd7ba0bfe1cac6e0cc0b6c14946273ef8b8f184","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2012-09-25 16:39:13.000000000","message":"Patch Set 2:\n\nLGTM, modulo the misspelling…","accounts_in_message":[],"_revision_number":2},{"id":"b165a761705652754016d4f8980fe3d8689f6572","author":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"date":"2012-09-25 16:41:43.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"36d0ae06df76d89d4a5db05d6ab8bffa582103d7","author":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"date":"2012-09-25 16:42:16.000000000","message":"Patch Set 3:\n\nWoops. Thanks for the pointing out my typo","accounts_in_message":[],"_revision_number":3},{"id":"374ad139ef7a62f72ef908d580a439c6bf0d597f","author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"date":"2012-09-25 16:44:55.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":3},{"id":"5d42e30092ba6bf539fef2a6dd6b67b95a726173","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-09-25 16:45:00.000000000","message":"Patch Set 3:\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"b9251486885804d851079ded0e8654d8e1f43213","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-09-25 17:00:22.000000000","message":"Patch Set 3: Works for me\n\nSmokeStack Results (patch set 3):\n\tUnit Success: http://smokestack.openstack.org/?go\u003d/jobs/36721\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/36719\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/36720\n","accounts_in_message":[],"_revision_number":3},{"id":"e1031cf18429671128e2a145f82a523db20d92a9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-09-25 17:17:08.000000000","message":"Patch Set 3: Doesn\u0027t seem to work\n\nBuild failed\n\n- http://logs.openstack.org/13639/3/check/gate-nova-merge/9329 : SUCCESS\n- http://logs.openstack.org/13639/3/check/gate-nova-docs/5090 : SUCCESS\n- http://logs.openstack.org/13639/3/check/gate-nova-pep8/10400 : SUCCESS\n- http://logs.openstack.org/13639/3/check/gate-nova-python26/6545 : SUCCESS\n- http://logs.openstack.org/13639/3/check/gate-nova-python27/6610 : SUCCESS\n- http://logs.openstack.org/13639/3/check/gate-tempest-devstack-vm/12877 : FAILURE\n- http://logs.openstack.org/13639/3/check/gate-tempest-devstack-vm-cinder/2856 : SUCCESS\n- http://logs.openstack.org/13639/3/check/gate-nova-pylint/1174 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"e527a7464715d3ea0666f641c5dd23651786ae2d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-09-25 17:22:13.000000000","message":"Patch Set 3: Verified\n\nBuild successful\n\n- http://logs.openstack.org/13639/3/gate/gate-nova-merge/9331 : SUCCESS\n- http://logs.openstack.org/13639/3/gate/gate-nova-docs/5092 : SUCCESS\n- http://logs.openstack.org/13639/3/gate/gate-nova-pep8/10402 : SUCCESS\n- http://logs.openstack.org/13639/3/gate/gate-nova-python26/6547 : SUCCESS\n- http://logs.openstack.org/13639/3/gate/gate-nova-python27/6612 : SUCCESS\n- http://logs.openstack.org/13639/3/gate/gate-tempest-devstack-vm/12879 : SUCCESS\n- http://logs.openstack.org/13639/3/gate/gate-tempest-devstack-vm-cinder/2858 : SUCCESS\n","accounts_in_message":[],"_revision_number":3},{"id":"1ef7558f4e8e47b6c2db908de465a9c6e107ece9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-09-25 17:22:14.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"f38ce7749257b3a11f40790607713ab16142865e","revisions":{"1571c3388411bc121eb4080d1d4d4c54005cc408":{"kind":"REWORK","_number":1,"created":"2012-09-25 16:31:38.000000000","uploader":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"ref":"refs/changes/39/13639/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/39/13639/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/39/13639/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/39/13639/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/39/13639/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/39/13639/1"}}},"commit":{"parents":[{"commit":"13a08f6ec7969c4ec6bd4b3bd816bb2a40f9896a","subject":"Merge \"Remove refs to ATAoE from nova docs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/13a08f6ec7969c4ec6bd4b3bd816bb2a40f9896a"}]}],"author":{"name":"Johannes Erdfelt","email":"johannes.erdfelt@rackspace.com","date":"2012-09-25 16:16:39.000000000","tz":0},"committer":{"name":"Johannes Erdfelt","email":"johannes.erdfelt@rackspace.com","date":"2012-09-25 16:17:37.000000000","tz":0},"subject":"LOG.exception() should only be used in exception handler","message":"LOG.exception() should only be used in exception handler\n\nWhen used outside of an exception handler it will print the message\nand then \u0027None\u0027, which isn\u0027t useful at all.\n\nChange-Id: I797dbb0f6f0c5012859c02689907aa286c218808\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1571c3388411bc121eb4080d1d4d4c54005cc408"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1571c3388411bc121eb4080d1d4d4c54005cc408"}]},"branch":"refs/heads/master"},"3cc195d15f4d1d12ac2639cf4340ff3b44d49860":{"kind":"REWORK","_number":2,"created":"2012-09-25 16:34:33.000000000","uploader":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"ref":"refs/changes/39/13639/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/39/13639/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/39/13639/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/39/13639/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/39/13639/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/39/13639/2"}}},"commit":{"parents":[{"commit":"13a08f6ec7969c4ec6bd4b3bd816bb2a40f9896a","subject":"Merge \"Remove refs to ATAoE from nova docs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/13a08f6ec7969c4ec6bd4b3bd816bb2a40f9896a"}]}],"author":{"name":"Johannes Erdfelt","email":"johannes.erdfelt@rackspace.com","date":"2012-09-25 16:16:39.000000000","tz":0},"committer":{"name":"Johannes Erdfelt","email":"johannes.erdfelt@rackspace.com","date":"2012-09-25 16:33:17.000000000","tz":0},"subject":"LOG.exception() should only be used in exception handler","message":"LOG.exception() should only be used in exception handler\n\nWhen used outside of an exception handler it will print the message\nand then \u0027None\u0027, which isn\u0027t useful at all.\n\nChange-Id: I797dbb0f6f0c5012859c02689907aa286c218808\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3cc195d15f4d1d12ac2639cf4340ff3b44d49860"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3cc195d15f4d1d12ac2639cf4340ff3b44d49860"}]},"branch":"refs/heads/master"},"f38ce7749257b3a11f40790607713ab16142865e":{"kind":"REWORK","_number":3,"created":"2012-09-25 16:41:43.000000000","uploader":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"ref":"refs/changes/39/13639/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/39/13639/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/39/13639/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/39/13639/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/39/13639/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/39/13639/3"}}},"commit":{"parents":[{"commit":"13a08f6ec7969c4ec6bd4b3bd816bb2a40f9896a","subject":"Merge \"Remove refs to ATAoE from nova docs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/13a08f6ec7969c4ec6bd4b3bd816bb2a40f9896a"}]}],"author":{"name":"Johannes Erdfelt","email":"johannes.erdfelt@rackspace.com","date":"2012-09-25 16:16:39.000000000","tz":0},"committer":{"name":"Johannes Erdfelt","email":"johannes.erdfelt@rackspace.com","date":"2012-09-25 16:40:27.000000000","tz":0},"subject":"LOG.exception() should only be used in exception handler","message":"LOG.exception() should only be used in exception handler\n\nWhen used outside of an exception handler it will print the message\nand then \u0027None\u0027, which isn\u0027t useful at all.\n\nChange-Id: I797dbb0f6f0c5012859c02689907aa286c218808\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f38ce7749257b3a11f40790607713ab16142865e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f38ce7749257b3a11f40790607713ab16142865e"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}