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 recommened roles for Elastic Agent on Kubernetes #8188

Merged
merged 5 commits into from
Nov 22, 2024

Conversation

pebrc
Copy link
Collaborator

@pebrc pebrc commented Nov 11, 2024

Fixes #8168

I tried to amend the roles based on https://raw.githubusercontent.com/elastic/elastic-agent/v8.15.3/deploy/kubernetes/elastic-agent-standalone-kubernetes.yaml

I left out the cloudbeat recommendations, because I think we cannot cater for all integrations in the defaults.
But we can also go for a the maximal set of permissions, to avoid problems and allow scurity conscious users to trim them down to what is necessary for them.

It is problematic that we need to update ~ 4 places and effectively duplicate the roles that the Elastic Agent team defines without having a good process to update them when the source of truth changes.

@botelastic botelastic bot added the triage label Nov 11, 2024
@pebrc pebrc force-pushed the 8168-missing-roles branch from d748b58 to 5031542 Compare November 11, 2024 15:40
Copy link
Contributor

@barkbay barkbay left a comment

Choose a reason for hiding this comment

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

Indentation seems wrong in one file, LGTM otherwise.

deploy/eck-stack/charts/eck-agent/values.yaml Outdated Show resolved Hide resolved
deploy/eck-stack/charts/eck-agent/values.yaml Outdated Show resolved Hide resolved
@pebrc pebrc added >bug Something isn't working v2.16.0 labels Nov 13, 2024
@botelastic botelastic bot removed the triage label Nov 13, 2024
@pebrc
Copy link
Collaborator Author

pebrc commented Nov 13, 2024

buildkite test this -f p=gke,s=8.16.0,E2E_TAGS=agent

@pebrc
Copy link
Collaborator Author

pebrc commented Nov 14, 2024

buildkite test this -f p=gke,s=8.16.0,E2E_TAGS=agent

@pebrc
Copy link
Collaborator Author

pebrc commented Nov 21, 2024

buildkite test this -f p=gke,s=8.16.0,E2E_TAGS=agent

@pebrc pebrc merged commit 8cbc0ee into elastic:main Nov 22, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug Something isn't working v2.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Elastic Agent Kubernetes Integration missing default permissions
2 participants