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

Use bolt_supported beaker helper #521

Merged
merged 2 commits into from
May 2, 2024
Merged

Conversation

traylenator
Copy link
Contributor

@traylenator traylenator commented Apr 28, 2024

Pull Request (PR) description

Use new bolt_supported? helper to conditionally setup and run beaker tests.
The bolt_supported? helper is available with rubygem-beaker_puppet_helpers v1.3.0

Since bolt is available for RHEL9 at the time of writing this will
enable bolt tests for RHEL9.

@traylenator
Copy link
Contributor Author

Requires - voxpupuli/beaker_puppet_helpers#42.

@traylenator traylenator force-pushed the bolt_supported branch 2 times, most recently from f5d8318 to 19432a6 Compare April 30, 2024 08:07
Use new bolt_supported helper to conditionally setup and run
beaker tests.

Since bolt is available for RHEL9 at the time of writing this will
enable bolt tests for RHEL9.
@traylenator traylenator marked this pull request as ready for review May 2, 2024 08:47
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I'm really pedantic I'd prefer a release of voxpupuli-acceptance that ensures the presence, so I've opened voxpupuli/voxpupuli-acceptance#79. If we release that as 3.1.0 then we can ensure the helper is present by updating Gemfile here.

@traylenator
Copy link
Contributor Author

Somewhat guessing what is meant to be put in modulesync config to overide default version of voxpup-acceptance.

@ekohl
Copy link
Member

ekohl commented May 2, 2024

I took the liberty of pushing straight to master and https://github.com/voxpupuli/voxpupuli-acceptance/actions/runs/8921727091 completed.

@ekohl ekohl merged commit 4d5793c into voxpupuli:master May 2, 2024
24 checks passed
@traylenator traylenator deleted the bolt_supported branch May 2, 2024 10:04
@evgeni evgeni mentioned this pull request May 8, 2024
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.

4 participants