Skip to content

Fix ObjectExists method to properly handle wrapped storage.ErrObjectN…#239

Merged
kaovilai merged 1 commit intovmware-tanzu:mainfrom
PrasadJoshi12:fix-backup-issue
Sep 18, 2025
Merged

Fix ObjectExists method to properly handle wrapped storage.ErrObjectN…#239
kaovilai merged 1 commit intovmware-tanzu:mainfrom
PrasadJoshi12:fix-backup-issue

Conversation

@PrasadJoshi12
Copy link
Contributor

@PrasadJoshi12 PrasadJoshi12 commented Sep 17, 2025

Fix issue: #9268

kaovilai
kaovilai previously approved these changes Sep 17, 2025
…otExist errors from Google Cloud Storage API

Signed-off-by: Prasad Joshi <prajoshi@redhat.com>
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 28.39%. Comparing base (ee22e39) to head (b231a97).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #239   +/-   ##
=======================================
  Coverage   28.39%   28.39%           
=======================================
  Files           4        4           
  Lines         574      574           
=======================================
  Hits          163      163           
  Misses        399      399           
  Partials       12       12           

☔ 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.

@kaovilai kaovilai merged commit 15d4b84 into vmware-tanzu:main Sep 18, 2025
2 checks passed
@weshayutin
Copy link

@kaovilai @shubham-pampattiwar cherry-pick to 1.17?

@shubham-pampattiwar
Copy link
Collaborator

@kaovilai @shubham-pampattiwar cherry-pick to 1.17?

Already merged on 1.13 (plugin release for 1.17) #240

@polarn
Copy link

polarn commented Oct 20, 2025

Any reason for not making a 1.13.1 release to include this?

@kaovilai
Copy link
Collaborator

vmware-tanzu/velero#9335 (comment)
will include. just wait.

@Nexus2k
Copy link

Nexus2k commented Oct 29, 2025

FYI this is currently NOT in 1.13.0 and breaks all setups using GCP service-accounts for authentication to the GCS buckets.

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.

7 participants