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

Fixes #4384 - Fix wrong copy of ncf files wich included perserved wrong owner,group and permissions #275

Merged
merged 2 commits into from
Mar 4, 2014
Merged

Fixes #4384 - Fix wrong copy of ncf files wich included perserved wrong owner,group and permissions #275

merged 2 commits into from
Mar 4, 2014

Conversation

nperron
Copy link
Contributor

@nperron nperron commented Jan 17, 2014

Fixes #4384 - Fix wrong copy of ncf files wich included perserved wrong owner,group and permissions

cf http://www.rudder-project.org/redmine/issues/4384

@@ -126,29 +126,31 @@ bundle agent update_action

root_server::
"${g.rudder_ncf}/common"
copy_from => copy_digest("${g.rudder_ncf_origin_common}"),
copy_from => copy_digesti_without_perms("${g.rudder_ncf_origin_common}"),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's an extra i in here.

…hid and group to 'root' and to copy from /usr/share/ncf on Rudder Server
@nperron
Copy link
Contributor Author

nperron commented Feb 7, 2014

Updated:

  • Fix 'i' typo
  • Force copy of ncf files to mode 644 insteqd of 544
  • Copy only from /usr/share/ncf on Rudder Server

@jooooooon
Copy link
Member

This looks great. Merging. Thanks @nperron

jooooooon added a commit that referenced this pull request Mar 4, 2014
…cf_files

Fixes #4384 - Fix wrong copy of ncf files wich included perserved wrong owner,group and permissions
@jooooooon jooooooon merged commit 214ed83 into Normation:branches/rudder/2.9 Mar 4, 2014
amousset pushed a commit to amousset/rudder-techniques that referenced this pull request Dec 8, 2015
…gment_packages_into_idependent_entities

Fixes #4666: Change the dependencies of the packages to ease a per-servi...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants