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

(PUP-11935) Update JRuby in tests to 9.3.y.z #9087

Merged
merged 2 commits into from
Sep 16, 2023

Conversation

mhashizume
Copy link
Contributor

Puppetserver upgraded from the JRuby 9.2.y.z series to JRuby 9.3.y.z as part of SERVER-3133. This commit updates the JRuby version used in Rspec tests to match the most recent JRuby version used in the Puppetserver 7.x series.

@mhashizume mhashizume requested a review from a team as a code owner July 12, 2023 17:34
@mhashizume mhashizume changed the title (maint) Update JRuby in tests to 9.3.y.z (PA-5723) Update JRuby in tests to 9.3.y.z Sep 5, 2023
Puppetserver upgraded from the JRuby 9.2.y.z series to JRuby 9.3.y.z as
part of SERVER-3133. This commit updates the JRuby version used in Rspec
tests to match the most recent JRuby version used in the Puppetserver
7.x series.
@mhashizume mhashizume changed the title (PA-5723) Update JRuby in tests to 9.3.y.z (PUP-11935) Update JRuby in tests to 9.3.y.z Sep 5, 2023
Starting with jruby-openssl 0.13.0[1] (which is included in JRuby >=
9.3.5.0), certificate signing raises an error when there is a
discrepancy between the certificate and key. This behavior in JRuby
differs from MRI OpenSSL.

This commit adds a test for this JRuby-specific behavior and updates
existing tests to skip when running on affected versions of JRuby.

[1] jruby/jruby-openssl@4b2968b
@mhashizume
Copy link
Contributor Author

Testing revealed some new JRuby behavior when signing SSL certificates, I've updated existing tests and added a new one to address this.

@joshcooper
Copy link
Contributor

I'll update the required checks to account for the jruby 9.2.21.0 -> 9.3.9.0

@joshcooper joshcooper merged commit 617e136 into puppetlabs:7.x Sep 16, 2023
13 checks passed
@mhashizume mhashizume deleted the maint/7.x/jruby-tests branch September 16, 2023 03:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants