)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b3ed929d4a41ac8291fd1628741c8859ea30845c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"VMware: Handle concurrent registrations of the VC extension"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"During initialization, the Nova compute driver checks whether a VC"},{"line_number":10,"context_line":"extension with key \u0027org.openstack.compute\u0027 exists and if not, it"},{"line_number":11,"context_line":"registers one. This is a race condition. If multiple services try to"},{"line_number":12,"context_line":"register the same extension, only one of them will succeed. The fix is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff82abbf_bc4353af","line":9,"range":{"start_line":9,"start_character":64,"end_line":9,"end_character":66},"updated":"2017-11-22 15:36:13.000000000","message":"VC?","commit_id":"8f91874163b877c1374dcde9bdde19203c9115a9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"96b09008c64fbcefb382c34e7f052923ef1d1fd0","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"VMware: Handle concurrent registrations of the VC extension"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"During initialization, the Nova compute driver checks whether a VC"},{"line_number":10,"context_line":"extension with key \u0027org.openstack.compute\u0027 exists and if not, it"},{"line_number":11,"context_line":"registers one. This is a race condition. If multiple services try to"},{"line_number":12,"context_line":"register the same extension, only one of them will succeed. The fix is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff82abbf_2e72ece4","line":9,"range":{"start_line":9,"start_character":64,"end_line":9,"end_character":66},"in_reply_to":"ff82abbf_bc4353af","updated":"2017-11-30 19:07:45.000000000","message":"vCenter","commit_id":"8f91874163b877c1374dcde9bdde19203c9115a9"}],"nova/virt/vmwareapi/driver.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b3ed929d4a41ac8291fd1628741c8859ea30845c","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                    LOG.debug(\u0027Extension %s already exists.\u0027,"},{"line_number":194,"context_line":"                              constants.EXTENSION_KEY)"},{"line_number":195,"context_line":"                else:"},{"line_number":196,"context_line":"                    raise"},{"line_number":197,"context_line":"        else:"},{"line_number":198,"context_line":"            LOG.debug(\u0027Extension %s already exists.\u0027, constants.EXTENSION_KEY)"},{"line_number":199,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_1c0eff8c","line":196,"range":{"start_line":196,"start_character":20,"end_line":196,"end_character":25},"updated":"2017-11-22 15:36:13.000000000","message":"raise e?","commit_id":"8f91874163b877c1374dcde9bdde19203c9115a9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"96b09008c64fbcefb382c34e7f052923ef1d1fd0","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                    LOG.debug(\u0027Extension %s already exists.\u0027,"},{"line_number":194,"context_line":"                              constants.EXTENSION_KEY)"},{"line_number":195,"context_line":"                else:"},{"line_number":196,"context_line":"                    raise"},{"line_number":197,"context_line":"        else:"},{"line_number":198,"context_line":"            LOG.debug(\u0027Extension %s already exists.\u0027, constants.EXTENSION_KEY)"},{"line_number":199,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_ce2ca0ae","line":196,"range":{"start_line":196,"start_character":20,"end_line":196,"end_character":25},"in_reply_to":"ff82abbf_1c0eff8c","updated":"2017-11-30 19:07:45.000000000","message":"Hmm, the better thing to use here would be the excutils code from oslo_utils, so something like:\n\nexcept vexc.VimFaultException as e:\n    with excutils.save_and_reraise_exception() as ctx:\n        if \u0027InvalidArgument\u0027 in e.fault_list:\n            ...\n            ctx.reraise \u003d False","commit_id":"8f91874163b877c1374dcde9bdde19203c9115a9"}]}