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

$worker_connections must be an integer #460

Closed
jacobbuck opened this issue Sep 24, 2014 · 19 comments · Fixed by #461
Closed

$worker_connections must be an integer #460

jacobbuck opened this issue Sep 24, 2014 · 19 comments · Fixed by #461

Comments

@jacobbuck
Copy link

When following the quick start guide, I get stuck at the following:

Running Puppet with site.pp...
stdin: is not a tty
warning: Could not retrieve fact fqdn
$worker_connections must be an integer. at /tmp/vagrant-puppet-1/modules-0/nginx/manifests/config.pp:87 on node precise32
The following SSH command responded with a non-zero exit status.
Vagrant assumes that this means the command failed!

puppet apply --modulepath '/tmp/vagrant-puppet-1/modules-0:/etc/puppet/modules' --manifestdir /tmp/vagrant-puppet-1/manifests --detailed-exitcodes /tmp/vagrant-puppet-1/manifests/site.pp || [ $? -eq 2 ]

Stdout from the command:

warning: Could not retrieve fact fqdn


Stderr from the command:

stdin: is not a tty
$worker_connections must be an integer. at /tmp/vagrant-puppet-1/modules-0/nginx/manifests/config.pp:87 on node precise32

My site.pp file looks like:

class { 'nginx': }

nginx::resource::vhost { 'example.com':
  www_root => '/vagrant/www',
}

Just started learning this Puppet stuff, and not to a good start 😢

@jfryman
Copy link
Contributor

jfryman commented Sep 24, 2014

Can you please tell me what version of the module you're currently using?

Jacob Buck wrote:

When following the quick start guide, I get stuck at the following:

|Running Puppet with site.pp...
stdin: is not a tty
warning: Could not retrieve fact fqdn
$worker_connections must be an integer. at
/tmp/vagrant-puppet-1/modules-0/nginx/manifests/config.pp:87 on node
precise32
The following SSH command responded with a non-zero exit status.
Vagrant assumes that this means the command failed!

puppet apply --modulepath
'/tmp/vagrant-puppet-1/modules-0:/etc/puppet/modules' --manifestdir
/tmp/vagrant-puppet-1/manifests --detailed-exitcodes
/tmp/vagrant-puppet-1/manifests/site.pp || [ $? -eq 2 ]

Stdout from the command:

warning: Could not retrieve fact fqdn

Stderr from the command:

stdin: is not a tty
$worker_connections must be an integer. at
/tmp/vagrant-puppet-1/modules-0/nginx/manifests/config.pp:87 on node
precise32
|

My site.pp file looks like:

|class { 'nginx': }

nginx::resource::vhost { 'example.com':
www_root => '/vagrant/www',
}
|

Just started learning this Puppet stuff, and not to a good start 😢


Reply to this email directly or view it on GitHub
#460.

@jacobbuck
Copy link
Author

Latest version installed via puppet module install, which I believe is v0.1.0.

@jfryman
Copy link
Contributor

jfryman commented Sep 24, 2014

Interesting. How about the version of Puppet?

@jacobbuck
Copy link
Author

Puppet 3.7.1, installed via gem.

@jfryman
Copy link
Contributor

jfryman commented Sep 24, 2014

@jacobbuck Sorry for all the questions! Is this an masterless-less install or with a Puppet Master?

@jfryman
Copy link
Contributor

jfryman commented Sep 24, 2014

@jacobbuck ah, nm. I see the details I need in the output. Give me just a few minutes to try and reproduce...

@jacobbuck
Copy link
Author

@jfryman That's alright. I'm using the Puppet Apply Provisioner in Vagrant (v1.5.4).

@jfryman
Copy link
Contributor

jfryman commented Sep 24, 2014

@jacobbuck Last question, I promise! :)

Are you using librarian-puppet or r10k to install the module, or did you just download the module?

@jacobbuck
Copy link
Author

Neither, I'm just using the puppet module subcommand:

puppet module install jfryman-nginx --modulepath ./modules

Though I might investigate using librarian-puppet or r10k later.

@jfryman
Copy link
Contributor

jfryman commented Sep 24, 2014

🆒

Can I have you run the following command, and see if everything works afterword?

puppet module install ripienaar-module_data --modulepath ./modules

EDIT: Misspelled ripienaar...

@jacobbuck
Copy link
Author

Hmm...

$ puppet module install ripinnear-module_data --modulepath ./modules
Notice: Preparing to install into /[redacted]/modules ...
Notice: Downloading from https://forgeapi.puppetlabs.com ...
Error: Could not install 'ripinnear-module_data' (latest)
  No releases are available from https://forgeapi.puppetlabs.com
    Does 'ripinnear-module_data' have at least one published release?

@jfryman
Copy link
Contributor

jfryman commented Sep 24, 2014

@jacobbuck sorry, I misspelt ripienaar. Always gets me. 😢

puppet module install ripienaar-module_data --modulepath ./modules

@jacobbuck
Copy link
Author

@jfryman Easily done 😄 Looks much better:

$ puppet module install ripienaar-module_data --modulepath ./modules
Notice: Preparing to install into /[redacted]/modules ...
Notice: Downloading from https://forgeapi.puppetlabs.com ...
Notice: Installing -- do not interrupt ...
/[redacted]/modules
└── ripienaar-module_data (v0.0.3)

Just running a vagrant up now

@jacobbuck
Copy link
Author

@jfryman Hang on, forgot I was trying out apache. Started again with nginx:

$ puppet module install jfryman-nginx --modulepath ./modules
Notice: Preparing to install into /path/to/vagrant/project/modules ...
Notice: Downloading from https://forgeapi.puppetlabs.com ...
Notice: Installing -- do not interrupt ...
/path/to/vagrant/project/modules
└─┬ jfryman-nginx (v0.1.0)
  ├── puppetlabs-apt (v1.6.0)
  ├── puppetlabs-concat (v1.1.1)
  ├── puppetlabs-stdlib (v4.3.2)
  └── ripienaar-module_data (v0.0.3)
$ puppet module install ripienaar-module_data --modulepath ./modules
Notice: Preparing to install into /path/to/vagrant/project/modules ...
Notice: Module ripienaar-module_data 0.0.3 is already installed.

Still get the same errors when running vagrant up.

@jfryman
Copy link
Contributor

jfryman commented Sep 25, 2014

@jacobbuck ok, trying to get a vagrant environment to reproduce. One moment...

@jfryman
Copy link
Contributor

jfryman commented Sep 25, 2014

OK. I can reproduce now... working on a fix.

@jfryman
Copy link
Contributor

jfryman commented Sep 25, 2014

@jacobbuck OK! I have a fix.

I apologize that this has been a bit of a pain right off the bat. We're actually right in the middle of some major changes to this module, and so anything in the 0.1.x branch is probably going to be a bit rough for a bit. If you want/need some stability in the meantime, I recommend that you use v0.0.10 for the time being.

Anyway... here's what I did.

What is missing is you'll need to configure Hiera in your Vagrantfile to load the module_data backend. I thought this wasn't necessary, but it seems it is (/cc @3flex)

Take a look at this gist... https://gist.github.com/jfryman/e9f08affec54307e5198

I'm going to push out an update to the documentation so this is less confusing for new folks. Sorry for the inconvenience, and I hope this doesn't spoil your fun. :)

@jacobbuck
Copy link
Author

@jfryman Great stuff. I also realised that the hashicorp/precise32 box is running Puppet v2.7.19 (ahem @mitchellh). Might find a better supported box, or use packer. For now v0.0.10 runs without a hitch.
Thanks for your help.

@deepjoy
Copy link

deepjoy commented Sep 26, 2014

I'm trying to install 0.0.10 but can't unless I download the tgz. Since this is a vagrant based setup I'd like to avoid doing that. I have tried

puppet-module install jfryman/nginx -v 0.0.10
puppet module install jfryman/nginx -v 0.0.10
puppet-module install jfryman-nginx --version 0.0.10
puppet-module install jfryman-nginx --version=0.0.10
puppet-module install jfryman-nginx --version="<0.1.0"

etc.

No Matter what I do puppet module always installs version 0.1.1 instead. Am I missing something? What bash command can I run to install version 0.0.10?

Using Puppet 2.7 with precise64 loaded from https://cloud-images.ubuntu.com/vagrant/precise/current/

chriskuehl added a commit to ocf/puppet that referenced this issue Oct 13, 2014
Current nginx module complains:

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

This appears to be non-trivial to fix, with the maintainer stating [1]:

> We're actually right in the middle of some major changes to this
> module, and so anything in the 0.1.x branch is probably going to be a
> bit rough for a bit. If you want/need some stability in the meantime,
> I recommend that you use v0.0.10 for the time being.

So, downgrading seems like a reasonable option for now.

[1] voxpupuli/puppet-nginx#460 (comment)
chriskuehl added a commit to ocf/puppet that referenced this issue Oct 13, 2014
Current nginx module complains:

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

This appears to be non-trivial to fix, with the maintainer stating [1]:

> We're actually right in the middle of some major changes to this
> module, and so anything in the 0.1.x branch is probably going to be a
> bit rough for a bit. If you want/need some stability in the meantime,
> I recommend that you use v0.0.10 for the time being.

So, downgrading seems like a reasonable option for now.

[1] voxpupuli/puppet-nginx#460 (comment)
chriskuehl added a commit to ocf/puppet that referenced this issue Oct 13, 2014
Current nginx module complains:

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

This appears to be non-trivial to fix, with the maintainer stating [1]:

> We're actually right in the middle of some major changes to this
> module, and so anything in the 0.1.x branch is probably going to be a
> bit rough for a bit. If you want/need some stability in the meantime,
> I recommend that you use v0.0.10 for the time being.

So, downgrading seems like a reasonable option for now.

[1] voxpupuli/puppet-nginx#460 (comment)
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 a pull request may close this issue.

3 participants