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

Jump to bottom

Amazon linux fails to comile #837

Closed
ryno75 opened this issue Jul 24, 2016 · 0 comments
Closed

Amazon linux fails to comile #837

ryno75 opened this issue Jul 24, 2016 · 0 comments

Comments

@ryno75
Copy link

ryno75 commented Jul 24, 2016

Attempting to use this module on Amazon linux results in:

Warning: Unknown variable: '_module_os_overrides'. at /etc/puppetlabs/code/environments/prod/modules/nginx/manifests/params.pp:81:50

This is due to the case $::osfamily block catching for RedHat but then not matching on the if statement on line 47 in params.pp (which looks for an $::operatingsystem value of RedHat and CentOS and thus does not match Amazon. Since there is no else statement that defines $_module_os_overrides so it remains uninitialized and bombs.
ryno75 pushed a commit to ryno75/puppet-nginx that referenced this issue Jul 24, 2016
fixing issue voxpupuli#837
ddc52d1
@ryno75 ryno75 mentioned this issue Jul 25, 2016
Merged
3flex added a commit that referenced this issue Jul 25, 2016
@3flex
Merge pull request #838 from ryno75/master
63f2629 
fixing issue #837
darken99 pushed a commit to darken99/puppet-nginx that referenced this issue Oct 13, 2016
fixing issue voxpupuli#837
f9db32b
Slm0n87 pushed a commit to Slm0n87/puppet-nginx that referenced this issue Mar 7, 2019
fixing issue voxpupuli#837
dbb8aec
Slm0n87 pushed a commit to Slm0n87/puppet-nginx that referenced this issue Mar 7, 2019
@3flex
Merge pull request voxpupuli#838 from ryno75/master
47b3b8b 
fixing issue voxpupuli#837
cegeka-jenkins pushed a commit to cegeka/puppet-nginx that referenced this issue Sep 13, 2019
fixing issue voxpupuli#837
61ce246
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ryno75