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

pam: Add Rocky and AlmaLinux support #72

Open
hedasp opened this issue Mar 4, 2022 · 3 comments
Open

pam: Add Rocky and AlmaLinux support #72

hedasp opened this issue Mar 4, 2022 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@hedasp
Copy link

hedasp commented Mar 4, 2022

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 7.14.0
  • Ruby: 2.7.5
  • Distribution: Rocky/AlmaLinux
  • Module version: 4.0.0

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

Using module on Rocky or AlmaLinux fails.

What are you seeing

Rocky and AlmaLinux not recognized as valid OS names.

What behaviour did you expect instead

Rocky and AlmaLinux recognized as valid OS names.

Output log

Error while evaluating a Resource Statement, Evaluation Error: Error while evaluating a Function Call, not supported on Rocky (file: /etc/puppetlabs/code/environments/production/modules/googleauthenticator/manifests/pam.pp, line: 40, column: 25)

Any additional information you'd like to impart

Please update manifests/pam.pp and manifests/pam/common.pp with:
/RedHat|CentOS|Rocky|AlmaLinux/

@smortex smortex added the good first issue Good for newcomers label Mar 4, 2022
@kenyon kenyon added the enhancement New feature or request label Mar 8, 2022
@sbrowne-godaddy
Copy link

sbrowne-godaddy commented Jun 17, 2024

I don't have puppet on an Alma or Rocky box right this minute, but. What's the output of:

facter -p os.family os.name

It may be just as easy as replacing facts['os']['name'] with facts['os']['family'].

@TheMeier
Copy link

You can always look up the mock facts in facterdb, e.g. here for alama 9 https://github.com/voxpupuli/facterdb/blob/master/facts/4.7/almalinux-9-x86_64.facts

@sbrowne-godaddy
Copy link

Huh; cool. Didn't know that existed. So yes facts['os']['family'] is RedHat for both.

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

No branches or pull requests

5 participants