Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Invalidate discovery cache periodically #1859

Merged

Conversation

2opremio
Copy link
Contributor

The cluster resource definitions may not be stable throughout the life of the cluster.

We were already invalidating the cache every time a CRD object was altered and replaying the CRD changes every 5 minutes.

However, that's not enough because CRD-unrelated changes seem to happen when the cluster bootstraps.

Fixes #1855

@2opremio 2opremio requested a review from squaremo March 25, 2019 17:42
The cluster resource definitions may not be stable throughout the life of the
cluster.

We were already invalidating the cache every time a CRD object was altered and
replaying the CRD changes every 5 minutes.

However, that's not enough because CRD-unrelated changes seem to happen when the
cluster bootstraps.
@2opremio 2opremio force-pushed the 1855-reset-discovery-cache-periodically branch from c2f21ef to 33abf1d Compare March 25, 2019 17:43
@2opremio 2opremio requested a review from squaremo March 26, 2019 17:59
Copy link
Member

@squaremo squaremo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable to me! Thanks Fons

@2opremio 2opremio merged commit 949139b into fluxcd:master Mar 27, 2019
@2opremio 2opremio deleted the 1855-reset-discovery-cache-periodically branch March 27, 2019 10:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants