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