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

Use cluster role for the SDK deployment when running in EC #4952

Merged

Conversation

divolgin
Copy link
Member

@divolgin divolgin commented Oct 15, 2024

What this PR does / why we need it:

In EC installs, Replicated SDK is not being granted cluster level access, which limits its functionality resulting in bad data being reported.

Relevant Replicated SDK PR: replicatedhq/replicated-sdk#218

Which issue(s) this PR fixes:

https://app.shortcut.com/replicated/story/114128/replicated-sdk-reports-k0s-when-installed-in-embedded-cluster-causing-distribution-to-flip-flop-in-events-view

Does this PR require a test?

Does this PR require a release note?

When included, Replicated SDK will be deployed with the same ClusterRole as the Admin Console.

Does this PR require documentation?

@divolgin divolgin added type::bug Something isn't working bug::normal labels Oct 15, 2024
@divolgin divolgin force-pushed the divolgin/sc-114128/replicated-sdk-reports-k0s-when-installed branch from 643f35d to cce70fd Compare October 15, 2024 21:24
pkg/upstream/helm.go Outdated Show resolved Hide resolved
@divolgin divolgin force-pushed the divolgin/sc-114128/replicated-sdk-reports-k0s-when-installed branch from cce70fd to 5f9ee0b Compare October 15, 2024 21:33
@divolgin divolgin force-pushed the divolgin/sc-114128/replicated-sdk-reports-k0s-when-installed branch from 5f9ee0b to 30446b3 Compare October 16, 2024 20:50
@divolgin divolgin merged commit 61e20dd into main Oct 17, 2024
122 checks passed
@divolgin divolgin deleted the divolgin/sc-114128/replicated-sdk-reports-k0s-when-installed branch October 17, 2024 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug::normal type::bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants