)]}'
{"test/probe/test_account_reaper.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"607f647676bc3a6ad36df9d7517f42ef00f019ce","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        # This test can leave an unreclaimable account (for a week in default)"},{"line_number":63,"context_line":"        # which blocks to create AUTH_test account in your Swift environment"},{"line_number":64,"context_line":"        # so that call resetswift explicitly after all tests ran."},{"line_number":65,"context_line":"        resetswift()"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def _verify_account_reaped(self):"},{"line_number":68,"context_line":"        for policy, container, obj in self.all_objects:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_844c9471","line":65,"updated":"2016-08-16 08:47:52.000000000","message":"I\u0027m not sure about this ... in general, it can be useful for debugging if the swift test cluster is left in whatever state the tests create. With this change it would be impossible to debug the state of the cluster if a test failed.\n\nYes, it does mean that a call to resetswift may be needed after running probe tests (something that perhaps we could document).","commit_id":"5e94e52f2a121c164fd44d8748bc4e2e5546c2cb"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"d3de553ff6baadf71019104f67ef8a10f23a2056","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        # This test can leave an unreclaimable account (for a week in default)"},{"line_number":63,"context_line":"        # which blocks to create AUTH_test account in your Swift environment"},{"line_number":64,"context_line":"        # so that call resetswift explicitly after all tests ran."},{"line_number":65,"context_line":"        resetswift()"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def _verify_account_reaped(self):"},{"line_number":68,"context_line":"        for policy, container, obj in self.all_objects:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_9de5850a","line":65,"in_reply_to":"3ac371cc_844c9471","updated":"2016-08-18 09:35:13.000000000","message":"Fair point. I am thinking this test_account_reaper is especially special case because if account is deleted (and reaped), we cannot do anything until resetswift() or reclaim_age passed.\n\nHowever, I understand your opinion, sweeping everything may be worse for debugging purpose.","commit_id":"5e94e52f2a121c164fd44d8748bc4e2e5546c2cb"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"f75f4abf7e3efde83a9ee6821201c28d751d041e","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        # This test can leave an unreclaimable account (for a week in default)"},{"line_number":63,"context_line":"        # which blocks to create AUTH_test account in your Swift environment"},{"line_number":64,"context_line":"        # so that call resetswift explicitly after all tests ran."},{"line_number":65,"context_line":"        resetswift()"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def _verify_account_reaped(self):"},{"line_number":68,"context_line":"        for policy, container, obj in self.all_objects:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_da16d84a","line":65,"in_reply_to":"3ac371cc_9de5850a","updated":"2016-08-18 10:17:25.000000000","message":"Done","commit_id":"5e94e52f2a121c164fd44d8748bc4e2e5546c2cb"}]}