Re: [kubernetes/kubernetes] StatefulSet: update forbidden because of PVC storage units (#66137)

[k8s-ci-robot]

@derekperkins: Reiterating the mentions to trigger a notification:
@kubernetes/sig-storage-bugs

In response to this:

@kubernetes/sig-storage-bugs

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

[Michelle Au]

cc @k8s-mirror-api-machinery-bugs

This is probably an issue for any resource.Quantity type.

[krmayankk]

/sig api-machinery

[Jordan Liggitt]

the issue is use of !reflect.DeepEqual in statefulset update validation, instead of apiequality.Semantic.DeepEqual, which explicitly compares quantity objects semantically:

https://github.com/kubernetes/kubernetes/blob/c174e4121fa3cacaf9c2f762446517a03997411e/staging/src/k8s.io/apimachinery/pkg/api/equality/semantic.go#L29-L36

[Jordan Liggitt]

/sig apps

[Jordan Liggitt]

fix is in #66165 with picks open to 1.11, 1.10, 1.9

[Kubernetes Submit Queue]

Closed #66137 via #66165.