Skip to content

Conversation

@oharan2
Copy link
Contributor

@oharan2 oharan2 commented Aug 3, 2023

Short description:
More details:
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
Bug:

@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 re-run tox comment /retest tox in the PR.
  • To re-run build-container command /retest build-container in the PR.
  • To re-run python-module-install command /retest python-module-install in the PR.
  • 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 labels
  • hold
  • verified
  • wip
  • lgtm

@myakove
Copy link
Collaborator

myakove commented Aug 5, 2023

/retest sonarqube

@oharan2 oharan2 changed the title WIP: Update Backup and ClusterRole constructors and to_dict function Update Backup and ClusterRole constructors and to_dict function Aug 7, 2023
myakove
myakove previously approved these changes Aug 10, 2023
rnetser
rnetser previously approved these changes Aug 10, 2023
myakove
myakove previously approved these changes Aug 10, 2023
@oharan2
Copy link
Contributor Author

oharan2 commented Aug 10, 2023

/verified

@myakove
Copy link
Collaborator

myakove commented Aug 13, 2023

/cherry-pick v4.14

redhat-qe-bot pushed a commit that referenced this pull request Aug 13, 2023
* Update Backup and ClusterRole constuctors and to_dict function

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

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

* Update backup docs

* Update docs

* Update documentations

* Remove if not self.yaml_file

* Remove upper super.to_dict()

* Remove and not self.yaml_file

* Update documentaions

* Use res.update with excluded resources

* Update docs

* Remove update() usage to add excluded_resources to spec. update() overrides spec content.

* Verified

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@redhat-qe-bot
Copy link
Contributor

Cherry-picked PR Update Backup and ClusterRole constructors and to_dict function into v4.14

redhat-qe-bot added a commit that referenced this pull request Aug 13, 2023
… (#1355)

* Update Backup and ClusterRole constuctors and to_dict function

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

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

* Update backup docs

* Update docs

* Update documentations

* Remove if not self.yaml_file

* Remove upper super.to_dict()

* Remove and not self.yaml_file

* Update documentaions

* Use res.update with excluded resources

* Update docs

* Remove update() usage to add excluded_resources to spec. update() overrides spec content.

* Verified

---------

Co-authored-by: oharan2 <oharan@redhat.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
myakove pushed a commit that referenced this pull request Aug 11, 2025
* Update Backup and ClusterRole constuctors and to_dict function

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

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

* Update backup docs

* Update docs

* Update documentations

* Remove if not self.yaml_file

* Remove upper super.to_dict()

* Remove and not self.yaml_file

* Update documentaions

* Use res.update with excluded resources

* Update docs

* Remove update() usage to add excluded_resources to spec. update() overrides spec content.

* Verified

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@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.

5 participants