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

MacOSX CI sometimes gets KILL'ed during ppx_lib install #182

Closed
jmid opened this issue Nov 11, 2022 · 5 comments · Fixed by #198
Closed

MacOSX CI sometimes gets KILL'ed during ppx_lib install #182

jmid opened this issue Nov 11, 2022 · 5 comments · Fixed by #198

Comments

@jmid
Copy link
Collaborator

jmid commented Nov 11, 2022

Also from #88:

Originally posted by @jmid in #88 (comment)

We have observed this previously in #157 and #162
CC: @shym

@shym
Copy link
Collaborator

shym commented Nov 23, 2022

Should be fixed since #198.

@jmid jmid linked a pull request Nov 23, 2022 that will close this issue
@jmid jmid closed this as completed Nov 23, 2022
@jmid
Copy link
Collaborator Author

jmid commented Nov 24, 2022

Dang, reopening as this just happened again
https://github.com/jmid/multicoretests/actions/runs/3538600388/jobs/5939594832

Processing 46/57: [ppxlib: dune build]
+ /Users/runner/.opam/opam-init/hooks/sandbox.sh "build" "dune" "build" "-p" "ppxlib" "-j" "1" "@install" (CWD=/Users/runner/work/multicoretests/multicoretests/_opam/.opam-switch/build/ppxlib.0.28.0)
- File "ast/dune", line 16, characters 5-58:
- 16 |      (run %{exe:pp/pp.exe} %{ocaml_version} %{input-file}))
-           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
- (cd _build/default && ast/pp/pp.exe 5.0.0+dev8-2022-10-12 ast/versions.ml) > _build/default/ast/versions.pp.ml
- Command got signal KILL.
[ERROR] The compilation of ppxlib.0.28.0 failed at "dune build -p ppxlib -j 1 @install".
Processing 47/57: [domainslib: dune build]
Processing 48/57: [domainslib: dune build]
+ /Users/runner/.opam/opam-init/hooks/sandbox.sh "build" "dune" "build" "-p" "domainslib" "-j" "1" (CWD=/Users/runner/work/multicoretests/multicoretests/_opam/.opam-switch/build/domainslib.0.5.0)
λ compiled  domainslib.0.5.0
Processing 50/57: [qcheck-ounit: dune build]
∗ installed domainslib.0.5.0
Processing 51/57: [qcheck-ounit: dune build]
Processing 52/57: [qcheck-ounit: dune build]
+ /Users/runner/.opam/opam-init/hooks/sandbox.sh "build" "dune" "build" "-p" "qcheck-ounit" "-j" "1" (CWD=/Users/runner/work/multicoretests/multicoretests/_opam/.opam-switch/build/qcheck-ounit.0.20)
λ compiled  qcheck-ounit.0.20
∗ installed qcheck-ounit.0.20
Processing 54/57: [qcheck: dune build]
+ /Users/runner/.opam/opam-init/hooks/sandbox.sh "build" "dune" "build" "-p" "qcheck" "-j" "1" (CWD=/Users/runner/work/multicoretests/multicoretests/_opam/.opam-switch/build/qcheck.0.20)
λ compiled  qcheck.0.20
∗ installed qcheck.0.20

#=== ERROR while compiling ppxlib.0.28.0 ======================================#
# context     2.1.3 | macos/x86_64 | ocaml-variants.5.0.0+trunk | git+https://github.com/ocaml/opam-repository.git
# path        ~/work/multicoretests/multicoretests/_opam/.opam-switch/build/ppxlib.0.28.0
# command     ~/.opam/opam-init/hooks/sandbox.sh build dune build -p ppxlib -j 1 @install
# exit-code   1
# env-file    ~/.opam/log/ppxlib-52187-22a8ae.env
# output-file ~/.opam/log/ppxlib-52187-22a8ae.out
### output ###
# File "ast/dune", line 16, characters 5-58:
# 16 |      (run %{exe:pp/pp.exe} %{ocaml_version} %{input-file}))
#           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
# (cd _build/default && ast/pp/pp.exe 5.0.0+dev8-2022-10-12 ast/versions.ml) > _build/default/ast/versions.pp.ml
# Command got signal KILL.

@jmid jmid reopened this Nov 24, 2022
@jmid
Copy link
Collaborator Author

jmid commented Dec 21, 2022

We just hit this again on macOS 5.1.0+trunk
https://github.com/ocaml-multicore/multicoretests/actions/runs/3751523987/jobs/6372603387

#=== ERROR while compiling ppxlib.0.28.0 ======================================#
# context     2.1.4 | macos/x86_64 | ocaml-variants.5.1.0+trunk | pinned(git+https://github.com/shym/ppxlib.git#5.1#96e453d0)
# path        ~/work/multicoretests/multicoretests/_opam/.opam-switch/build/ppxlib.0.28.0
# command     ~/.opam/opam-init/hooks/sandbox.sh build dune build -p ppxlib -j 3 @install
# exit-code   1
# env-file    ~/.opam/log/ppxlib-53889-4d8d39.env
# output-file ~/.opam/log/ppxlib-53889-4d8d39.out
### output ###
# File "src/dune", line 33, characters 0-143:
# 33 | (rule
# 34 |  (targets ast_builder_generated.ml)
# 35 |  (deps gen/gen_ast_builder.exe)
# 36 |  (action
# 37 |   (run ./gen/gen_ast_builder.exe %{lib:ppxlib.ast:ast.ml})))
# (cd _build/default/src && gen/gen_ast_builder.exe ../../install/default/lib/ppxlib/ast/ast.ml)
# Command got signal KILL.



<><> Error report <><><><><><><><><><><><><><><><><><><><><><><><><><><><><>  🐫 
┌─ The following actions failed
│ λ build ppxlib 0.28.0
└─ 
┌─ The following changes have been performed
│ ∗ install ocaml-compiler-libs v0.12.4
│ ∗ install ppx_derivers        1.2.1
│ ∗ install sexplib0            v0.15.1
│ ∗ install stdlib-shims        0.3.0
└─ 
[NOTE] Pinning command successful, but your installed packages may be out of sync.
# Run eval $(opam env) to update the current shell environment

The former state can be restored with:
    /Users/runner/hostedtoolcache/opam/2.1.4/x86_64/opam switch import "/Users/runner/work/multicoretests/multicoretests/_opam/.opam-switch/backup/state-20221221174953.export"
'opam pin add ppxlib git+https://github.com/shym/ppxlib.git#5.1' failed.
Error: Process completed with exit code 31.
0s

@shym
Copy link
Collaborator

shym commented Dec 21, 2022

As it seems it appears a lot less when dropping parallelism during dependency builds, #278 enforces also OPAMJOBS to 1 during that build of ppxlib.

@jmid
Copy link
Collaborator Author

jmid commented Feb 1, 2023

Closing again as I don't recall having seen this in the past month

@jmid jmid closed this as completed Feb 1, 2023
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 a pull request may close this issue.

2 participants