)]}'
{"designate_tempest_plugin/tests/api/v2/test_zones_imports.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4fbe4ffe63df0f79ff0b569397478c887a08513c","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"class BaseZonesImportTest(base.BaseDnsV2Test):"},{"line_number":25,"context_line":"    excluded_keys \u003d [\u0027created_at\u0027, \u0027updated_at\u0027, \u0027version\u0027, \u0027links\u0027,"},{"line_number":26,"context_line":"                     \u0027status\u0027, \u0027message\u0027, \u0027zone_id\u0027, \u0027status\u0027]"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"class ZonesImportTest(BaseZonesImportTest):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_53aba673","line":26,"updated":"2017-11-28 11:36:38.000000000","message":"status is already present here","commit_id":"5ec967cbcc3691936b85904201c8493e2c4dcaca"},{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"16524e3c5033fe9f0c67f3829d750a6df24a6ee2","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"class BaseZonesImportTest(base.BaseDnsV2Test):"},{"line_number":25,"context_line":"    excluded_keys \u003d [\u0027created_at\u0027, \u0027updated_at\u0027, \u0027version\u0027, \u0027links\u0027,"},{"line_number":26,"context_line":"                     \u0027status\u0027, \u0027message\u0027, \u0027zone_id\u0027, \u0027status\u0027]"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"class ZonesImportTest(BaseZonesImportTest):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_932a8ed8","line":26,"in_reply_to":"ff82abbf_53aba673","updated":"2017-11-28 11:40:23.000000000","message":"Done","commit_id":"5ec967cbcc3691936b85904201c8493e2c4dcaca"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4fbe4ffe63df0f79ff0b569397478c887a08513c","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        self.addCleanup(self.client.delete_zone_import, zone_import[\u0027id\u0027])"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        LOG.info(\u0027Re-Fetch the zone import\u0027)"},{"line_number":52,"context_line":"        resp, body \u003d self.client.show_zone_import(zone_import[\u0027id\u0027])"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        LOG.info(\u0027Ensure the fetched response matches the expected one\u0027)"},{"line_number":55,"context_line":"        self.assertExpected(zone_import, body, self.excluded_keys)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_73d4aae9","line":52,"updated":"2017-11-28 11:36:38.000000000","message":"Wouldn\u0027t you rather need to insert a waiter here? That would be more in line with what the bug report indicates.","commit_id":"5ec967cbcc3691936b85904201c8493e2c4dcaca"},{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"16524e3c5033fe9f0c67f3829d750a6df24a6ee2","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        self.addCleanup(self.client.delete_zone_import, zone_import[\u0027id\u0027])"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        LOG.info(\u0027Re-Fetch the zone import\u0027)"},{"line_number":52,"context_line":"        resp, body \u003d self.client.show_zone_import(zone_import[\u0027id\u0027])"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        LOG.info(\u0027Ensure the fetched response matches the expected one\u0027)"},{"line_number":55,"context_line":"        self.assertExpected(zone_import, body, self.excluded_keys)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_f30f5a85","line":52,"in_reply_to":"ff82abbf_73d4aae9","updated":"2017-11-28 11:40:23.000000000","message":"Yeah that\u0027s what I thought first, but the test is about comparing the response between show and create, and that wouldn\u0027t work if I use a waiter. I would just compare 2 show response, which doesn\u0027t make much sense.","commit_id":"5ec967cbcc3691936b85904201c8493e2c4dcaca"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"68ad732103956cb145959f25909b864145034430","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        self.addCleanup(self.client.delete_zone_import, zone_import[\u0027id\u0027])"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        LOG.info(\u0027Re-Fetch the zone import\u0027)"},{"line_number":52,"context_line":"        resp, body \u003d self.client.show_zone_import(zone_import[\u0027id\u0027])"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        LOG.info(\u0027Ensure the fetched response matches the expected one\u0027)"},{"line_number":55,"context_line":"        self.assertExpected(zone_import, body, self.excluded_keys)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_9e27f5ca","line":52,"in_reply_to":"ff82abbf_f30f5a85","updated":"2017-11-28 12:52:45.000000000","message":"That sounds plausible indeed, thanks for explaining.","commit_id":"5ec967cbcc3691936b85904201c8493e2c4dcaca"}]}
