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

fix(kubernetes): Core caching agent is authoritative for artifacts #4247

Merged
merged 1 commit into from
Jan 9, 2020
Merged

fix(kubernetes): Core caching agent is authoritative for artifacts #4247

merged 1 commit into from
Jan 9, 2020

Conversation

ezimanyi
Copy link
Contributor

@ezimanyi ezimanyi commented Jan 9, 2020

Fixes spinnaker/spinnaker#5274.

There is currently no caching agent that is authoritative for kubernetes artifacts; this is causing them to fail to be cached by the SQL provider (which strictly enforces that only caching agents can only write types for which they are authoritative).

The KubernetesCoreCachingAgent should be authoritative for artifacts, just as it is for other logical kinds (clusters and applications).

There is currently no caching agent that is authoritative for kubernetes
artifacts; this is causing them to fail to be cached by the SQL provider
(which strictly enforces that only caching agents can only write types
for which they are authoritative).

The KubernetesCoreCachingAgent should be authoritative for artifacts,
just as it is for other logical kinds (clusters and applications).
Copy link
Contributor

@ethanfrogers ethanfrogers left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@maggieneterval maggieneterval left a comment

Choose a reason for hiding this comment

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

👥 ❓ ➡️ 👨‍🏫 🏺

@ezimanyi ezimanyi added the ready to merge Approved and ready for a merge label Jan 9, 2020
@mergify mergify bot merged commit 008e111 into spinnaker:master Jan 9, 2020
@mergify mergify bot added the auto merged Merged automatically by a bot label Jan 9, 2020
@ezimanyi ezimanyi deleted the fix-artifact-caching branch January 10, 2020 00:45
@jasonmcintosh
Copy link
Member

Hrmm, can this be cherry picked to 1.17?

@alchen99
Copy link

@spinnakerbot cherry-pick 1.17

spinnakerbot pushed a commit that referenced this pull request Jan 10, 2020
…4247)

There is currently no caching agent that is authoritative for kubernetes
artifacts; this is causing them to fail to be cached by the SQL provider
(which strictly enforces that only caching agents can only write types
for which they are authoritative).

The KubernetesCoreCachingAgent should be authoritative for artifacts,
just as it is for other logical kinds (clusters and applications).
@spinnakerbot
Copy link
Contributor

Cherry pick successful: #4248

duftler pushed a commit that referenced this pull request Jan 13, 2020
…4247) (#4248)

There is currently no caching agent that is authoritative for kubernetes
artifacts; this is causing them to fail to be cached by the SQL provider
(which strictly enforces that only caching agents can only write types
for which they are authoritative).

The KubernetesCoreCachingAgent should be authoritative for artifacts,
just as it is for other logical kinds (clusters and applications).

Co-authored-by: Eric Zimanyi <ezimanyi@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged Merged automatically by a bot ready to merge Approved and ready for a merge target-release/1.18
Projects
None yet
6 participants