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

Add cache time for / index request #631

Merged
merged 2 commits into from
Sep 7, 2020

Conversation

ruflin
Copy link
Member

@ruflin ruflin commented Sep 2, 2020

The current request to / was cached the the as all the static artifacts. But this info changes every time the registry is deployed with a new version. To have the new version available as soon as possible and not potentially cached for a long time, an additional configuration cache option for this endpoint was introduced.

The current request to / was cached the the as all the static artifacts. But this info changes every time the registry is deployed with a new version. To have the new version available as soon as possible and not potentially cached for a long time, an additional configuration cache option for this endpoint was introduced.
@ruflin ruflin requested a review from mtojek September 2, 2020 07:18
@ruflin ruflin self-assigned this Sep 2, 2020
@elasticmachine
Copy link

elasticmachine commented Sep 2, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Branch indexing]

  • Start Time: 2020-09-06T16:09:06.436+0000

  • Duration: 8 min 1 sec

Test stats 🧪

Test Results
Failed 0
Passed 161
Skipped 0
Total 161

@ruflin ruflin merged commit f5ca9a0 into elastic:master Sep 7, 2020
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.

3 participants