-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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] Irmin project (3.7.1) #23749
Conversation
…min-pack, irmin-pack-tools, irmin-mirage, irmin-mirage-graphql, irmin-mirage-git, irmin-http, irmin-graphql, irmin-git, irmin-fs, irmin-containers, irmin-cli, irmin-chunk and irmin-bench (3.7.1) CHANGES: ### Fixed - **irmin-pack** - Fix issue when migrating v2 stores to use lower layer (@metanivek, mirage/irmin#2241) - Fix issue when calling GC for a commit in the lower after migration (@metanivek, mirage/irmin#2242)
CI failures appear unrelated. I think this is good for merging. |
The |
Thanks for merging! It appears the CI failures are isolated to CI failure pattern
I have been able to reproduce locally with the following (from the CI log for Steps for CI reproduction from log
|
Don't worry about the opam solver failures. These are just for opam 2.0 and are a known issue in the old versions of opam. The failure I am talking about was in https://toxis.caelum.ci.dev/github/ocaml/opam-repository/commit/46c94c9fa6333ca11246200dde327fbee1cc9b4c/variant/extras,arm32-ocaml-4.14,irmin-http.3.7.1,tests This is now cancelled due to the merge, but it was cancelled twice because it hung for 2 hours either testing irmin-git or irmin-http. |
CHANGES:
Fixed
(@metanivek, irmin-pack: fix GC when target commit is in lower mirage/irmin#2242)