"worker_connections must be an integer" error

[andrewnoruk]

When attempting to install nginx (using the first example from the README) I am seeing this error. I have tried with the latest version of the plugin (from master) and also with the 0.0.10 version as I read on another thread.

In both cases, I get this error:

err: Could not retrieve catalog from remote server: Error 400 on SERVER: $worker_connections must be an integer

[jfryman]

How are you declaring the parameter? I think this is a valid error. For example:

class { 'nginx':
  worker_connections => 1,
}

is different than...

class { 'nginx':
  worker_connections => '1',
}

If you are doing the latter, please remove the quotes to make it an integer. If you are already doing the latter... well then we might have to dig in a bit. 😁

Please let me know if this resolves things for you.

[3flex]

@the-andrew which version of puppetlabs-stdlib are you using? If it's older than 4.2.0 then that will be the problem... from the 4.2.0 changelog:

Fix is_integer() to return true if passed integer strings

Since c87744a we should update the dependency to >= 4.2.0 <5.0.0. I'll raise a PR.

[mrmagcore]

Why are worker_connections, worker_rlimit_nofile and proxy_set_header required? Couldn't they be left undef'd and set to nginx's own defaults?

[andrewnoruk]

I didn't specify this on the original post, but I was not actually passing any parameters for this class. The error was coming up when the defaults should have been used.

[ebuzzz]

I'm still experiencing this issue with master 3.7.2-4, puppetlabs-stdlib v4.9.0 and the v0.3.0 version of this plugin. Any specific reason why the config.pp doesn't just use the integer values instead of the strings?