Merge pull request #13345 from k8s-infra-cherrypick-robot/cherry-pick-13336-to-release-1.11

k8s-ci-robot · web-flow · commit 13c66ae686a3 · 2026-02-17T16:11:05.000+05:30
[release-1.11] 🐛 Fix bug while setting status for deprecated fields
diff --git a/util/deprecated/v1beta1/patch/utils.go b/util/deprecated/v1beta1/patch/utils.go
@@ -187,7 +187,7 @@ func identifyConditionsFieldsPath(obj runtime.Object) ([]string, []string, error
 			// We assume the type is implemented according to transition guidelines
 			clusterv1ConditionsFields = []string{"status", "deprecated", "v1beta1", "conditions"}
 		} else {
-			if v1Beta1Field := deprecatedElem.FieldByName("V1Beta1"); deprecatedField != (reflect.Value{}) {
+			if v1Beta1Field := deprecatedElem.FieldByName("V1Beta1"); v1Beta1Field != (reflect.Value{}) {
 				if v1Beta1Field.Kind() != reflect.Pointer {
 					return nil, nil, errors.New("obj.status.deprecated.v1beta1 must be a pointer")
 				}
diff --git a/util/patch/utils.go b/util/patch/utils.go
@@ -187,7 +187,7 @@ func identifyConditionsFieldsPath(obj runtime.Object) ([]string, []string, error
 			// We assume the type is implemented according to transition guidelines
 			clusterv1ConditionsFields = []string{"status", "deprecated", "v1beta1", "conditions"}
 		} else {
-			if v1Beta1Field := deprecatedElem.FieldByName("V1Beta1"); deprecatedField != (reflect.Value{}) {
+			if v1Beta1Field := deprecatedElem.FieldByName("V1Beta1"); v1Beta1Field != (reflect.Value{}) {
 				if v1Beta1Field.Kind() != reflect.Pointer {
 					return nil, nil, errors.New("obj.status.deprecated.v1beta1 must be a pointer")
 				}

Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,7 @@ func identifyConditionsFieldsPath(obj runtime.Object) ([]string, []string, error`
`187`	`187`	`// We assume the type is implemented according to transition guidelines`
`188`	`188`	`clusterv1ConditionsFields = []string{"status", "deprecated", "v1beta1", "conditions"}`
`189`	`189`	`} else {`
`190`		`- if v1Beta1Field := deprecatedElem.FieldByName("V1Beta1"); deprecatedField != (reflect.Value{}) {`
	`190`	`+ if v1Beta1Field := deprecatedElem.FieldByName("V1Beta1"); v1Beta1Field != (reflect.Value{}) {`
`191`	`191`	`if v1Beta1Field.Kind() != reflect.Pointer {`
`192`	`192`	`return nil, nil, errors.New("obj.status.deprecated.v1beta1 must be a pointer")`
`193`	`193`	`}`