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
ceph: 14.2.4 -> 14.2.5 #77167
ceph: 14.2.4 -> 14.2.5 #77167
Conversation
Ready for review. I'll hopefully be able to deploy this to a test cluster soonish, but I think it's good from testing in VMs. There's a critical bug in the current 14.2.4 version which may cause OSD corruption when bluestore WALs are on separate devices. Note that the multi node test currently fails because of a regression in the python test suite which causes files to no longer be shared among VMs in |
Oh, and I'll probably set up a full backport of floklis fixes to ceph/rocksdb and this, if this passes review, since we really need to get this version in stable as well, and the changes should be safe. :) |
cc @7c6f434c w.r.t |
Hm, maybe we should split the shared reference into internal_shared and external_shared then? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't really see anything wrong with the changes. Thumbs up from me!
Motivation for this change
Fixes a critical integrity bug n 14.2.3 and 14.2.4. Adds newly required deps on libnl and libcap_ng.
This needs backporting to 19.09, but I think we should also backport the various fixes to rocksdb etc. before doing so. I can author that.
WIP while my poor laptop chugs away at building it...
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @