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.

Sign up for GitHub

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

Jump to bottom

s3: update timestamps in Contains() also #625

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

s3: update timestamps in Contains() also #625

wants to merge 1 commit into from

Commits on Jan 4, 2023

  1. s3: update timestamps in Contains() also 

    When the s3.update_timestamps option is enabled, we update the timestamp
of any blob after it is successfully retried from the backend. We should
do the same when checking for blob existence in the Contains() function.

I don't think this is sufficient to make builds-without-the-bytes work
with the s3 proxy backend when eviction is enabled, but it might help.

Relates to buchgr#624.
    @mostynb
    mostynb committed Jan 4, 2023
    Configuration menu
    Copy the full SHA
    2dc254f View commit details
    Browse the repository at this point in the history