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

Update dependencies and remove capnp-rpc-mirage #276

Merged
merged 3 commits into from
Sep 26, 2024

Conversation

talex5
Copy link
Contributor

@talex5 talex5 commented Sep 25, 2024

Update for breaking changes in tls, mirage-crypto and mirage-flow.

capnp-rpc-mirage is gone for now as there were further changes needed there and once the conversion to Eio is done we won't need a separate API for unikernels anyway.

Update for breaking changes in tls, mirage-crypto and mirage-flow.

capnp-rpc-mirage is gone for now as there were further changes needed
there and once the conversion to Eio is done we won't need a separate
API for unikernels anyway.
@talex5 talex5 changed the title Update dependencies Update dependencies and remove capnp-rpc-mirage Sep 25, 2024
@talex5 talex5 force-pushed the update-deps branch 2 times, most recently from 814187b to 62f734c Compare September 25, 2024 14:33
- In `sturdy-refs-3`, `for_alice` wasn't explicitly released. If it gets
  GC'd while the connection is still up, it logs a warning. This was
  causing CI to fail in some cases.

- The restorer table takes ownership of the reference (back-port
  from Eio branch). Ensure vats are shutdown cleanly to make errors
  easier to find (Eio does this automatically).
@talex5
Copy link
Contributor Author

talex5 commented Sep 25, 2024

Also added a couple of commits to fix flaky tests (bad ref-counting could result in warnings being logged, which MDX flags as an error; and close can return ECONNRESET on FreeBSD).

The only remaining CI failure is on x86_32, where Debian stable's old version of the capnproto compiler fails if it gets an inode number that doesn't fit in 32-bits.

@talex5 talex5 merged commit ee0686b into mirage:master Sep 26, 2024
2 of 3 checks passed
@talex5 talex5 deleted the update-deps branch September 26, 2024 09:12
talex5 added a commit to talex5/opam-repository that referenced this pull request Nov 28, 2024
CHANGES:

- Update dependencies and remove capnp-rpc-mirage (@talex5 mirage/capnp-rpc#276).
  - Update for breaking changes in tls, mirage-crypto and mirage-flow.
  - capnp-rpc-mirage isn't used much, and won't be needed after switching to Eio.

- Update tests to Cap'n Proto 1.0.1 (@MisterDA mirage/capnp-rpc#274).

- Update links to ocaml-ci and capnp-ocaml (@tmcgilchrist mirage/capnp-rpc#271).

- Opam doesn't allow with-test in conflicts (@talex5 mirage/capnp-rpc#269).

- Disable opam tests on macos (@talex5 mirage/capnp-rpc#268).

- Update CI (@MisterDA mirage/capnp-rpc#272, @talex5 mirage/capnp-rpc#277).
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.

1 participant