)]}'
{"nova/tests/functional/db/test_archive.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2aadbcb8797a5b23d45bfecf64031e5b7b8beb3c","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        # were archived."},{"line_number":73,"context_line":"        self.assertIn(\u0027instances\u0027, results,"},{"line_number":74,"context_line":"                      \u0027instances records should have been archived.\u0027)"},{"line_number":75,"context_line":"        self.assertEqual(1, results[\u0027instances\u0027])"},{"line_number":76,"context_line":"        # FIXME(mriedem): instance_actions aren\u0027t soft deleted so they aren\u0027t"},{"line_number":77,"context_line":"        # archive, which we need to fix."},{"line_number":78,"context_line":"        self.assertNotIn(\u0027instance_actions\u0027, results)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba8a016a_a37fcd15","line":75,"updated":"2015-11-18 22:36:26.000000000","message":"Actually I\u0027m surprised this worked since this should fail on a referential constraint with instance_actions.","commit_id":"cc4be8abb1e00d5ca2d495ef1feba6535e9388d5"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"bfb18b3c0fa8fdcbaab3c507be98e82a4219540d","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        found_server \u003d self.api.get_server(created_server_id)"},{"line_number":66,"context_line":"        self.assertEqual(created_server_id, found_server[\u0027id\u0027])"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        found_server \u003d self._wait_for_state_change(found_server, \u0027BUILD\u0027)"},{"line_number":69,"context_line":"        # It should be available..."},{"line_number":70,"context_line":"        self.assertEqual(\u0027ACTIVE\u0027, found_server[\u0027status\u0027])"},{"line_number":71,"context_line":"        return found_server"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba8a016a_ce916254","line":68,"updated":"2015-11-20 20:14:43.000000000","message":"s/BUILD/ACTIVE/ no?","commit_id":"91b989701d88ca4a223a172b05f978527d352dd3"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"59bafebbf8ed6bebcb6b59f09c9f9a1811384d4e","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        found_server \u003d self.api.get_server(created_server_id)"},{"line_number":66,"context_line":"        self.assertEqual(created_server_id, found_server[\u0027id\u0027])"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        found_server \u003d self._wait_for_state_change(found_server, \u0027BUILD\u0027)"},{"line_number":69,"context_line":"        # It should be available..."},{"line_number":70,"context_line":"        self.assertEqual(\u0027ACTIVE\u0027, found_server[\u0027status\u0027])"},{"line_number":71,"context_line":"        return found_server"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba8a016a_3824e2ad","line":68,"in_reply_to":"ba8a016a_869a5ac4","updated":"2015-11-21 14:16:40.000000000","message":"@mriedem: LOL, thanks for clueing me in :)","commit_id":"91b989701d88ca4a223a172b05f978527d352dd3"},{"author":{"_account_id":14819,"name":"Timofey Durakov","email":"timofei.nd@gmail.com","username":"tdurakov"},"change_message_id":"0c72ae15ededfe9fca4cbf10bf128828dd85f7be","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        found_server \u003d self.api.get_server(created_server_id)"},{"line_number":66,"context_line":"        self.assertEqual(created_server_id, found_server[\u0027id\u0027])"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        found_server \u003d self._wait_for_state_change(found_server, \u0027BUILD\u0027)"},{"line_number":69,"context_line":"        # It should be available..."},{"line_number":70,"context_line":"        self.assertEqual(\u0027ACTIVE\u0027, found_server[\u0027status\u0027])"},{"line_number":71,"context_line":"        return found_server"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba8a016a_0c6c4807","line":68,"in_reply_to":"ba8a016a_869a5ac4","updated":"2015-11-21 08:08:43.000000000","message":"big plus for cleaning this, also was caught by strange \u0027ACTIVE\u0027 param on line 70.","commit_id":"91b989701d88ca4a223a172b05f978527d352dd3"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"09372eeffcdee48b8266cdf1f3738204f0ec3807","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        found_server \u003d self.api.get_server(created_server_id)"},{"line_number":66,"context_line":"        self.assertEqual(created_server_id, found_server[\u0027id\u0027])"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        found_server \u003d self._wait_for_state_change(found_server, \u0027BUILD\u0027)"},{"line_number":69,"context_line":"        # It should be available..."},{"line_number":70,"context_line":"        self.assertEqual(\u0027ACTIVE\u0027, found_server[\u0027status\u0027])"},{"line_number":71,"context_line":"        return found_server"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba8a016a_869a5ac4","line":68,"in_reply_to":"ba8a016a_ce916254","updated":"2015-11-20 23:55:51.000000000","message":"No, this is actually correct if you look at the method in the parent class (I copied this from another test in test_servers.py) - it\u0027s waiting for the state to change *from* BUILD, not for it to go to ACTIVE. It\u0027s weird, I know, and we should probably clean that up at some point.","commit_id":"91b989701d88ca4a223a172b05f978527d352dd3"}]}