Bug 2257480: Reconcile related drpcs when drcluster is deleted #173
Conversation
|
@nirs: No Bugzilla bug is referenced in the title of this pull request. In response to this:
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. |
- When filtering update events, consider also the update when the new object is marked for deletion. - When filtering drpcs consider all drpcs referencing a deleted drcluster. With this change, when a drcluster is deleted, the drpc controller updates the VRG ManifestWork on the remaining cluster. Previously this happened minutes after a drcluster was deleted. Notes: - We don't get a delete event when the drcluster is deleted, but an update event. I don't know if this is a bug in controller runtime or expected behavior. Signed-off-by: Nir Soffer <[email protected]> (cherry picked from commit 2b66f98) Signed-off-by: Nir Soffer <[email protected]>
nirs
force-pushed
the
update-vrg-release-4.15
branch
from
d01cfc1 to
d107f69
Compare
nirs
changed the title
|
/bugzilla refresh |
|
@ShyamsundarR: Bugzilla bug 2257480 is in a bug group that is not in the allowed groups for this repo.
In response to this:
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. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: nirs, ShyamsundarR The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
ShyamsundarR
merged commit 397676e
into
red-hat-storage:release-4.15
With this change, when a drcluster is deleted, the drpc controller updates the VRG ManifestWork on the remaining cluster. Previously this happened minutes after a drcluster was deleted.
Notes:
Signed-off-by: Nir Soffer [email protected]
(cherry picked from commit 2b66f98)
Change tested upstream with drenv and ocp.