ocaml-dockerfile icon indicating copy to clipboard operation
ocaml-dockerfile copied to clipboard

Published 20 hours ago • verified publisher icon ocurrent

Unset the jobs variable from the opam config (faulty opam 2.0 behaviour)

Open kit-ty-kate opened this issue 3 years ago • 4 comments

opam 2.0 by default stores the number of cores of the current machine. While not too much of a big deal for users, this is extremely bad for the base images where the number stored will be around 72 but these can be used on any other machines.

This might help fighting the IO overload on the cluster.

kit-ty-kate avatar Mar 22 '22 17:03 kit-ty-kate

The problem doing this in the base is that it breaks 2.0 - opam 2.1 recomputes the jobs variable if it's missing, but opam 2.0 just assumes "1". I think this wants to be in workflows as part of the root upgrade step?

dra27 avatar Sep 08 '22 07:09 dra27

@kit-ty-kate can this be merged now that the opam PR is merged?

benmandrew avatar Jan 23 '24 11:01 benmandrew

@benmandrew the opam PR only fixes things when upgrading to opam 2.2. David’s comment above is still very much the main blocker, so until the images are upgraded to a more recent opam by default this PR can’t be merged.

kit-ty-kate avatar Jan 23 '24 13:01 kit-ty-kate

iiuc, addressing https://github.com/ocurrent/docker-base-images/issues/132 would unblock this.

shonfeder avatar Aug 26 '24 14:08 shonfeder