openebs-archive / maya

Manage Container Attached Storage (CAS) - Data Engines in Kubernetes
https://docs.openebs.io
Apache License 2.0
184 stars 201 forks source link

fix(restore): set targetip on CVRs after restore is completed #1761

Closed zlymeda closed 4 years ago

zlymeda commented 4 years ago

Auto-setting of targetip on CVR if openebs.io/restore-completed is found

Pull Request template

Why is this PR required? What issue does it fix?: Currently, users have to set targetip on CVRs after restore. Corresponding change in https://github.com/openebs/velero-plugin/pull/131 will mark CVRs which needs to update the targetip. Changes in this PR will check the annotation and set the targetip.

What this PR does?: Sets targetip on CVRs marked with openebs.io/restore-completed annotation.

Does this PR require any upgrade changes?: No

If the changes in this PR are manually verified, list down the scenarios covered:: make test in velero-plugin repository with changes deployed.

Any additional information for your reviewer? : Mention if this PR is part of any design or a continuation of previous PRs

related velero-plugin PR: https://github.com/openebs/velero-plugin/pull/131 related cstor-operators PR: https://github.com/openebs/cstor-operators/pull/198

Checklist:

zlymeda commented 4 years ago

I rebased with origin/master and squashed commits into single commit