)]}'
{"nodepool/tests/unit/test_driver_aws.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"c0ef4c4872c006ad433f1219ba258680cfb3f10e","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        self.security_group_id \u003d self.security_group[\u0027GroupId\u0027]"},{"line_number":65,"context_line":"        self.patch(nodepool.driver.statemachine, \u0027nodescan\u0027, fake_nodescan)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def tearDown(self):"},{"line_number":68,"context_line":"        self.mock_ec2.stop()"},{"line_number":69,"context_line":"        super().tearDown()"},{"line_number":70,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3f85bbd4_68f887d2","line":67,"updated":"2022-03-16 18:51:52.000000000","message":"Good point, I\u0027ll try to avoid these in the future.","commit_id":"89b3d6a4df1a73c42c44294004a50653354ca800"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"cffc76af8e0699a9b5ebed255d66791a0ff5835d","unresolved":true,"context_lines":[{"line_number":64,"context_line":"        self.security_group_id \u003d self.security_group[\u0027GroupId\u0027]"},{"line_number":65,"context_line":"        self.patch(nodepool.driver.statemachine, \u0027nodescan\u0027, fake_nodescan)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def tearDown(self):"},{"line_number":68,"context_line":"        self.mock_ec2.stop()"},{"line_number":69,"context_line":"        super().tearDown()"},{"line_number":70,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3bba20dd_7b2d0f23","line":67,"updated":"2022-03-16 18:35:47.000000000","message":"I think this matters less with python3 but tearDowns don\u0027t run in all situations. In particular I think if there is an exception throw in setUp() then this isn\u0027t called? using addCleanup() to add cleanup methods in setUp() has been the long time workaround for this.","commit_id":"89b3d6a4df1a73c42c44294004a50653354ca800"}]}