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

[new release] ocluster, ocluster-worker, ocluster-api and current_ocluster (0.2.1) #23443

Merged
merged 1 commit into from
Mar 3, 2023

Conversation

MisterDA
Copy link
Contributor

@MisterDA MisterDA commented Mar 2, 2023

Distribute build jobs to workers

CHANGES:

@MisterDA MisterDA marked this pull request as draft March 2, 2023 18:35
@kit-ty-kate
Copy link
Member

kit-ty-kate commented Mar 2, 2023

The tests fail with:

#=== ERROR while compiling ocluster-worker.0.2.1 ==============================#
# context              2.2.0~alpha~dev | linux/x86_64 | ocaml-base-compiler.4.14.1 | pinned(https://github.com/ocurrent/ocluster/releases/download/v0.2.1/ocluster-0.2.1.tbz)
# path                 ~/.opam/4.14/.opam-switch/build/ocluster-worker.0.2.1
# command              ~/.opam/opam-init/hooks/sandbox.sh build dune build -p ocluster-worker -j 47 @install @runtest
# exit-code            1
# env-file             ~/.opam/log/ocluster-worker-7-3bfdf1.env
# output-file          ~/.opam/log/ocluster-worker-7-3bfdf1.out
### output ###
# File "test/dune", line 21, characters 19-29:
# 21 |   (preprocess (pps ppx_expect))
#                         ^^^^^^^^^^
# Error: Library "ppx_expect" not found.
# -> required by library "ocluster_expect_tests" in _build/default/test
# -> required by
#    _build/default/test/.ocluster_expect_tests.inline-tests/inline_test_runner_ocluster_expect_tests.exe
# -> required by alias test/runtest in test/dune:20

@MisterDA
Copy link
Contributor Author

MisterDA commented Mar 2, 2023

yep, it's another library that's not attached to the right package…

@MisterDA MisterDA marked this pull request as ready for review March 3, 2023 08:01
@MisterDA
Copy link
Contributor Author

MisterDA commented Mar 3, 2023

Seems to be fixed now!
EDIT: apparently the tests hang, I'm looking at it.

…uster (0.2.1)

CHANGES:

- Expose the ocluster-worker library in the ocluster-worker package
  (@MisterDA @art-w, ocurrent/ocluster#219 ocurrent/ocluster#217 ocurrent/ocluster#151, reviewed by @tmcgilchrist)
- Remove corrupted repositories from the cache (@kit-ty-kate ocurrent/ocluster#216, reviewed by @talex5)
- Allow workers to report additional prometheus metrics (@patricoferris ocurrent/ocluster#210, reviewed by @tmcgilchrist, @MisterDA)
- Smother Cap'n Proto and TLS debug logs (@MisterDA ocurrent/ocluster#213, reviewed by @talex5)
- Added command line option to set obuilder health check period (@mtelvers ocurrent/ocluster#214, reviewed by @tmcgilchrist)
- Conditionally compile macos user_temp fetcher (@tmcgilchrist ocurrent/ocluster#209, reviewed by @MisterDA, @mtelvers)
- Make rsync-mode mandatory when using rsync store (@tmcgilchrist ocurrent/ocluster#202, reviewed by @MisterDA)
- Windows service bugfixes (@MisterDA ocurrent/ocluster#200, reviewed by @tmcgilchrist)
- Fix build and opam metadata (@MisterDA @tmcgilchrist ocurrent/ocluster#199 ocurrent/ocluster#203)
@MisterDA
Copy link
Contributor Author

MisterDA commented Mar 3, 2023

Finally! I think I've fixed all the problems. Some private libraries were not correctly attached to the ocluster package, and the tests were missing the new mirage-crypto-rng-lwt.

@kit-ty-kate
Copy link
Member

Thanks

@kit-ty-kate kit-ty-kate merged commit dfda1be into ocaml:master Mar 3, 2023
@MisterDA MisterDA deleted the release-ocluster-v0.2.1 branch March 3, 2023 19:07
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