Skip to content

Conversation

shubham-pampattiwar
Copy link
Member

This PR creates a registry secret per BSL and then updates the registry secret with the cloud provider (aws and azure) secret data, this sensitive data is then consumed by the registry deployment's env vars needed for the registry storage drivers.

@codecov-commenter
Copy link

Codecov Report

Merging #535 (7135e87) into master (978550f) will decrease coverage by 1.04%.
The diff coverage is 41.24%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #535      +/-   ##
==========================================
- Coverage   39.78%   38.73%   -1.05%     
==========================================
  Files          13       13              
  Lines        2569     2695     +126     
==========================================
+ Hits         1022     1044      +22     
- Misses       1462     1570     +108     
+ Partials       85       81       -4     
Impacted Files Coverage Δ
controllers/dpa_controller.go 0.00% <0.00%> (ø)
controllers/registry.go 40.41% <41.47%> (-3.88%) ⬇️

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 978550f...7135e87. Read the comment docs.

Copy link
Member

@dymurray dymurray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Tested with AWS config and everything worked as expected

@openshift-ci
Copy link

openshift-ci bot commented Jan 13, 2022

@shubham-pampattiwar: 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.

@dymurray dymurray merged commit d06ac1a into openshift:master Jan 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants