search

csi-addons / volume-replication-operator

Apache License 2.0
16 stars 23 forks source link

replication: set status.State to secondary when VR is marked for secondary #116

Closed Madhu-1 closed 3 years ago

Madhu-1 commented 3 years ago

when the VR is marked as secondary the current instance object still contains the status.State as primary, not secondary. we need to set the status.State to secondary when the volume is demoted successfully.

Signed-off-by: Madhu Rajanna madhupr007@gmail.com

Madhu-1 commented 3 years ago

@oriyarde PTAL

Madhu-1 commented 3 years ago

Looks like mergify merged the PR due to a lack of check in mergify configurations.

Madhu-1 commented 3 years ago

https://github.com/csi-addons/volume-replication-operator/pull/117 should avoid this case.