Skip to content

No tmp_dir for fileblob persistence#277

Merged
haritzsaiz merged 1 commit intomainfrom
fix/fileblob-notmpdir
Jun 27, 2025
Merged

No tmp_dir for fileblob persistence#277
haritzsaiz merged 1 commit intomainfrom
fix/fileblob-notmpdir

Conversation

@haritzsaiz
Copy link
Collaborator

This pull request introduces a small but important change to the Register function in engines/fs-storage/localfs/registrar.go. The change modifies the uri string to include a no_tmp_dir query parameter, which impacts how temporary directories are handled during bucket initialization: https://pkg.go.dev/gocloud.dev/blob/fileblob

Signed-off-by: Haritz S. Sierra <[email protected]>
@sonarqubecloud
Copy link

@codecov
Copy link

codecov bot commented Jun 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.82%. Comparing base (c1b44ac) to head (1b78f34).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #277   +/-   ##
=======================================
  Coverage   45.82%   45.82%           
=======================================
  Files         128      128           
  Lines        8570     8570           
=======================================
  Hits         3927     3927           
  Misses       4307     4307           
  Partials      336      336           
Flag Coverage Δ
connector-awsiot 4.91% <ø> (ø)
core 61.97% <ø> (ø)
monolithic 0.00% <ø> (ø)
sdk 0.00% <ø> (ø)
shared-http 7.07% <ø> (ø)
storage-postgres 57.67% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@haritzsaiz haritzsaiz merged commit 7c4cb0e into main Jun 27, 2025
37 checks passed
@haritzsaiz haritzsaiz deleted the fix/fileblob-notmpdir branch June 27, 2025 08:52
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.

2 participants