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

Update to not require puppet apt module < 5.0 #713

Closed
meltingrobot opened this issue Aug 17, 2018 · 6 comments
Closed

Update to not require puppet apt module < 5.0 #713

meltingrobot opened this issue Aug 17, 2018 · 6 comments

Comments

@meltingrobot
Copy link
Contributor

Hi, having librarian barf out on this module when I'm trying to update the apt module the 5.0.1 due to this line in spec/spec_helper_acceptance.rb

install_module_from_forge('puppetlabs-apt', '>= 4.1.0 < 5.0.0') if fact('os.family') == 'Debian'

I need the newer apt module for Ubuntu 18.04.

@wyardley
Copy link
Contributor

#714

@wyardley
Copy link
Contributor

See my comments in #714 - are you sure this is the reason, though?

Also note that Ubuntu 18.04 is not currently listed in supported versions in the module's metadata anyway. I'm not sure if we're testing 18 in acceptance tests yet, but might make sense to submit a separate PR to add support for Ubuntu 18. It's possible that the module might need some other fixes for that, aside from metadata.

@meltingrobot
Copy link
Contributor Author

The puppet server itself isn't getting updated to 18.04, we just need to update the apt module to handle 18.04 puppet clients.

@meltingrobot
Copy link
Contributor Author

Link to the merged commit that adds support for 18.04 in the apt puppet module:

puppetlabs/puppetlabs-apt#769

@wyardley
Copy link
Contributor

Right, but modules themselves list the OSes they support — in this case, it would apply to where you’re running the module, not to the puppet server itself

That’s not to say that the module will not work on Ubuntu 18, but it’s not currently listed as supported in the rabbitmq module, and we don’t test against it.

@wyardley
Copy link
Contributor

I've merged #714 -- let me know if that changes anything.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants