)]}'
{"nova/objects/instance_group.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5f1e44dd5a4a3c21837fffbd262c8e57560d0a78","unresolved":false,"context_lines":[{"line_number":356,"context_line":""},{"line_number":357,"context_line":"    @base.remotable"},{"line_number":358,"context_line":"    def create(self):"},{"line_number":359,"context_line":"        # NOTE(danms): This is just for the migration routine, and"},{"line_number":360,"context_line":"        # can be removed once we\u0027re no longer supporting the migration"},{"line_number":361,"context_line":"        # of instance groups from the main to api database."},{"line_number":362,"context_line":"        if self.obj_attr_is_set(\u0027id\u0027):"},{"line_number":363,"context_line":"            raise exception.ObjectActionError(action\u003d\u0027create\u0027,"},{"line_number":364,"context_line":"                                              reason\u003d\u0027already created\u0027)"},{"line_number":365,"context_line":"        updates \u003d self.obj_get_changes()"},{"line_number":366,"context_line":"        payload \u003d dict(updates)"},{"line_number":367,"context_line":"        updates.pop(\u0027id\u0027, None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_52304307","line":364,"range":{"start_line":359,"start_character":8,"end_line":364,"end_character":71},"updated":"2018-06-27 13:43:17.000000000","message":"I missed this.","commit_id":"b5fb9acf7223c7c1a686576c7b30b57cef7c15cc"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"34d755c50b30e6305762064a5de47f5430ad0ea3","unresolved":false,"context_lines":[{"line_number":356,"context_line":""},{"line_number":357,"context_line":"    @base.remotable"},{"line_number":358,"context_line":"    def create(self):"},{"line_number":359,"context_line":"        # NOTE(danms): This is just for the migration routine, and"},{"line_number":360,"context_line":"        # can be removed once we\u0027re no longer supporting the migration"},{"line_number":361,"context_line":"        # of instance groups from the main to api database."},{"line_number":362,"context_line":"        if self.obj_attr_is_set(\u0027id\u0027):"},{"line_number":363,"context_line":"            raise exception.ObjectActionError(action\u003d\u0027create\u0027,"},{"line_number":364,"context_line":"                                              reason\u003d\u0027already created\u0027)"},{"line_number":365,"context_line":"        updates \u003d self.obj_get_changes()"},{"line_number":366,"context_line":"        payload \u003d dict(updates)"},{"line_number":367,"context_line":"        updates.pop(\u0027id\u0027, None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_b22437c4","line":364,"range":{"start_line":359,"start_character":8,"end_line":364,"end_character":71},"in_reply_to":"5f7c97a3_52304307","updated":"2018-06-27 13:44:24.000000000","message":"To be clear, I think the comment was more about the skipcheck kwarg and not the id check, which is pretty common on object create() methods.","commit_id":"b5fb9acf7223c7c1a686576c7b30b57cef7c15cc"}]}