From e6fe7eb7d363051fdf01b6177ffbd5eb59bb3eba Mon Sep 17 00:00:00 2001 From: suwliang3 <105624645+suwliang3@users.noreply.github.com> Date: Mon, 19 Dec 2022 12:08:42 +0800 Subject: [PATCH] Feat: rollout support statefulsets (#5202) * rollout support workload Signed-off-by: suwanliang_yewu * modify the error output Signed-off-by: suwanliang_yewu Signed-off-by: suwanliang_yewu --- pkg/controller/standard.oam.dev/v1alpha1/rollout/utils.go | 2 ++ runtime/rollout/charts/templates/vela-runtime-rollout.yaml | 2 ++ 2 files changed, 4 insertions(+) diff --git a/pkg/controller/standard.oam.dev/v1alpha1/rollout/utils.go b/pkg/controller/standard.oam.dev/v1alpha1/rollout/utils.go index 6178b9598..5d5a8e22d 100644 --- a/pkg/controller/standard.oam.dev/v1alpha1/rollout/utils.go +++ b/pkg/controller/standard.oam.dev/v1alpha1/rollout/utils.go @@ -114,6 +114,7 @@ func HandleReplicas(ctx context.Context, rolloutComp string, c client.Client) as klog.InfoS("assemble force set workload replicas to 0", "Kind", u.GetKind(), "name", u.GetName()) return nil } + klog.Errorf("fail to get workload %s: %v", u.GetName(), err) return err } // the workload already exist, we cannot reset the replicas with manifest @@ -122,6 +123,7 @@ func HandleReplicas(ctx context.Context, rolloutComp string, c client.Client) as wlpv := fieldpath.Pave(workload.UnstructuredContent()) replicas, err := wlpv.GetInteger(replicasFieldPath) if err != nil { + klog.Errorf("fail to get `spec.replicas` field from workload %s: %v", u.GetName(), err) return err } if err = pv.SetNumber(replicasFieldPath, float64(replicas)); err != nil { diff --git a/runtime/rollout/charts/templates/vela-runtime-rollout.yaml b/runtime/rollout/charts/templates/vela-runtime-rollout.yaml index 1fb0fa2f2..0999c8cbf 100644 --- a/runtime/rollout/charts/templates/vela-runtime-rollout.yaml +++ b/runtime/rollout/charts/templates/vela-runtime-rollout.yaml @@ -49,6 +49,8 @@ rules: - apiGroups: - "apps" resources: + - statefulsets + - statefulsets/status - deployments - deployments/status - controllerrevisions