refactor(crds)!: consistently use message field instead of error field

api/v1alpha1/stage_types.go

@@ -511,9 +511,9 @@ type StageStatus struct {
 	History FreightReferenceStack `json:"history,omitempty"`
 	// Health is the Stage's last observed health.
 	Health *Health `json:"health,omitempty"`
-	// Error describes any errors that are preventing the Stage controller
+	// Message describes any errors that are preventing the Stage controller
 	// from assessing Stage health or from finding new Freight.
-	Error string `json:"error,omitempty"`
+	Message string `json:"message,omitempty"`
 	// ObservedGeneration represents the .metadata.generation that this Stage
 	// status was reconciled against.
 	ObservedGeneration int64 `json:"observedGeneration,omitempty"`

api/v1alpha1/types.proto

@@ -268,7 +268,7 @@ message FreightReference {
 message StageStatus {
   optional FreightReference current_freight = 2 [json_name = "currentFreight"];
   repeated FreightReference history = 3 [json_name = "history"];
-  string error = 4 [json_name = "error"];
+  string message = 4 [json_name = "message"];
   optional Health health = 5 [json_name = "health"];
   optional PromotionInfo current_promotion = 6 [json_name = "currentPromotion"];
   string phase = 7 [json_name = "phase"];
@@ -296,7 +296,7 @@ message WarehouseSpec {
 }
 
 message WarehouseStatus {
-  string error = 1 [json_name = "error"];
+  string message = 1 [json_name = "message"];
   int64 observed_generation = 2 [json_name = "observedGeneration"];
 }
 

api/v1alpha1/warehouse_types.go

@@ -210,9 +210,9 @@ type ChartSubscription struct {
 
 // WarehouseStatus describes a Warehouse's most recently observed state.
 type WarehouseStatus struct {
-	// Error describes any errors that are preventing the Warehouse controller
+	// Message describes any errors that are preventing the Warehouse controller
 	// from polling repositories to discover new Freight.
-	Error string `json:"error,omitempty"`
+	Message string `json:"message,omitempty"`
 	// ObservedGeneration represents the .metadata.generation that this Warehouse
 	// was reconciled against.
 	ObservedGeneration int64 `json:"observedGeneration,omitempty"`

charts/kargo/crds/kargo.akuity.io_stages.yaml

@@ -844,11 +844,6 @@ spec:
                 - freight
                 - name
                 type: object
-              error:
-                description: |-
-                  Error describes any errors that are preventing the Stage controller
-                  from assessing Stage health or from finding new Freight.
-                type: string
               health:
                 description: Health is the Stage's last observed health.
                 properties:
@@ -1056,6 +1051,11 @@ spec:
                       type: object
                   type: object
                 type: array
+              message:
+                description: |-
+                  Message describes any errors that are preventing the Stage controller
+                  from assessing Stage health or from finding new Freight.
+                type: string
               observedGeneration:
                 description: |-
                   ObservedGeneration represents the .metadata.generation that this Stage

charts/kargo/crds/kargo.akuity.io_warehouses.yaml

@@ -235,9 +235,9 @@ spec:
           status:
             description: Status describes the Warehouse's most recently observed state.
             properties:
-              error:
+              message:
                 description: |-
-                  Error describes any errors that are preventing the Warehouse controller
+                  Message describes any errors that are preventing the Warehouse controller
                   from polling repositories to discover new Freight.
                 type: string
               observedGeneration:

internal/api/types/v1alpha1/types.go

@@ -90,7 +90,7 @@
 		CurrentFreight: FromFreightReferenceProto(s.GetCurrentFreight()),
 		History:        history,
 		Health:         FromHealthProto(s.GetHealth()),
-		Error:          s.GetError(),
+		Message:        s.GetMessage(),
 	}
 }
 
@@ -760,7 +760,7 @@
 			CurrentPromotion: currentPromotion,
 			History:          history,
 			Health:           health,
-			Error:            e.Status.Error,
+			Message:          e.Status.Message,
 		},
 	}
 }
@@ -1082,7 +1082,7 @@
 	var status *v1alpha1.WarehouseStatus
 	if w.GetStatus() != nil {
 		status = &v1alpha1.WarehouseStatus{
-			Error:              w.GetStatus().Error,
+			Message:            w.GetStatus().Message,
 			ObservedGeneration: w.GetStatus().ObservedGeneration,
 		}
 	}

internal/controller/stages/stages.go

@@ -521,12 +521,12 @@
 		newStatus, err = r.syncNormalStage(ctx, stage)
 	}
 	if err != nil {
-		newStatus.Error = err.Error()
-		logger.Errorf("error syncing Stage: %s", stage.Status.Error)
+		newStatus.Message = err.Error()
+		logger.Errorf("error syncing Stage: %s", stage.Status.Message)
 	} else {
 		// Be sure to blank this out in case there's an error in this field from
 		// the previous reconciliation
-		newStatus.Error = ""
+		newStatus.Message = ""
 	}
 
 	updateErr := kubeclient.PatchStatus(ctx, r.kargoClient, stage, func(status *kargoapi.StageStatus) {

internal/controller/warehouses/warehouses.go

@@ -191,7 +191,7 @@
 
 	newStatus, err := r.syncWarehouse(ctx, warehouse)
 	if err != nil {
-		newStatus.Error = err.Error()
+		newStatus.Message = err.Error()
 		logger.Errorf("error syncing Warehouse: %s", err)
 	}
 
@@ -236,7 +236,7 @@
 ) (kargoapi.WarehouseStatus, error) {
 	status := *warehouse.Status.DeepCopy()
 	status.ObservedGeneration = warehouse.Generation
-	status.Error = "" // Clear any previous error
+	status.Message = "" // Clear any previous error
 
 	logger := logging.LoggerFromContext(ctx)