Skip to content

Keep psych minor version in line with jruby 9.4.13.0 #18507

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.

Sign up for GitHub

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

Jump to bottom
Merged
donoghuc merged 1 commit into from
Dec 17, 2025
Merged

Keep psych minor version in line with jruby 9.4.13.0 #18507

donoghuc merged 1 commit into from
Dec 17, 2025
+1 −0

Conversation

@donoghuc
Copy link
Member

@donoghuc donoghuc commented Dec 17, 2025

The latest psych 5.3 minor version has bumped snakeyaml-engine ruby/psych#762. This is causing issues in bundler. Constrain the psych gem to the 5.2 minor version while we are still on jruby 9.4.13.0.

The issue presents as:

Errno::ENOENT: No such file or directory - /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/gems/shared/gems/psych-5.3.1-java/deps.lst
  org/jruby/RubyIO.java:1279:in `sysopen'
          org/jruby/RubyIO.java:4275:in `read'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/jars/installer.rb:75:in `load_from_maven'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/jars/installer.rb:220:in `install_dependencies'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/jars/installer.rb:209:in `do_install'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/jars/installer.rb:154:in `vendor_jars'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/rubygems/defaults/jruby.rb:110:in `block in <main>'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/rubygems/installer.rb:367:in `block in run_post_install_hooks'
          org/jruby/RubyArray.java:2009:in `each'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/rubygems/installer.rb:366:in `run_post_install_hooks'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/bundler/rubygems_gem_installer.rb:48:in `install'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/bundler/source/rubygems.rb:205:in `install'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/bundler/installer/gem_installer.rb:55:in `install'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/bundler/installer/gem_installer.rb:17:in `install_from_spec'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/bundler/installer/parallel_installer.rb:133:in `do_install'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/bundler/installer/parallel_installer.rb:124:in `block in worker_pool'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/bundler/worker.rb:62:in `apply_func'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/bundler/worker.rb:57:in `block in process_queue'
          org/jruby/RubyKernel.java:1725:in `loop'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/bundler/worker.rb:54:in `process_queue'
          /Users/cas/elastic-repos/logstash/vendor/jruby/lib/ruby/stdlib/bundler/worker.rb:90:in `block in create_threads'

There has been a long standing string of issues around this:

The current issue seems to be that when psych depends on a different version of snakeyaml-engine that is shipped in jruby (Logstash is on 9.4.13.0 which vendors snakeyaml-engine 2.9).

@donoghuc
Keep psych minor version in line with jruby 9.4.13.0
b508cbf 
The latest psych 5.3 minor version has bumped snakeyaml-engine
ruby/psych#762. This is causing issues in bundler.
Constrain the psych gem to the 5.2 minor version while we are still on jruby
9.4.13.0.
@github-actions
Copy link
Contributor

github-actions bot commented Dec 17, 2025

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)
  • /run exhaustive tests : Run the exhaustive tests Buildkite pipeline.

@mergify
Copy link
Contributor

mergify bot commented Dec 17, 2025

This pull request does not have a backport label. Could you fix it @donoghuc? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit.
  • If no backport is necessary, please add the backport-skip label

@mashhurs mashhurs added the backport-9.3 Automated backport to the 9.3 branch label Dec 17, 2025
mashhurs
mashhurs approved these changes Dec 17, 2025
View reviewed changes
Copy link
Contributor

@mashhurs mashhurs left a comment

Choose a reason for hiding this comment

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

LGTM with 🟢 CI!

This was referenced Dec 17, 2025
Merged
Merged
@elasticmachine
Copy link

elasticmachine commented Dec 17, 2025

💚 Build Succeeded

@donoghuc donoghuc merged commit a089904 into elastic:main Dec 17, 2025
15 checks passed
@mashhurs
Copy link
Contributor

mashhurs commented Dec 17, 2025

@Mergifyio backport 9.3

@mergify
Copy link
Contributor

mergify bot commented Dec 17, 2025 •
edited
Loading

backport 9.3

✅ Backports have been created

Details

mergify bot pushed a commit that referenced this pull request Dec 17, 2025
@donoghuc @mergify
Keep psych minor version in line with jruby 9.4.13.0 (#18507)
b9d5281 
The latest psych 5.3 minor version has bumped snakeyaml-engine
ruby/psych#762. This is causing issues in bundler.
Constrain the psych gem to the 5.2 minor version while we are still on jruby
9.4.13.0.

(cherry picked from commit a089904)
@mergify mergify bot mentioned this pull request Dec 17, 2025
Merged
mashhurs pushed a commit that referenced this pull request Dec 17, 2025
@mergify @donoghuc
Keep psych minor version in line with jruby 9.4.13.0 (#18507) (#18508)
a55bc73 
The latest psych 5.3 minor version has bumped snakeyaml-engine
ruby/psych#762. This is causing issues in bundler.
Constrain the psych gem to the 5.2 minor version while we are still on jruby
9.4.13.0.

(cherry picked from commit a089904)

Co-authored-by: Cas Donoghue <[email protected]>
donoghuc added a commit that referenced this pull request Dec 18, 2025
@donoghuc
Revert "Keep psych minor version in line with jruby 9.4.13.0 (#18507)"
7ee5c8a 
This reverts commit a089904.
@donoghuc donoghuc mentioned this pull request Dec 18, 2025
Merged
donoghuc added a commit that referenced this pull request Dec 18, 2025
@donoghuc
Revert "Keep psych minor version in line with jruby 9.4.13.0 (#18507) (…
89b507f 
…#18508)"

This reverts commit a55bc73.
donoghuc added a commit that referenced this pull request Dec 18, 2025
@donoghuc
Revert "Keep psych minor version in line with jruby 9.4.13.0 (#18507)" (
cf1eee3 
#18516)

This reverts commit a089904.
mashhurs pushed a commit that referenced this pull request Dec 18, 2025
@donoghuc
Revert "Keep psych minor version in line with jruby 9.4.13.0 (#18507) (…
55eb62b 
…#18508)" (#18517)

This reverts commit a55bc73.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mashhurs mashhurs mashhurs approved these changes

Assignees

No one assigned

Labels

backport-9.3 Automated backport to the 9.3 branch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@donoghuc @elasticmachine @mashhurs