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

[8.x](backport #5593) [helm] support using user-created serviceAccount and clusterRole #5977

Merged
merged 2 commits into from
Nov 8, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Nov 8, 2024

What does this PR do?

This PR introduces the capability for users to bind agent presets with custom service accounts and cluster roles, rather than relying solely on auto-generated or default configurations. This allows greater control over security and permissions for agent interactions. Additionally, it provides support for adding annotations to these bindings, allowing for custom metadata that can be leveraged by observability and monitoring tools to enhance tracking, auditing, and configuration management.

Why is it important?

This added flexibility ensures that the Helm chart can be better aligned with organizational policies and infrastructure requirements of users.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in ./changelog/fragments using the changelog tool
  • I have added an integration test or an E2E test

Disruptive User Impact

N/A

How to test this PR locally

mage helm:renderExamples
mage integration:kubernetesMatrix

Related issues

* feat: support using user-created serviceAccount and clusterRole

* feat: add support for defining automountServiceAccountToken per preset

* fix: add preset-wide annotations to serviceaccount, clusterrole, clusterrolebinding

(cherry picked from commit 5ac225d)
@mergify mergify bot requested a review from a team as a code owner November 8, 2024 08:31
@mergify mergify bot added the backport label Nov 8, 2024
@mergify mergify bot requested review from michel-laterman and swiatekm and removed request for a team November 8, 2024 08:31
@michalpristas
Copy link
Contributor

manifest needs to be probably regenerated @pkoutsovasilis

@michalpristas michalpristas added the Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team label Nov 8, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@pkoutsovasilis pkoutsovasilis merged commit 047b10c into 8.x Nov 8, 2024
9 checks passed
@pkoutsovasilis pkoutsovasilis deleted the mergify/bp/8.x/pr-5593 branch November 8, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants