forked from evolvingweb/puppet-apt
-
Notifications
You must be signed in to change notification settings - Fork 462
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
Wrong error message: Unsupported osfamily or lsbdistid #247
Comments
Hi, I hit that as well. #248 |
Same problem |
This was solved with PR #248 Cheers. |
Ok. Apparently I was using outdated forked version. I think issue can be closed already :) |
Hello! We are doing some house keeping and noticed that this issue has been open for a long time. We're going to close it but please do raise another issue if the issue still persists. 😄 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
apt::params
looks for$::lsbdistid
and$::lsbdistcodename
.If any combination of those values aren't supported, catalog compilation fails with this error:
Unsupported osfamily (XXX) or lsbdistid (YYY)
lsbdistcodename should be mentioned instead of osfamily.
Note: This bug is particularly frustrating as a new default clause as been added to the case in
apt::params
. Having a misleading error message makes debugging harder, especially for existing rspec tests in other modules which are now all suddenly failing.Those rspec tests might not always inject those facts due to puppetlabs-apt not having a hard requirement on them before ; because OS specific configurations defined in params were only used by
apt::backports
andapt::ppa
.The text was updated successfully, but these errors were encountered: