Skip to content

Conversation

@dalia-frank
Copy link
Contributor

@dalia-frank dalia-frank commented Dec 11, 2023

Short description:

Add snapshot_move_data and storage_location fields to backup

More details:

https://docs.openshift.com/container-platform/4.14/backup_and_restore/application_backup_and_restore/installing/oadp-backup-restore-csi-snapshots.html

What this PR does / why we need it:

Add the capability to make data mover backups

@redhat-qe-bot
Copy link
Contributor

Report bugs in Issues

The following are automatically added:

  • Add reviewers from OWNER file (in the root of the repository) under reviewers section.
  • Set PR size label.
  • New issue is created for the PR. (Closed when PR is merged/closed)
  • Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

  • To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
  • To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
  • To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
    verified label removed on each new commit push.
  • To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.
    • Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
    • Cherry-pick will be started when PR is merged
  • To build and push container image command /build-and-push-container in the PR (tag will be the PR number).
  • To add a label by comment use /<label name>, to remove, use /<label name> cancel
Supported /retest check runs
  • /retest tox: Retest tox
  • /retest sonarqube: Retest sonarqube
  • /retest python-module-install: Retest python-module-install
Supported labels
  • hold
  • verified
  • wip
  • lgtm

@dalia-frank
Copy link
Contributor Author

/verified

myakove
myakove previously approved these changes Dec 19, 2023
@redhat-qe-bot redhat-qe-bot requested a review from rnetser January 1, 2024 09:46
@dalia-frank
Copy link
Contributor Author

/verified

@myakove
Copy link
Collaborator

myakove commented Jan 4, 2024

/lgtm

@myakove
Copy link
Collaborator

myakove commented Jan 4, 2024

/verified

@dalia-frank
Copy link
Contributor Author

/cherry-pick v4.15

redhat-qe-bot pushed a commit that referenced this pull request Jan 17, 2024
* Add data mover params to backup

* add new args to docstring

* Fix

* add backup doc

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* remove duplication

---------

Co-authored-by: Dalia Frank <dafrank@dafrank-thinkpadx1nanogen2.remote.csb>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Meni Yakove <441263+myakove@users.noreply.github.com>
@redhat-qe-bot
Copy link
Contributor

Cherry-picked PR Add data mover params to backup into v4.15

redhat-qe-bot added a commit that referenced this pull request Jan 17, 2024
* Add data mover params to backup

* add new args to docstring

* Fix

* add backup doc

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* remove duplication

---------

Co-authored-by: dalia-frank <65129017+dalia-frank@users.noreply.github.com>
Co-authored-by: Dalia Frank <dafrank@dafrank-thinkpadx1nanogen2.remote.csb>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Meni Yakove <441263+myakove@users.noreply.github.com>
myakove added a commit that referenced this pull request Aug 11, 2025
* Add data mover params to backup

* add new args to docstring

* Fix

* add backup doc

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* remove duplication

---------

Co-authored-by: Dalia Frank <dafrank@dafrank-thinkpadx1nanogen2.remote.csb>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Meni Yakove <441263+myakove@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants