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

Ubuntu 18.04 gives warning #361

Closed
fnoop opened this issue Apr 28, 2018 · 4 comments
Closed

Ubuntu 18.04 gives warning #361

fnoop opened this issue Apr 28, 2018 · 4 comments
Labels
enhancement New feature or request

Comments

@fnoop
Copy link
Contributor

fnoop commented Apr 28, 2018

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 5.5.1
  • Ruby: 2.5.1
  • Distribution: Ubuntu 18.04 (bionic beaver)
  • Module version: latest

How to reproduce (e.g Puppet code you use)

What are you seeing

Warning: Scope(Class[Nodejs::Params]): The nodejs module might not work on Ubuntu 18.04. Sensible defaults will be attempted.

What behaviour did you expect instead

No warning

Any additional information you'd like to impart

It looks like install works because the nodesource repo now exists for bionic.

@fnoop fnoop changed the title Ubuntu 18.04 not supported Ubuntu 18.04 gives warning Apr 28, 2018
@fnoop
Copy link
Contributor Author

fnoop commented Apr 28, 2018

Note: Ubuntu 18.04 is now the latest LTS distro, important that it is fully supported now.

@bastelfreak
Copy link
Member

Hi @fnoop, thanks for reporting this. Could you try adding acceptance tests for ubuntu 18.04?

@juniorsysadmin juniorsysadmin added the enhancement New feature or request label Apr 28, 2018
fnoop added a commit to fnoop/puppet-nodejs that referenced this issue Apr 28, 2018
@fnoop
Copy link
Contributor Author

fnoop commented Apr 28, 2018

@bastelfreak - Sorry acceptance tests completely baffle me, I have no idea even how to run them. I see in spec/acceptance/nodesets/ I could add an xml file for 18.04, but the files say:

# This file is managed via modulesync
# https://github.com/voxpupuli/modulesync
# https://github.com/voxpupuli/modulesync_config

So I'm guessing adding it manually is the wrong thing to do.

If you could point me in the right direction, be happy to have a go though?

fnoop added a commit to fnoop/puppet-nodejs that referenced this issue Apr 28, 2018
juniorsysadmin added a commit that referenced this issue Apr 29, 2018
Add support for ubuntu bionic 18.04, #361
@juniorsysadmin
Copy link
Member

Fixed by #362

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants