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

setting configure_firewall true returns error, port is undefined #174

Closed
mmarseglia opened this issue Sep 9, 2014 · 5 comments
Closed
Labels
needs-feedback Further information is requested

Comments

@mmarseglia
Copy link

Hi,

I enabled the firewall using the following code but received an error.
class { 'jenkins':
configure_firewall => false,
}

include jenkins::master

Error: Failed to apply catalog: Parameter dport failed on Firewall[500 allow Jenkins inbound traffic]: Munging failed for value "undef" in class dport: no such service undef/tcp at /etc/puppetlabs/puppet/modules/jenkins/manifests/firewall.pp:16
Wrapped exception:
Munging failed for value "undef" in class dport: no such service undef/tcp

the firewall module is using $::jenkins::port but I don't see that defined.

@rtyler
Copy link

rtyler commented Sep 14, 2014

@mmarseglia What version of the module are you using? I just tried this with an RSpec test and couldn't reproduce it :/

@mmarseglia
Copy link
Author

the puppetlabs forge version swarm was still at version 1.9 so I git cloned and I have commit 09b8cbc

@rtyler
Copy link

rtyler commented Sep 22, 2014

@mmarseglia Yeah, I cannot figure out how to reproduce this at all :(

Can you create a sample reproduction case with a small git repo or something like that?

@rtyler rtyler added the needs-feedback Further information is requested label Sep 22, 2014
@eshamow
Copy link
Contributor

eshamow commented Oct 24, 2014

@rtyler I think it's because strict_variables is set to true in my environment - you can override the value in hiera, but with strict_variables on, if it's not explicitly undefined, compilation fails.

rtyler pushed a commit that referenced this issue Oct 24, 2014
Fix for #174 allows setting $jenkins::port
@rtyler rtyler added this to the 1.3.0 - Bernard milestone Oct 24, 2014
@rtyler
Copy link

rtyler commented Oct 24, 2014

Noted

@rtyler rtyler closed this as completed Oct 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-feedback Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants