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

[postinst] Make dd-agent user and group own /opt/datadog-agent #97

Merged
merged 1 commit into from
Oct 27, 2016

Conversation

olivielpeau
Copy link
Member

What this PR does

Makes dd-agent user and group own /opt/datadog-agent (which is the home directory of dd-agent)

Motivation

Customer request: with this change, sysadmins can give no permissions at all to other users on the /opt/datadog-agent directory (required for PCI compliance), and the Agent can still work fine and not break the sysadmins' changes on upgrade.

Testing

Tested the install of a local debian build: from scratch and as an upgrade ; both work fine.

Additional notes

Also, on debian, set the home directory of dd-agent to
/opt/datadog-agent directly on the adduser call, for the sake of
brevity.

Users of the Agent can then give no permissions at all to other users
on the `/opt/datadog-agent` directory (which is the home directory of
`dd-agent`), and the Agent can still work fine and not break the users'
changes on upgrade.

Also, on debian, set the home directory of `dd-agent` to
`/opt/datadog-agent` directly on the `adduser` call, for the sake of
brevity.
@olivielpeau olivielpeau added this to the 5.10.0 milestone Oct 26, 2016
@degemer
Copy link
Member

degemer commented Oct 27, 2016

Please add a big disclaimer in the CHANGELOG. People might not expect this to happen.

@olivielpeau olivielpeau merged commit 31d7044 into master Oct 27, 2016
@olivielpeau olivielpeau deleted the olivielpeau/make-dd-agent-own-home branch October 27, 2016 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants