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

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

Closed
wants to merge 4 commits into from

Conversation

sf-nwaller
Copy link
Contributor

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

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
Copy link

codecov bot commented Sep 29, 2023

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