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

Beaker 5 compatibility #240

Merged
merged 7 commits into from
Feb 9, 2024

Conversation

mhashizume
Copy link
Contributor

This PR adds Beaker 5 compatibility by dropping Beaker 4-specific code and updating the gemspec.

This PR supercedes #216

The assert_no_match method in Beaker was long deprecated then eventually
removed altogether in voxpupuli/beaker@6282311

This commit updates assert_no_match to refute_match.
@mhashizume mhashizume force-pushed the PA-5782/master/drop-beaker-4 branch 3 times, most recently from f475ddd to 014d5a8 Compare February 8, 2024 00:02
mhashizume and others added 6 commits February 8, 2024 08:02
Passenger support was dropped in Puppet 6 and support for
passenger-related methods was removed from Beaker in
voxpupuli/beaker@74047d1.

This commit removes all passenger-related Beaker methods.

The start_puppet_from_source_on! method is unused as a result of this
removal. Additionally, since this method's purpose was to control the
WEBrick-based server that was removed from Puppet in Puppet 6
(see puppetlabs/puppet@9275e62), I've also removed the
start_puppet_from_source_on! method.

Co-authored-by: Tim Meusel <[email protected]>
Beaker removed all install_puppet_agent_* method in
voxpupuli/beaker@cdaedad. This commit removes beaker-puppet methods that
relied on those install_puppet_agent_* methods.

Co-authored-by: Tim Meusel <[email protected]>
Beaker 5.0.0 was released in March 2023 and all development effort is
now focused on that stream. This commit updates beaker-puppet to
require at least Beaker 5.0.
Ruby 3.2 removed Object#taint and #tainted?. One of beaker-puppet's
depenencies, inifile, uses those methods.

This commit updates the gemspec to limit usage on Ruby 3.2 and above.

Also see: puppetlabs#206
This commit removes the deprecated get_puppet_collection method.
@mhashizume
Copy link
Contributor Author

@mhashizume mhashizume marked this pull request as ready for review February 9, 2024 16:51
@mhashizume mhashizume requested a review from a team as a code owner February 9, 2024 16:51
@bastelfreak
Copy link
Contributor

I want to repeat what I'm saying since 2017:
I think it was a huge design mistake that Puppet made the Jenkins private but still has public repos that depend on pipelines on that specific Jenkins. That excludes all outside collaborators from properly reviewing PRs.

@cthorn42 cthorn42 merged commit 7175b5c into puppetlabs:master Feb 9, 2024
6 checks passed
@mhashizume mhashizume deleted the PA-5782/master/drop-beaker-4 branch February 9, 2024 21:58
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