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

Add some CentOS 9 legacy facts #219

Merged
merged 1 commit into from
Dec 14, 2021

Conversation

traylenator
Copy link
Contributor

@traylenator traylenator commented Dec 14, 2021

This is only manual addition of

facter --version 4.2.5
facter operatingsystem operatingsystemmajrelease operatingsystemrelease osfamily hardwaremodel architecture hostname ipaddress

To last CentOS 9 facts.

It's certainly enough for rspec-puppet-facts to consider this a valid
fact set.

In addition it also allows the redhat provider for service to
operate correctly.

@traylenator
Copy link
Contributor Author

This is obviously not the full run on a vagrant machine, would be better of course but as non vagrant user....

@bastelfreak
Copy link
Member

thanks for the PR, can you please remove this line: https://github.com/voxpupuli/facterdb/blob/master/spec/facts_spec.rb#L93 ?

@traylenator
Copy link
Contributor Author

Indeed , it missed the commit - thanks.

This is only manual addition of

```
facter --version 4.2.5
facter operatingsystem operatingsystemmajrelease operatingsystemrelease osfamily hardwaremodel architecture
```

It's certainly enough for rspec-puppet-facts to consider this a valid
fact set.

In addition it also allows the `redhat` provider for `service` to
operate correctly.
@bastelfreak
Copy link
Member

thanks for the fix!

@bastelfreak bastelfreak merged commit e36d8e2 into voxpupuli:master Dec 14, 2021
@traylenator traylenator deleted the centos9legacy branch December 14, 2021 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants