Skip to content

Fix the actions CI#1107

Merged
koic merged 6 commits intobblimke:masterfrom
misdoro:master
Oct 22, 2025
Merged

Fix the actions CI#1107
koic merged 6 commits intobblimke:masterfrom
misdoro:master

Conversation

@misdoro
Copy link
Contributor

@misdoro misdoro commented Oct 20, 2025

Fix the CI:

Fixes #1045

httpstat.us is currently inoperative, run a local service in CI instead.

NB:
Run as a container to allow service access by `http://httpstatus/` URL.

Need to install ruby build and runtime dependencies to make `setup-ruby` action work
`https://github.com/jruby/jruby/issues/8606` seem to be resolved, we should be able to enable back jruby-head.

Install jdk for jruby-head only

Require base64 on jruby
@misdoro misdoro changed the title Fix actions CI Mostly fix the actions CI Oct 20, 2025
@koic
Copy link
Collaborator

koic commented Oct 21, 2025

Thank you for working on stabilizing the CI.
The re-enabled jruby-head matrix is still failing, and ruby-head matrix is also failing. What is the current status of how these are being handled in this PR?

@misdoro
Copy link
Contributor Author

misdoro commented Oct 21, 2025

Thank you for working on stabilizing the CI. The re-enabled jruby-head matrix is still failing, and ruby-head matrix is also failing. What is the current status of how these are being handled in this PR?

The -head checks were configured as 'allowed to fail' previously in the ci. I could try to fix them too but it will take some time.

  • Ruby-head Installation is not working as expected inside a docker image, I'll likely need to tune the workflow to build ruby from source in that case.
  • jruby has some tests failing, I haven't looked in details yet.

@misdoro
Copy link
Contributor Author

misdoro commented Oct 21, 2025

@koic In the end, ruby-head needed a trivial fix, defining a HOME corresponding to what setup-ruby action expects.

Jruby implementation seem to have some issues with httpclient acceptance test concurrent/sequential requests specifically. I've marked those tests to be skipped on jruby for now.

@misdoro misdoro changed the title Mostly fix the actions CI Fix the actions CI Oct 21, 2025
@koic koic merged commit 2cd79fa into bblimke:master Oct 22, 2025
9 checks passed
@koic
Copy link
Collaborator

koic commented Oct 22, 2025

Fantastic work! Thank you @misdoro!

@bblimke
Copy link
Owner

bblimke commented Oct 26, 2025

Thank you @misdoro . Awesome work on bringing CI to be green again.

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.

httpstat.us site used by many tests is defunct

3 participants