Closed vdkotkar closed 1 week ago
Hi @vdkotkar. Thanks for your PR.
I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
/approve
/ok-to-test
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: deepakkinni, vdkotkar, xing-yang
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/lgtm
What this PR does / why we need it: As part of upgrade from 8.0u3 or earlier versions to 9.0, we need to re-evaluate storage quota calculations. We might end up in mixed host scenarios where few hosts are of version 8.0u3 or prior and few hosts are of 9.0. We need to calculate AggregateSnapshotSize of a volume considering these versions. This PR has changes to watch on AvailabilityZone CRs in supervisor cluster. Will add further changes to watch on host events etc. in cluster belonging to AvailabilityZone in separate PR.
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged): fixes #Testing done: Verified that we are receiving events for AvailaibilityZone CRs in syncer container:
Special notes for your reviewer:
Release note: