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 facts_match_regex parameter in katello class #498

Merged
merged 1 commit into from
Jun 27, 2024

Conversation

archanaserver
Copy link
Contributor

@archanaserver archanaserver commented Jun 3, 2024

For reference: theforeman/puppet-candlepin#262 (review)

This PR adds facts_match_regex parameter support to katello::candlepin. It enables filtering consumer facts via regex patterns, enhancing configuration flexibility and control.

Changes Made

  • Introduced facts_match_regex parameter in katello::candlepin.
  • Updated katello class to pass facts_match_regex to katello::candlepin.
  • Added unit tests to validate parameter functionality.

@ehelms
Copy link
Member

ehelms commented Jun 3, 2024

The acceptance tests are failing due to outside forces. EL8 due to CentOS 8 Stream repositories going away and EL9 due to nightl ybeing in transition post branching. A basic unit test for this parameter would be nice to have.

@archanaserver
Copy link
Contributor Author

This is working fine now \o/

@archanaserver archanaserver changed the title Add facts_match_regex paramter in katello class Add facts_match_regex parameter in katello class Jun 27, 2024
@ehelms ehelms merged commit e057cf3 into theforeman:master Jun 27, 2024
11 of 13 checks passed
@archanaserver archanaserver deleted the add-parameter branch June 28, 2024 15:21
@ekohl
Copy link
Member

ekohl commented Aug 8, 2024

#504 fixes up the minimum version dependency.

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