)]}'
{"nova/tests/functional/integrated_helpers.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"12d676fe1e1a080686e539de166db86380c7198a","unresolved":false,"context_lines":[{"line_number":269,"context_line":"        except api_client.OpenStackApiNotFoundException:"},{"line_number":270,"context_line":"            return"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"    def _wait_for_action_fail_completion("},{"line_number":273,"context_line":"            self, server, expected_action, event_name):"},{"line_number":274,"context_line":"        \"\"\"Polls instance action events for the given instance, action and"},{"line_number":275,"context_line":"        action event name until it finds the action event with an error"},{"line_number":276,"context_line":"        result."},{"line_number":277,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_957ad921","line":274,"range":{"start_line":272,"start_character":0,"end_line":274,"end_character":1},"updated":"2017-09-25 08:46:13.000000000","message":"Well, now it is more generic and I\u0027m not sure any more if we can replace this by simply waiting for a compute.exception notification. Most probably it works for the current uses but will not work for every possible event_name.","commit_id":"4763dd86bf8940e81159772965ec496c6f829748"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"7aab21188527bd71db41d4285c9b2f69c0560174","unresolved":false,"context_lines":[{"line_number":269,"context_line":"        except api_client.OpenStackApiNotFoundException:"},{"line_number":270,"context_line":"            return"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"    def _wait_for_action_fail_completion("},{"line_number":273,"context_line":"            self, server, expected_action, event_name):"},{"line_number":274,"context_line":"        \"\"\"Polls instance action events for the given instance, action and"},{"line_number":275,"context_line":"        action event name until it finds the action event with an error"},{"line_number":276,"context_line":"        result."},{"line_number":277,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_fdab2b1a","line":274,"range":{"start_line":272,"start_character":0,"end_line":274,"end_character":1},"in_reply_to":"7f515b1d_91d66627","updated":"2017-09-25 14:57:33.000000000","message":"OK. I\u0027m convinced. Let\u0027s sneak in some test coverage for instance action events.","commit_id":"4763dd86bf8940e81159772965ec496c6f829748"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"34c5fbd2de65cfbb4be122ba0ea70ea81b693106","unresolved":false,"context_lines":[{"line_number":269,"context_line":"        except api_client.OpenStackApiNotFoundException:"},{"line_number":270,"context_line":"            return"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"    def _wait_for_action_fail_completion("},{"line_number":273,"context_line":"            self, server, expected_action, event_name):"},{"line_number":274,"context_line":"        \"\"\"Polls instance action events for the given instance, action and"},{"line_number":275,"context_line":"        action event name until it finds the action event with an error"},{"line_number":276,"context_line":"        result."},{"line_number":277,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_91d66627","line":274,"range":{"start_line":272,"start_character":0,"end_line":274,"end_character":1},"in_reply_to":"7f515b1d_957ad921","updated":"2017-09-25 13:07:18.000000000","message":"It should work for any method that has an instance action record and the method is decorated with the @wrap_instance_event decorator.\n\nPart of why I like using this too is it\u0027s something we don\u0027t test often but it something the end API user can do, and they can\u0027t listen for notifications. So for example, Tempest could do this.","commit_id":"4763dd86bf8940e81159772965ec496c6f829748"}]}