Skip to content

Conversation

@kahgoh
Copy link
Member

@kahgoh kahgoh commented Oct 31, 2025

  • This updates the test runner image to OCaml 5.4.
  • According to Docker hub, the previous Alpine 3.18 image has not been updated in over 2 years. With the previous base image, OPAM could only get v0.16.3 of Base even though v0.17.3 was available.
  • Change to a "no flat float array" image which is a smaller image. This disables OCaml's optimizations for using float arrays, but does not prevent it from being used. It is believed the track does not make much use of float arrays.
  • Also, switched to Debian 13 as the base image for Debian 13 is smaller than Alpine 3.22. Docker Hub shows the Debian 13 image as 617.49 MB and the Alpine 3.22 as 676.29 MB.

This is also for https://forum.exercism.org/t/ocaml-tests-failing-with-segfault-on-site-but-pass-locally/19744/9

- This updates the test runner image to OCaml 5.4.
- According to Docker hub, the previous Alpine 3.18 image has not been
  updated in over 2 years. With the previous base image, OPAM could
  only get v0.16.3 of Base even though v0.17.3 was available.
- Change to a "no flat float array" image which is a smaller image.
  This disables OCaml's optimizations for using float arrays, but does
  not prevent it from being used. It is believed the track does not
  make much use of float arrays.
@kahgoh kahgoh requested a review from a team as a code owner October 31, 2025 04:56
@kahgoh
Copy link
Member Author

kahgoh commented Oct 31, 2025

Just a quick note, I've noticed the CI has failed and currently looking at it.

@kahgoh kahgoh marked this pull request as draft October 31, 2025 05:17
@kahgoh kahgoh force-pushed the ocaml-5.4 branch 4 times, most recently from 9fb3b65 to a261e3c Compare November 1, 2025 08:26
The Debian 13 version base image seems smaller than the Alpine one.
Might give us a smaller image.
@kahgoh
Copy link
Member Author

kahgoh commented Nov 1, 2025

CI passes now, so ready for review @exercism/guardians

@kahgoh kahgoh marked this pull request as ready for review November 1, 2025 08:31
@kahgoh kahgoh merged commit 291fcfa into exercism:main Nov 1, 2025
1 check passed
@kahgoh kahgoh deleted the ocaml-5.4 branch November 1, 2025 23:30
kahgoh added a commit to kahgoh/exercism-ocaml that referenced this pull request Nov 1, 2025
This is to reflect the update to the test runner in
exercism/ocaml-test-runner#68.
georgyo pushed a commit to exercism/ocaml that referenced this pull request Nov 28, 2025
* Update docs for OCaml 5.4

This is to reflect the update to the test runner in
exercism/ocaml-test-runner#68.

* Update Docker image to OCaml 5.4

This is to match the version used by the test runner.
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