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

changed user and group for the extract resource #147

Merged
merged 1 commit into from
May 20, 2016

Conversation

sacchettom
Copy link
Contributor

$catalina_home gets created with $user:$group as the owner. But, extraction was happening with the default/hiera user and group ($::tomcat::group and $::tomcat::user). If these are not the same or can't access to each other's files, extraction will fail with permission issues.

$catalina_home gets created with $user:$group as the owner. But, extraction is happening with the default/hiera user and group ($::tomcat::group and $::tomcat::user). If these are not the same or can't access to each other's files, extraction will fail with permission issues.
@DavidS
Copy link
Contributor

DavidS commented May 19, 2016

Good catch. Sadly the travis tests timed out. I've restarted them, and will check back in a while to see if everything else is fine.

@sacchettom
Copy link
Contributor Author

Hi @DavidS , unluckily seems like Travis got some troubles with your jobs, and not just for my commit; many others seem to be failing due to CI tests timing out.

@DavidS DavidS merged commit 6fdf125 into puppetlabs:master May 20, 2016
@DavidS
Copy link
Contributor

DavidS commented May 20, 2016

@sacchettom yeah, this is problematic. Merged, since the failures are not connected to your issue. I'll have a look to get those tests cleaned up.

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.

3 participants