Skip to content

Conversation

@hu-keyu
Copy link
Contributor

@hu-keyu hu-keyu commented Apr 22, 2025

Thank you for contributing to Velero!

Please add a summary of your change

Pass the logger in kopia related operations.

Does your change fix a particular issue?

Fixes #8827

Please indicate you've done the following:

hu-keyu and others added 4 commits April 22, 2025 15:32
@codecov
Copy link

codecov bot commented Apr 22, 2025

Codecov Report

Attention: Patch coverage is 63.63636% with 8 lines in your changes missing coverage. Please review.

Project coverage is 59.56%. Comparing base (0dbff6d) to head (3f5c771).
Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
pkg/repository/udmrepo/kopialib/backend/azure.go 50.00% 2 Missing ⚠️
pkg/repository/udmrepo/kopialib/lib_repo.go 0.00% 2 Missing ⚠️
...po/kopialib/backend/azure/azure_storage_wrapper.go 50.00% 1 Missing ⚠️
...repository/udmrepo/kopialib/backend/file_system.go 50.00% 1 Missing ⚠️
pkg/repository/udmrepo/kopialib/backend/gcs.go 50.00% 1 Missing ⚠️
pkg/repository/udmrepo/kopialib/backend/s3.go 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8875      +/-   ##
==========================================
- Coverage   59.56%   59.56%   -0.01%     
==========================================
  Files         370      370              
  Lines       40246    40248       +2     
==========================================
+ Hits        23973    23974       +1     
- Misses      14773    14774       +1     
  Partials     1500     1500              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ywk253100 ywk253100 merged commit e06b62e into vmware-tanzu:main Apr 23, 2025
42 checks passed
Lyndon-Li pushed a commit to Lyndon-Li/velero that referenced this pull request Apr 28, 2025
* Pass logger in call chain

Signed-off-by: hu-keyu <[email protected]>
@MaloLelandais MaloLelandais mentioned this pull request Jul 28, 2025
3 tasks
MaloLelandais pushed a commit to MaloLelandais/velero that referenced this pull request Jul 28, 2025
* Pass logger in call chain

Signed-off-by: hu-keyu <[email protected]>
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.

Log output during BackupRepository creation of Azure repository is in plaintext with --log-format=json

3 participants