Override not editable fields during updates

.gitignore

@@ -3,6 +3,7 @@
 model-registry
 metadata.sqlite.db
 vendor
+coverage.txt
 
 # Robot Framework files
 log.html

Makefile

@@ -112,7 +112,7 @@ bin/golangci-lint:
 	curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(PROJECT_BIN) v1.54.2
 
 bin/goverter:
-	GOBIN=$(PROJECT_PATH)/bin go install github.com/jmattheis/goverter/cmd/goverter@v1.0.0
+	GOBIN=$(PROJECT_PATH)/bin go install github.com/jmattheis/goverter/cmd/goverter@v1.1.1
 
 OPENAPI_GENERATOR ?= ${PROJECT_BIN}/openapi-generator-cli
 NPM ?= "$(shell which npm)"

internal/converter/opeanpi_converter.go

@@ -2,6 +2,32 @@ package converter
 
 import "github.com/opendatahub-io/model-registry/pkg/openapi"
 
+type OpenapiUpdateWrapper[
+	M openapi.RegisteredModel |
+		openapi.ModelVersion |
+		openapi.ModelArtifact |
+		openapi.ServingEnvironment |
+		openapi.InferenceService |
+		openapi.ServeModel,
+] struct {
+	Existing *M
+	Update   *M
+}
+
+func NewOpenapiUpdateWrapper[
+	M openapi.RegisteredModel |
+		openapi.ModelVersion |
+		openapi.ModelArtifact |
+		openapi.ServingEnvironment |
+		openapi.InferenceService |
+		openapi.ServeModel,
+](existing *M, update *M) OpenapiUpdateWrapper[M] {
+	return OpenapiUpdateWrapper[M]{
+		Existing: existing,
+		Update:   update,
+	}
+}
+
 // goverter:converter
 // goverter:output:file ./generated/openapi_converter.gen.go
 // goverter:wrapErrors
@@ -43,4 +69,82 @@ type OpenAPIConverter interface {
 
 	// goverter:ignore Id CreateTimeSinceEpoch LastUpdateTimeSinceEpoch Name ModelVersionId
 	ConvertServeModelUpdate(source *openapi.ServeModelUpdate) (*openapi.ServeModel, error)
+
+	// Ignore all fields that ARE editable
+	// goverter:default InitRegisteredModelWithUpdate
+	// goverter:autoMap Existing
+	// goverter:ignore Id CreateTimeSinceEpoch LastUpdateTimeSinceEpoch Description ExternalID CustomProperties
+	OverrideNotEditableForRegisteredModel(source OpenapiUpdateWrapper[openapi.RegisteredModel]) (openapi.RegisteredModel, error)
+
+	// Ignore all fields that ARE editable
+	// goverter:default InitModelVersionWithUpdate
+	// goverter:autoMap Existing
+	// goverter:ignore Id CreateTimeSinceEpoch LastUpdateTimeSinceEpoch Description ExternalID CustomProperties
+	OverrideNotEditableForModelVersion(source OpenapiUpdateWrapper[openapi.ModelVersion]) (openapi.ModelVersion, error)
+
+	// Ignore all fields that ARE editable
+	// goverter:default InitModelArtifactWithUpdate
+	// goverter:autoMap Existing
+	// goverter:ignore Id CreateTimeSinceEpoch LastUpdateTimeSinceEpoch Description ExternalID CustomProperties Uri State ServiceAccountName ModelFormatName ModelFormatVersion StorageKey StoragePath
+	OverrideNotEditableForModelArtifact(source OpenapiUpdateWrapper[openapi.ModelArtifact]) (openapi.ModelArtifact, error)
+
+	// Ignore all fields that ARE editable
+	// goverter:default InitServingEnvironmentWithUpdate
+	// goverter:autoMap Existing
+	// goverter:ignore Id CreateTimeSinceEpoch LastUpdateTimeSinceEpoch Description ExternalID CustomProperties
+	OverrideNotEditableForServingEnvironment(source OpenapiUpdateWrapper[openapi.ServingEnvironment]) (openapi.ServingEnvironment, error)
+
+	// Ignore all fields that ARE editable
+	// goverter:default InitInferenceServiceWithUpdate
+	// goverter:autoMap Existing
+	// goverter:ignore Id CreateTimeSinceEpoch LastUpdateTimeSinceEpoch Description ExternalID CustomProperties ModelVersionId Runtime State
+	OverrideNotEditableForInferenceService(source OpenapiUpdateWrapper[openapi.InferenceService]) (openapi.InferenceService, error)
+
+	// Ignore all fields that ARE editable
+	// goverter:default InitServeModelWithUpdate
+	// goverter:autoMap Existing
+	// goverter:ignore Id CreateTimeSinceEpoch LastUpdateTimeSinceEpoch Description ExternalID CustomProperties LastKnownState
+	OverrideNotEditableForServeModel(source OpenapiUpdateWrapper[openapi.ServeModel]) (openapi.ServeModel, error)
+}
+
+func InitRegisteredModelWithUpdate(source OpenapiUpdateWrapper[openapi.RegisteredModel]) openapi.RegisteredModel {
+	if source.Update != nil {
+		return *source.Update
+	}
+	return openapi.RegisteredModel{}
+}
+
+func InitModelVersionWithUpdate(source OpenapiUpdateWrapper[openapi.ModelVersion]) openapi.ModelVersion {
+	if source.Update != nil {
+		return *source.Update
+	}
+	return openapi.ModelVersion{}
+}
+
+func InitModelArtifactWithUpdate(source OpenapiUpdateWrapper[openapi.ModelArtifact]) openapi.ModelArtifact {
+	if source.Update != nil {
+		return *source.Update
+	}
+	return openapi.ModelArtifact{}
+}
+
+func InitServingEnvironmentWithUpdate(source OpenapiUpdateWrapper[openapi.ServingEnvironment]) openapi.ServingEnvironment {
+	if source.Update != nil {
+		return *source.Update
+	}
+	return openapi.ServingEnvironment{}
+}
+
+func InitInferenceServiceWithUpdate(source OpenapiUpdateWrapper[openapi.InferenceService]) openapi.InferenceService {
+	if source.Update != nil {
+		return *source.Update
+	}
+	return openapi.InferenceService{}
+}
+
+func InitServeModelWithUpdate(source OpenapiUpdateWrapper[openapi.ServeModel]) openapi.ServeModel {
+	if source.Update != nil {
+		return *source.Update
+	}
+	return openapi.ServeModel{}
 }