-
Notifications
You must be signed in to change notification settings - Fork 126
Ensure CAP_CHOWN capability is included if root user is required #2331
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
Conversation
test integrations |
Created or updated PR in integrations repository to test this version. Check elastic/integrations#12295 |
Package healthcheck:
test: "elastic-agent status"
retries: 180
interval: 1s
cap_drop:
- ALL Package healthcheck:
test: "elastic-agent status"
retries: 180
interval: 1s
pid: host
user: root
cap_add: [AUDIT_CONTROL,AUDIT_READ,CAP_CHOWN]
cap_drop:
- ALL Package healthcheck:
test: "elastic-agent status"
retries: 180
interval: 1s
user: root
cap_add: [CAP_CHOWN]
cap_drop:
- ALL |
💚 Build Succeeded
History
cc @mrodm |
test integrations |
Created or updated PR in integrations repository to test this version. Check elastic/integrations#12295 |
// and it should not be overwritten by the value in the manifest | ||
if info.Agent.User == "" && agentManifest.Privileges.Root { | ||
// and it should not be overwritten by the value in the package or DataStream manifest | ||
if info.Agent.User == "" && (r.pkgManifest.Agent.Privileges.Root || r.dataStreamManifest.Agent.Privileges.Root) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previously, it was just considered the privileges settings from package manifest. It was no taken into account the one from the data stream manifest.
Ensure that CAP_CHOWN capability is present in the docker-compose scenario for the Elastic Agent container when the package requires or sets root privileges.
Fixes #2330
Additional:
system
package defines root privileges in the manifest of some of its data streams: example data stream manifest linkAuthor's checklist