[Feature] [Platform] Chart Overrides (#1913)

Author: ajanikow

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## [master](https://github.com/arangodb/kube-arangodb/tree/master) (N/A)
 - (Feature) (Platform) MetaV1 Integration Service
+- (Feature) (Platform) Chart Overrides
 
 ## [1.2.49](https://github.com/arangodb/kube-arangodb/tree/1.2.49) (2025-06-17)
 - (Maintenance) Optimize go.mod

docs/api/ArangoPlatformChart.V1Alpha1.md

@@ -10,5 +10,15 @@ title: ArangoPlatformChart V1Alpha1
 
 ### .spec.definition
 
-Type: `array` <sup>[\[ref\]](https://github.com/arangodb/kube-arangodb/blob/1.2.49/pkg/apis/platform/v1alpha1/chart_spec.go#L30)</sup>
+Type: `array` <sup>[\[ref\]](https://github.com/arangodb/kube-arangodb/blob/1.2.49/pkg/apis/platform/v1alpha1/chart_spec.go#L31)</sup>
+
+Definition keeps the Chart base64 encoded definition
+
+***
+
+### .spec.overrides
+
+Type: `Object` <sup>[\[ref\]](https://github.com/arangodb/kube-arangodb/blob/1.2.49/pkg/apis/platform/v1alpha1/chart_spec.go#L35)</sup>
+
+Overrides keeps the Chart overrides
 

integrations/scheduler/v2/chart.go

@@ -69,14 +69,17 @@ func (i *implementation) ListCharts(req *pbSchedulerV2.SchedulerV2ListChartsRequ
 func (i *implementation) GetChart(ctx context.Context, in *pbSchedulerV2.SchedulerV2GetChartRequest) (*pbSchedulerV2.SchedulerV2GetChartResponse, error) {
 	resp, err := i.kclient.Arango().PlatformV1alpha1().ArangoPlatformCharts(i.client.Namespace()).Get(ctx, in.GetName(), meta.GetOptions{})
 	if err != nil {
-		logger.Err(err).Warn("Unable to run action: GetChart")
 		return nil, asGRPCError(err)
 	}
 
 	if !resp.Status.Conditions.IsTrue(platformApi.SpecValidCondition) {
 		return nil, status.Errorf(codes.Unavailable, "Chart Spec is invalid")
 	}
 
+	if !resp.Ready() {
+		return nil, status.Errorf(codes.Unavailable, "Chart is not Ready")
+	}
+
 	if info := resp.Status.Info; info == nil {
 		return nil, status.Errorf(codes.Unavailable, "Chart Infos are missing")
 	} else {
@@ -91,8 +94,9 @@ func (i *implementation) GetChart(ctx context.Context, in *pbSchedulerV2.Schedul
 				return nil, status.Errorf(codes.Unavailable, "Chart Details are missing")
 			} else {
 				return &pbSchedulerV2.SchedulerV2GetChartResponse{
-					Chart: info.Definition,
-					Info:  chartInfoDetailsAsInfo(details),
+					Chart:     info.Definition,
+					Info:      chartInfoDetailsAsInfo(details),
+					Overrides: info.Overrides,
 				}, nil
 			}
 		}

integrations/scheduler/v2/chart_test.go

@@ -62,6 +62,10 @@ func Test_Chart_List(t *testing.T) {
 							Type:   platformApi.SpecValidCondition,
 							Status: core.ConditionTrue,
 						},
+						{
+							Type:   platformApi.ReadyCondition,
+							Status: core.ConditionTrue,
+						},
 					},
 					Info: &platformApi.ChartStatusInfo{
 						Definition: make([]byte, 128),
@@ -208,6 +212,10 @@ func Test_Chart_Get(t *testing.T) {
 					Type:   platformApi.SpecValidCondition,
 					Status: core.ConditionTrue,
 				},
+				{
+					Type:   platformApi.ReadyCondition,
+					Status: core.ConditionTrue,
+				},
 			},
 			Info: &platformApi.ChartStatusInfo{
 				Definition: make([]byte, 128),
@@ -235,25 +243,25 @@ func Test_Chart_Get(t *testing.T) {
 
 	t.Run("Invalid Info", func(t *testing.T) {
 		resp, err := scheduler.GetChart(context.Background(), &pbSchedulerV2.SchedulerV2GetChartRequest{Name: t2.GetName()})
-		tgrpc.AsGRPCError(t, err).Code(t, codes.Unavailable).Errorf(t, "Chart Infos are missing")
+		tgrpc.AsGRPCError(t, err).Code(t, codes.Unavailable).Errorf(t, "Chart is not Ready")
 		require.Nil(t, resp)
 	})
 
 	t.Run("Invalid", func(t *testing.T) {
 		resp, err := scheduler.GetChart(context.Background(), &pbSchedulerV2.SchedulerV2GetChartRequest{Name: t3.GetName()})
-		tgrpc.AsGRPCError(t, err).Code(t, codes.Unavailable).Errorf(t, "Chart is not Valid")
+		tgrpc.AsGRPCError(t, err).Code(t, codes.Unavailable).Errorf(t, "Chart is not Ready")
 		require.Nil(t, resp)
 	})
 
 	t.Run("Invalid with message", func(t *testing.T) {
 		resp, err := scheduler.GetChart(context.Background(), &pbSchedulerV2.SchedulerV2GetChartRequest{Name: t4.GetName()})
-		tgrpc.AsGRPCError(t, err).Code(t, codes.Unavailable).Errorf(t, "Chart is not Valid: Invalid XxX")
+		tgrpc.AsGRPCError(t, err).Code(t, codes.Unavailable).Errorf(t, "Chart is not Ready")
 		require.Nil(t, resp)
 	})
 
 	t.Run("Valid with missing details", func(t *testing.T) {
 		resp, err := scheduler.GetChart(context.Background(), &pbSchedulerV2.SchedulerV2GetChartRequest{Name: t5.GetName()})
-		tgrpc.AsGRPCError(t, err).Code(t, codes.Unavailable).Errorf(t, "Chart Details are missing")
+		tgrpc.AsGRPCError(t, err).Code(t, codes.Unavailable).Errorf(t, "Chart is not Ready")
 		require.Nil(t, resp)
 	})
 

integrations/scheduler/v2/definition/chart.pb.go

@@ -64,4 +64,6 @@ message SchedulerV2GetChartResponse {
   bytes chart = 1;
   // Chart Info
   SchedulerV2ChartInfo info = 2;
+  // Chart Overrides
+  bytes overrides = 3;
 }