Skip to content

clean stale entries from the pathCache when removing packages from S3 storage - #1224

Closed
sf-nwaller wants to merge 4 commits into
aptly-dev:masterfrom
sf-nwaller:clean-s3-pathCache
Closed

clean stale entries from the pathCache when removing packages from S3 storage#1224
sf-nwaller wants to merge 4 commits into
aptly-dev:masterfrom
sf-nwaller:clean-s3-pathCache

Conversation

@sf-nwaller

@sf-nwaller sf-nwaller commented Sep 29, 2023

Copy link
Copy Markdown
Contributor

Fixes issue #1223 "file already exists and is different" after un-publishing package from S3

Description of the Change

Properly invalidate the pathCache in the S3 publisher when unpublishing packages and Aptly is running in api serve mode.

This prevents Aptly from getting stuck in a broken state where publishing to S3 no longer works.

Checklist

  • successfully completed end-to-end integration test
  • documentation updated
  • author name in AUTHORS

@codecov

codecov Bot commented Sep 29, 2023

Copy link
Copy Markdown

Codecov Report

Merging #1224 (6822ace) into master (322e5c1) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1224   +/-   ##
=======================================
  Coverage   65.96%   65.97%           
=======================================
  Files         143      143           
  Lines       16188    16189    +1     
=======================================
+ Hits        10679    10680    +1     
  Misses       4756     4756           
  Partials      753      753           
Files Coverage Δ
s3/public.go 57.63% <100.00%> (+0.12%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@neolynx
neolynx self-requested a review January 2, 2024 18:21
@sf-nwaller

Copy link
Copy Markdown
Contributor Author

Superseded by #1241

@sf-nwaller sf-nwaller closed this Feb 6, 2024
@neolynx
neolynx removed their request for review April 24, 2024 19:33
@neolynx neolynx mentioned this pull request Jun 19, 2026
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.

1 participant