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

Adds OracleLinux value to operating system matching in params #177

Closed
wants to merge 1 commit into from
Closed

Adds OracleLinux value to operating system matching in params #177

wants to merge 1 commit into from

Conversation

antonvorobyev
Copy link

Fix #176 "Add OracleLinux value to operating system matching in params".

@3flex
Copy link
Contributor

3flex commented Nov 12, 2013

To make this more flexible (so operating systems don't have to be individually added) what about using the osfamily fact instead? This fact is already used by the package manifest so shouldn't affect platforms supported by the module.

You'd want to check for 'debian', 'redhat', 'suse' and have a default case where nx_daemon_user is just set to 'nginx'.

@3flex
Copy link
Contributor

3flex commented Nov 25, 2013

Hi @antonvorobyev - fixed in #183, so this can be closed.

@antonvorobyev
Copy link
Author

Hi @3flex, thank you.

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.

Add OracleLinux value to operating system matching in params
2 participants