Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modify BackupStoreGetter to avoid BSL spec changes #5134

Merged
merged 1 commit into from
Jul 20, 2022

Conversation

sseago
Copy link
Collaborator

@sseago sseago commented Jul 19, 2022

(cherry-picked from #5122 to release-1.9)

Pass in a new copy of the map of config values rather than
modifying the BSL Spec.Config and then pass in that field.

Signed-off-by: Scott Seago [email protected]

Thank you for contributing to Velero!

Please add a summary of your change

Does your change fix a particular issue?

Fixes #(issue)

Please indicate you've done the following:

  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Created a changelog file or added /kind changelog-not-required as a comment on this pull request.
  • Updated the corresponding documentation in site/content/docs/main.

Pass in a new copy of the map of config values rather than
modifying the BSL Spec.Config and then pass in that field.

Signed-off-by: Scott Seago <[email protected]>
@sseago sseago force-pushed the backupstoregettermap branch from bd31f23 to b5c14d9 Compare July 19, 2022 15:07
@codecov-commenter
Copy link

codecov-commenter commented Jul 19, 2022

Codecov Report

Merging #5134 (b5c14d9) into release-1.9 (a6fb4bb) will decrease coverage by 0.01%.
The diff coverage is 50.00%.

@@               Coverage Diff               @@
##           release-1.9    #5134      +/-   ##
===============================================
- Coverage        41.34%   41.33%   -0.02%     
===============================================
  Files              211      211              
  Lines            18438    18440       +2     
===============================================
- Hits              7624     7622       -2     
- Misses           10242    10245       +3     
- Partials           572      573       +1     
Impacted Files Coverage Δ
pkg/persistence/object_store.go 56.85% <50.00%> (-0.92%) ⬇️

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 a6fb4bb...b5c14d9. Read the comment docs.

Copy link
Contributor

@blackpiglet blackpiglet left a comment

Choose a reason for hiding this comment

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

Checked with the code calling backupStoreGetter.Get
There is no code relies on the BSL modification after this method.

@sseago sseago merged commit e76b697 into vmware-tanzu:release-1.9 Jul 20, 2022
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