Skip to content

Conversation

kaovilai
Copy link
Member

No description provided.

@kaovilai kaovilai requested a review from hhpatel14 December 13, 2021 15:10
@codecov-commenter
Copy link

Codecov Report

Merging #510 (aa28cab) into master (59c3a33) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #510   +/-   ##
=======================================
  Coverage   38.42%   38.42%           
=======================================
  Files          12       12           
  Lines        2303     2303           
=======================================
  Hits          885      885           
  Misses       1350     1350           
  Partials       68       68           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update af177d1...aa28cab. Read the comment docs.

@openshift-ci openshift-ci bot requested review from jmontleon and sseago December 13, 2021 15:17
@openshift-ci
Copy link

openshift-ci bot commented Dec 13, 2021

@kaovilai: all tests passed!

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@hhpatel14
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 13, 2021
@hhpatel14 hhpatel14 merged commit 2a595c5 into openshift:master Dec 13, 2021
kaovilai added a commit to kaovilai/oadp-operator that referenced this pull request Dec 22, 2021
* api v1 upgrade CR typo (openshift#501)

* Removing kube-rbac-proxy from the containers needed (openshift#504)

* fix indentation (openshift#505)

* turn OCP versions from int to string (openshift#502)

* Ensure velero is deleted (openshift#510)

* Timeout  for must-gather failed backups (openshift#497)

* Adding known limitations to must-gather

* Adding implementation details for timeout

* remove VSPHERE env configs for csv (openshift#512)

* update troubleshooting doc (openshift#509)

* Adding support for service principal credential for Azure plugin (openshift#507)

* Adding validation for azure creds

* Adding account key check

* Adding checks to validate SP if storage key is not present

* Adding unit test#1

* Adding a test case for service principal

* Adding review comments

* Remove: logging sensitive info

* make deploy velero namespace fix (openshift#506)

* make deploy velero namespace fix

* add changes for undeploy

* add deploy-tmp-cleanup

* fix aws registry env vars (openshift#515)

* Azure SP docs (openshift#514)

* Adding Azure SP related doc

* Fixing title

* Fixing nit

* Registry should not be deployed when Azure SP is used (openshift#518)

* Registry should not be deployed when Azure SP is used

* Fixing unit tests

* Adding review comments

* Fixing typos

* Adding registry label to BSL

* Updating azure credentials documentation (openshift#519)

* AWS plugin config: BSL Region not required when s3ForcePathStyle is false and BackupImages is false (openshift#517)

* OADP-153, Close openshift#424

* Nil restic Config should delete previous restic daemonset

* only check restic config if it is not nil

* installCase wantError implement

* Make err more verbose

* commit metav1

* Changes for BackupImages considerations

* fake client fix

* removed vsphere from source manager config (openshift#520)

* Update README.md

* badge relocate (openshift#521)

Co-authored-by: Tiger Kaovilai <[email protected]>
Co-authored-by: Shawn Hurley <[email protected]>
Co-authored-by: Emily McMullan <[email protected]>
Co-authored-by: Savitha Raghunathan <[email protected]>
Co-authored-by: Wesley Hayutin <[email protected]>
Co-authored-by: Dylan Murray <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants