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 JRuby in tests to 9.4.8.0 #9354

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mhashizume
Copy link
Contributor

@mhashizume mhashizume commented May 20, 2024

This commit updates the version of JRuby used in spec tests from 9.4.3.0 to 9.4.8.0, which matches what is used in the latest version of Puppet Server 8.x. (See: puppetlabs/jruby-utils@0e23173.)

JRuby 9.4.8.0 includes bug fixes to several issues that we have worked around in Puppet in the past:

Because of these bug fixes in JRuby, this commit also removes Puppet's workarounds.

@mhashizume mhashizume added the maintenance Maintenance chores are excluded from changelogs label May 20, 2024
@mhashizume mhashizume requested a review from a team as a code owner May 20, 2024 21:27
@mhashizume mhashizume force-pushed the maint/main/puppetserver-main-jruby branch 2 times, most recently from ab460e6 to ba82d90 Compare May 21, 2024 20:59
@mhashizume
Copy link
Contributor Author

Two notes here:

  • We can probably close PUP-8612 after merging this.
  • It looks like tests are failing because some string formatting issue. Not sure what's going on here, I need to dig deeper.

This commit updates the version of JRuby used in spec tests from 9.4.3.0
to 9.4.8.0, which matches what is used in the latest version of Puppet
Server 8.x. (See: puppetlabs/jruby-utils@0e23173.)

JRuby 9.4.8.0 includes bug fixes to several issues that we have worked
around in Puppet in the past:

- Dir.glob not properly supporting wildcard syntax (worked around in
  Puppet in 663062a, resolved in JRuby in jruby/jruby@a39cd49).
- format and sprintf not supporting %a and %A (worked around in Puppet
  in 334ea05, resolved in JRuby in jruby/jruby#7996).

Because of these bug fixes in JRuby, this commit also removes Puppet's
workarounds.
@mhashizume mhashizume force-pushed the maint/main/puppetserver-main-jruby branch from ba82d90 to 833d1cc Compare July 26, 2024 18:11
@mhashizume mhashizume changed the title Update JRuby in tests to 9.4.7.0 Update JRuby in tests to 9.4.8.0 Jul 26, 2024
@joshcooper
Copy link
Contributor

Similar bump was made in puppetlabs/facter#2740

@mhashizume mhashizume marked this pull request as draft August 21, 2024 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Maintenance chores are excluded from changelogs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants