Fix bug while setting status for deprecated fields

Karthik-K-N · k8s-infra-cherrypick-robot · commit 2b5cadd3b7e5 · 2026-02-17T09:34:30.000Z
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`	`}`