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

Update to gha-puppet v3 & voxpupuli-test 9.x #82

Merged
merged 3 commits into from
Mar 21, 2025

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Mar 4, 2025

No description provided.

@ekohl
Copy link
Member Author

ekohl commented Mar 5, 2025

Wut?

Error: Facter: Error while resolving custom fact fact='rh_certificate', resolution='<anonymous>': break from proc-closure

ekohl added 2 commits March 5, 2025 19:08
In Ruby you're supposed to use next to exit a block, otherwise you get
errors like:

    Error: Facter: Error while resolving custom fact fact='rh_certificate', resolution='<anonymous>': break from proc-closure
@ekohl
Copy link
Member Author

ekohl commented Mar 5, 2025

Error: Facter: Error while resolving custom fact fact='rh_certificate', resolution='<anonymous>': break from proc-closure

You're supposed to use next.

Now for the real failure: I think this is because a bash script has replaced the Ruby package so checks are failing.

@ofedoren I think that module effectively becomes obsolete, correct?

@ofedoren
Copy link
Member

ofedoren commented Mar 6, 2025

Now for the real failure: I think this is because a bash script has replaced the Ruby package so checks are failing.

Not sure how bash package would affect this... I mean, I tested with Ansible role and it worked. We probably need to adjust Puppet class somehow though?..

@ofedoren I think that module effectively becomes obsolete, correct?

You mean the entire puppet_foreman_scap_client? I wouldn't say so. We still need this to automatically install the package and generate the config file. Although, at some point, we'd need to explicitly install Bash version here instead of Ruby one.

Since 3.14 the Ruby implementation is replaced by bash. For now this
module is incompatible with those newer versions.
@evgeni
Copy link
Member

evgeni commented Mar 21, 2025

My understanding is that the new bash implementation is backwards compatible to the Ruby one, it uses the same config file, same executable name, everything. So unless this changes, this module is still usable (and required) as before. It "just" needs to know the right package name to become idempotent (again).

@ekohl ekohl merged commit e16bdca into theforeman:master Mar 21, 2025
11 checks passed
@ekohl ekohl deleted the gha-puppet-v3 branch March 21, 2025 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants