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 #5616) Fix ownership of component specs in elastic-agent docker image #5620

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Sep 27, 2024

What does this PR do?

Fix ownership of components' spec files in <agent install dir>/data/elastic-agent-<commit>/components/*.yml from root:root to <user>:<user> where user is the default user the agent process will be started with

Why is it important?

To prevent permissions mismatches between component binaries and their spec yaml files under ../components

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

How to test this PR locally

Related issues

Questions to ask yourself

  • How are we going to support this in production?
  • How are we going to measure its adoption?
  • How are we going to debug this?
  • What are the metrics I should take care of?
  • ...

This is an automatic backport of pull request #5616 done by [Mergify](https://mergify.com).

Co-authored-by: Julien Lind <julien.lind@elastic.co>
(cherry picked from commit 1a10c3d)
@mergify mergify bot requested a review from a team as a code owner September 27, 2024 12:56
@mergify mergify bot added the backport label Sep 27, 2024
@mergify mergify bot requested review from blakerouse and pkoutsovasilis and removed request for a team September 27, 2024 12:56
@mergify mergify bot assigned pchila Sep 27, 2024
@ycombinator ycombinator enabled auto-merge (squash) September 27, 2024 19:58
@ycombinator
Copy link
Contributor

Similar to #5616 (comment), this PR also needs to be force merged so Integrations Server on ESS can spin up without errors...

@ycombinator ycombinator merged commit 601aaeb into 8.x Sep 27, 2024
11 of 13 checks passed
@ycombinator ycombinator deleted the mergify/bp/8.x/pr-5616 branch September 27, 2024 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants