)]}'
{"cyborg_tempest_plugin/tests/scenario/test_accelerator_basic_ops.py":[{"author":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"change_message_id":"3748e141d8e927558463d1b6f82a69dfd95621a7","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"def server_test(test_func):"},{"line_number":24,"context_line":"    \"\"\"Decorator for server test methods: Creates/deletes VMs.\"\"\""},{"line_number":25,"context_line":"    def wrapper(self, *args, **kwargs):"},{"line_number":26,"context_line":"        devprof \u003d self.create_device_profile()"},{"line_number":27,"context_line":"        device_profile_name \u003d devprof[\"name\"]"},{"line_number":28,"context_line":"        flavor \u003d self.create_accel_flavor(device_profile_name)"}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_297496fd","line":25,"updated":"2020-04-01 05:56:56.000000000","message":"Add @functools.wraps(func)","commit_id":"60c03393820eb314c599b2507557c065827f2e8e"},{"author":{"_account_id":29745,"name":"Dustin Cowles","email":"cowlesd@gmail.com","username":"dustinc","status":"inactive"},"change_message_id":"9743ef81421344c9cee0cfa657628a29f3d19368","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    \"\"\"Test suite for basic VM operations with accelerators."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    Each test case follows this basic set of operations:"},{"line_number":43,"context_line":"     * Create a device profile and a flavor with that device profile."},{"line_number":44,"context_line":"     * Launch an instance with that flavor."},{"line_number":45,"context_line":"       \u003c-- Above two are done in the decorator \u0027server_test\u0027."}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_a2674bb8","line":42,"range":{"start_line":42,"start_character":4,"end_line":42,"end_character":55},"updated":"2019-11-19 18:31:16.000000000","message":"very helpful docstring","commit_id":"60c03393820eb314c599b2507557c065827f2e8e"},{"author":{"_account_id":29745,"name":"Dustin Cowles","email":"cowlesd@gmail.com","username":"dustinc","status":"inactive"},"change_message_id":"9fa093f9e7df73e80859703a3a8fd12d6d6f68fe","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        self.assertEqual(instance[\u0027status\u0027], \u0027ACTIVE\u0027)"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":62,"context_line":"    def _server_reboot(self, instance, reboot_type):"},{"line_number":63,"context_line":"        \"\"\"Test for rebooting a VM with attached accelerator"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"           :param reboot_type: \u0027SOFT\u0027 or \u0027HARD\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_60ca6b19","line":62,"updated":"2020-01-09 19:47:01.000000000","message":"here","commit_id":"60c03393820eb314c599b2507557c065827f2e8e"},{"author":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"change_message_id":"3748e141d8e927558463d1b6f82a69dfd95621a7","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        self.assertEqual(instance[\u0027status\u0027], \u0027ACTIVE\u0027)"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":62,"context_line":"    def _server_reboot(self, instance, reboot_type):"},{"line_number":63,"context_line":"        \"\"\"Test for rebooting a VM with attached accelerator"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"           :param reboot_type: \u0027SOFT\u0027 or \u0027HARD\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_497f5adf","line":62,"in_reply_to":"3fa7e38b_60ca6b19","updated":"2020-04-01 05:56:56.000000000","message":"Done","commit_id":"60c03393820eb314c599b2507557c065827f2e8e"},{"author":{"_account_id":29745,"name":"Dustin Cowles","email":"cowlesd@gmail.com","username":"dustinc","status":"inactive"},"change_message_id":"9743ef81421344c9cee0cfa657628a29f3d19368","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    def test_server_pause(self, instance):"},{"line_number":88,"context_line":"        \"\"\"Test for rebooting a VM with attached accelerator\"\"\""},{"line_number":89,"context_line":"        self.servers_client.pause_server(instance[\u0027id\u0027])"},{"line_number":90,"context_line":"        waiters.wait_for_server_status("},{"line_number":91,"context_line":"            self.servers_client, instance[\u0027id\u0027], \u0027PAUSED\u0027)"},{"line_number":92,"context_line":"        self.servers_client.unpause_server(instance[\u0027id\u0027])"},{"line_number":93,"context_line":"        waiters.wait_for_server_status("}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_c24b6725","line":90,"updated":"2019-11-19 18:31:16.000000000","message":"nit: I think it is cleaner to group all the helper functions at the top and keep tests grouped together (e.g. move this f() to before L58)","commit_id":"60c03393820eb314c599b2507557c065827f2e8e"},{"author":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"change_message_id":"3748e141d8e927558463d1b6f82a69dfd95621a7","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    def test_server_pause(self, instance):"},{"line_number":88,"context_line":"        \"\"\"Test for rebooting a VM with attached accelerator\"\"\""},{"line_number":89,"context_line":"        self.servers_client.pause_server(instance[\u0027id\u0027])"},{"line_number":90,"context_line":"        waiters.wait_for_server_status("},{"line_number":91,"context_line":"            self.servers_client, instance[\u0027id\u0027], \u0027PAUSED\u0027)"},{"line_number":92,"context_line":"        self.servers_client.unpause_server(instance[\u0027id\u0027])"},{"line_number":93,"context_line":"        waiters.wait_for_server_status("}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_a980a6c0","line":90,"in_reply_to":"3fa7e38b_0044d79d","updated":"2020-04-01 05:56:56.000000000","message":"ok","commit_id":"60c03393820eb314c599b2507557c065827f2e8e"},{"author":{"_account_id":29745,"name":"Dustin Cowles","email":"cowlesd@gmail.com","username":"dustinc","status":"inactive"},"change_message_id":"9fa093f9e7df73e80859703a3a8fd12d6d6f68fe","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    def test_server_pause(self, instance):"},{"line_number":88,"context_line":"        \"\"\"Test for rebooting a VM with attached accelerator\"\"\""},{"line_number":89,"context_line":"        self.servers_client.pause_server(instance[\u0027id\u0027])"},{"line_number":90,"context_line":"        waiters.wait_for_server_status("},{"line_number":91,"context_line":"            self.servers_client, instance[\u0027id\u0027], \u0027PAUSED\u0027)"},{"line_number":92,"context_line":"        self.servers_client.unpause_server(instance[\u0027id\u0027])"},{"line_number":93,"context_line":"        waiters.wait_for_server_status("}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_0044d79d","line":90,"in_reply_to":"3fa7e38b_7fc9dd21","updated":"2020-01-09 19:47:01.000000000","message":"Not sure why this is down here, I meant to leave it on line 62","commit_id":"60c03393820eb314c599b2507557c065827f2e8e"},{"author":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"change_message_id":"047b3e95f50a1b7f82ba89da342fd1b2aabd3914","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    def test_server_pause(self, instance):"},{"line_number":88,"context_line":"        \"\"\"Test for rebooting a VM with attached accelerator\"\"\""},{"line_number":89,"context_line":"        self.servers_client.pause_server(instance[\u0027id\u0027])"},{"line_number":90,"context_line":"        waiters.wait_for_server_status("},{"line_number":91,"context_line":"            self.servers_client, instance[\u0027id\u0027], \u0027PAUSED\u0027)"},{"line_number":92,"context_line":"        self.servers_client.unpause_server(instance[\u0027id\u0027])"},{"line_number":93,"context_line":"        waiters.wait_for_server_status("}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_7fc9dd21","line":90,"in_reply_to":"3fa7e38b_c24b6725","updated":"2020-01-06 23:57:34.000000000","message":"This function is a test case, not a helper function for another test case. I don\u0027t understand the comment.","commit_id":"60c03393820eb314c599b2507557c065827f2e8e"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"5fd97dda6f7f5d62fb318a38dbc8a12fcdd4d544","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"def server_test(test_func):"},{"line_number":24,"context_line":"    \"\"\"Decorator for server test methods: Creates/deletes VMs.\"\"\""},{"line_number":25,"context_line":"    @functools.wraps(test_func)"},{"line_number":26,"context_line":"    def wrapper(self, *args, **kwargs):"},{"line_number":27,"context_line":"        devprof \u003d self.create_device_profile()"},{"line_number":28,"context_line":"        device_profile_name \u003d devprof[\"name\"]"}],"source_content_type":"text/x-python","patch_set":4,"id":"df33271e_49ce395a","line":25,"updated":"2020-04-01 07:02:27.000000000","message":"F821 undefined name \u0027functools\u0027","commit_id":"23f2c9a2f6c19eff64ffe079653e9270cc609509"}]}