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

Switch to the Puppet Forge NodeJS module #11

Open
ntwb opened this issue Mar 25, 2017 · 6 comments
Open

Switch to the Puppet Forge NodeJS module #11

ntwb opened this issue Mar 25, 2017 · 6 comments

Comments

@ntwb
Copy link
Member

ntwb commented Mar 25, 2017

Chassis NodeJS should switch to https://forge.puppet.com/puppet/nodejs

Some of the benefits:
• Makes available a global npm provider which benefits Chassis/grunt & Chassis/bower extensions
• Primary NodeJS source package continues to be provided by https://nodesource.com/
• Supports .npmrc files
• Supports globally installing packages (edited)

@BronsonQuick
Copy link
Member

I had a bit of a crack at this tonight but I got stuck 😢 My gut feeling is that the version of puppet that precise64 ships with is out of date and this Puppet Forge module is expecting at last 4.x.x and not 3.8.x. That might be wrong though. I'll try and have another go at this later this week.

@ntwb
Copy link
Member Author

ntwb commented Mar 26, 2017

Thanks for taking a stab at it 🔪

The modules docs state: Puppet >=3.8.7 < 5.0.0

Having not tried creating a fix for this, I did bookmark the following, hoping it would give me a headstart: https://github.com/jherbage/puppet-conf/blob/master/manifests/nodejs.pp

@BronsonQuick
Copy link
Member

Hrm. We're on 3.8.7so maybe it was just PEBKAC or maybe it's a ruby dependency. Hopefully I'll learn more later in the week.

@johnbillion
Copy link

Can this be done since Chassis/Chassis#312 was merged?

@johnbillion
Copy link

Background: I'd really like npm available in order to run things like npm test within the Chassis VM.

@BronsonQuick
Copy link
Member

Hey @johnbillion! You shouldn't need us to switch to Puppet Forge NodeJS for this to get npm installed.

If you add:

extensions:
  - chassis/nodejs

To one of your yaml files you will get the following inside your VM after a vagrant provision:
chassis__vagrantvagrant___ssh__vagrant_ssh_2018-02-14_18-14-20

Sometimes the formatting of the yaml file can cause issues. I've often caught myself up but not uncommenting extensions: so it's been #extensions: and if that's the case your extensions won't be installed and provisioned.

If you're still stuck with this then please send me your yaml files so I can try and replicate it and give you a fix for it!

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

No branches or pull requests

3 participants