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

Unable to identify debian wheezy #123

Closed
x-drum opened this issue Jan 16, 2015 · 7 comments
Closed

Unable to identify debian wheezy #123

x-drum opened this issue Jan 16, 2015 · 7 comments

Comments

@x-drum
Copy link

x-drum commented Jan 16, 2015

When using a small (reduced) debian wheezy installation, openvpn::server is unable to identify it
via openvpn::params.

This is due the lack of the lsb-release package (in a minimal/reduced installation the dependency chain for lsb-release can be huge):
Without the lsb package installed $::lsbdistid and $::lsbdistrelease facts are empty and eval will fail with the message: Not supported OS / Distribution: Debian

@luxflux
Copy link
Contributor

luxflux commented Feb 3, 2015

Which facts do you suggest to use instead of $::lsbdistid and $::lsbdistrelease? Which ones are available?

@x-drum
Copy link
Author

x-drum commented Feb 4, 2015

A combination of $::operatingsystem and $::operatingsystemrelease would be possible in that situation,
for example:

[..]
'Debian': { # Debian/Ubuntu
      case $::operatingsystem {
        'Debian': {
          # Version > 8.0.0, jessie
          if(versioncmp($operatingsystemrelease, '8.0') >= 0) {
[..]

@luxflux
Copy link
Contributor

luxflux commented Feb 7, 2015

I added this now in a branch called amazon-wheezy-small-support and opened a pull request #134. Could you test it?

@peterbeck
Copy link
Contributor

by the way: shouldn't it be "8.0" instead of "8.0.0" in the Debian Jessie section ?

if(versioncmp($::lsbdistrelease, '8.0') >= 0) {

I had to change to 8.0 as there is no 8.0.x available yet and then puppet used the Wheezy section...

facter |grep lsbdistrelease
lsbdistrelease => 8.0

@luxflux
Copy link
Contributor

luxflux commented Feb 9, 2015

Thanks @peterbeck, I updated the branch. Do you mind checking it again?

@peterbeck
Copy link
Contributor

Hi @luxflux - just tried the branch and it seems to work as expected. Thanks !

@luxflux
Copy link
Contributor

luxflux commented Feb 25, 2015

Thanks for testing @peterbeck, I need to resolve #44 first before merging.

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