-
Notifications
You must be signed in to change notification settings - Fork 1
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 17.2.4 doesnt build #1
Comments
I don't build with I'm actively testing it in a test env at the moment, so you might want to take a look at what has been done there. You can take it and experiment with setting the select define if you like. See: feature/v17.2.4 |
I also had an compiler error with your feature branch
|
on a second try to build it worked, I have some aarch64 packages ready to test :) |
OSD crashes
|
That's surprising, I have two test clusters I've been doing perf benchmarking using the most recent iteration of this branch. Are you using the most recent HEAD? I tend to do a lot of rebase/force-push-ing on my feature branches. I'll rebuild the packages today in a clean chroot just to make sure |
You probably encountered another part of the racy legacy header generation that the upstream does since quincy. See https://github.com/bazaah/aur-ceph/blob/feature/v17.2.4/ceph-17.2.4-compressor-common-depends.patch for another example. Fortunately, the fix is pretty simple. I'll probably add a patch for that in a |
rebuilt + tested, still no issues. Unless you can provide me with a reproducible test failure, I'm going to chalk this up to environment. |
well, I nuked the failing OSD and recreated it on the same disk, the rest seems to be working fine, thank you so much for providing your work here. My cluster only consists of aarch64 machines (odroids, raspberries, and others), and the produced binaries work across all of them. Just out of curiosity, how much hassle could we have saved by using the containerized approach for ceph? |
between me and the storage layer for my whole org. Just cranky sysadmin behavior, I guess. Besides, for my specific usecase having an actual |
I'm closing this as I finally got a clean Make check stdoutctest 100% tests passed, 0 tests failed out of 228 Total Test time (real) = 3038.84 sec |
Hi,
my cluster is already running ceph 17.2.4 and since some arch upgrades, some dependency libraries were updated too and I cannot rebuild ceph anymore. I know this repo only builds ceph 16, but maybe you have an idea why it doesnt build with the newer version anymore?
would a c-style cast do the trick here?
The text was updated successfully, but these errors were encountered: