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

Set version defaults with hiera data #685

Merged
merged 2 commits into from
Jan 29, 2024
Merged

Conversation

traylenator
Copy link
Contributor

@traylenator traylenator commented Nov 10, 2023

Pull Request (PR) description

Use hiera module level data to specify default versions and if a repository should be configured.

This will make adding new OSes easier.

2nd commit fixes some rpsec by adding facts so the service provider is detected correctly.

@traylenator
Copy link
Contributor Author

Specifying $manage_package_repo as a strict boolean would make more sense but the module makes explicit use already of the undef, true, false possibilites.

@traylenator traylenator added the enhancement New feature or request label Nov 10, 2023
@kenyon kenyon mentioned this pull request Nov 29, 2023
@zilchms
Copy link
Contributor

zilchms commented Jan 28, 2024

@traylenator , can you resolve the merge conflicts? then i can merge this

@traylenator
Copy link
Contributor Author

@traylenator , can you resolve the merge conflicts? then i can merge this

Sure - will take a look tomorrow.

Use hiera module level data to specify default versions
and if a repository should be configured.

This will make adding new OSes easier.
With OS facts set the following error occured:

```
NoMethodError:
     #   undefined method `downcase' for nil:NilClass
     #   ./vendor/bundle/ruby/3.2.0/gems/puppet-8.3.1/lib/puppet/provider/service/init.rb:25:in `block in <top (required)>'
```

since the service provider could not be determined correctly.
@zilchms zilchms merged commit bc94e0d into voxpupuli:master Jan 29, 2024
12 checks passed
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

Successfully merging this pull request may close these issues.

3 participants