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.

Sign up for GitHub

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

Let the Windows installer set the correct rights on config files and directories. #588

Merged
merged 7 commits into from
Mar 18, 2019
Merged

Let the Windows installer set the correct rights on config files and directories. #588

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
24421d9
Let the Windows installer set the correct rights on config files and …
remeh Mar 7, 2019
19dd264
In datadog_monitor, do not force permissions on integration files.
remeh Mar 13, 2019
4b0cf11
Add a note in the README about the minimum datadog cookbook version f…
remeh Mar 13, 2019
7fe4e1d
Merge branch 'master' into remeh/windows-dd-agent-user
remeh Mar 13, 2019
f3b9efe
Keep the permissions on Administrators for Windows Agent v5.
remeh Mar 15, 2019
f6a3cbc
Nit in the README.
remeh Mar 15, 2019
b9f5c89
Permissions is only on config files.
remeh Mar 15, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
In datadog_monitor, do not force permissions on integration files.
  • Loading branch information
@remeh
remeh committed Mar 13, 2019
commit 19dd26478acf7c88463cf5b7f66c53217ff474c5
6 changes: 1 addition & 5 deletions providers/monitor.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,7 @@ def whyrun_supported?
action :add do
Chef::Log.debug "Adding monitoring for #{new_resource.name}"
template ::File.join(yaml_dir, "#{new_resource.name}.yaml") do
if node['platform_family'] == 'windows'
owner 'Administrators'
rights :full_control, 'Administrators'
inherits false
else
unless node['platform_family'] == 'windows'
olivielpeau marked this conversation as resolved.
Show resolved Hide resolved
owner 'dd-agent'
mode '600'
end
Expand Down