)]}'
{"designate/api/v2/controllers/zones/__init__.py":[{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"a8d9adb425c3c1b0fcaa723cd0d11c2c8a723d83","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        zone \u003d request.body_dict"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"        # We need to check the zone type before validating the schema since if"},{"line_number":87,"context_line":"        # it\u0027s the type is SECONDARY we need to set the email to the mgmt email"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"        if isinstance(zone, dict):"},{"line_number":90,"context_line":"            if \u0027type\u0027 not in zone:"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_44097b30","side":"PARENT","line":87,"updated":"2016-01-04 15:03:04.000000000","message":"This should probably move to a spot below","commit_id":"62f578174be319f41b90b5fedf5c6f2c35a6907f"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"a8d9adb425c3c1b0fcaa723cd0d11c2c8a723d83","unresolved":false,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        zone \u003d DesignateAdapter.parse(\u0027API_v2\u0027, zone, objects.Zone())"},{"line_number":94,"context_line":"        zone.validate()"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        if zone.type \u003d\u003d \u0027SECONDARY\u0027:"},{"line_number":97,"context_line":"            mgmt_email \u003d CONF[\u0027service:central\u0027].managed_resource_email"},{"line_number":98,"context_line":"            zone[\u0027email\u0027] \u003d mgmt_email"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_e4074f1a","side":"PARENT","line":95,"updated":"2016-01-04 15:03:04.000000000","message":"comment should be here","commit_id":"62f578174be319f41b90b5fedf5c6f2c35a6907f"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"c0d270af64aa0a98f9d2ff1a5a4f23c8f3c9e353","unresolved":false,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        zone \u003d DesignateAdapter.parse(\u0027API_v2\u0027, zone, objects.Zone())"},{"line_number":94,"context_line":"        zone.validate()"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        if zone.type \u003d\u003d \u0027SECONDARY\u0027:"},{"line_number":97,"context_line":"            mgmt_email \u003d CONF[\u0027service:central\u0027].managed_resource_email"},{"line_number":98,"context_line":"            zone[\u0027email\u0027] \u003d mgmt_email"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_7b4e193c","side":"PARENT","line":95,"in_reply_to":"fa69d971_e4074f1a","updated":"2016-01-07 04:10:02.000000000","message":"hi Graham Hayes, the current process is that we firstly valid the schema in zone.validate(), then we check zone type, so I think the comments are incorrect, we should remove them","commit_id":"62f578174be319f41b90b5fedf5c6f2c35a6907f"}]}