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

[Backport release-24.11] ceph: 19.2.0 -> 19.2.1 #388087

Merged

Conversation

benaryorg
Copy link
Contributor

@benaryorg benaryorg commented Mar 8, 2025

Manual backport of #381075 including several cherry-picks and one commit with partial picks to keep the code in sync with master, which, looking at the current date is probably not going to be useful to anyone as a new NixOS version is more likely to hit before another Ceph version.
Either way, this is the exact version that I have already tested in the original PR on 24.11, minus the trustme version, since that one is the current one in 24.11.

This is currently a draft, just so that nobody else starts working on the same backport while my tower is still compiling Ceph (and the linux kernel that's not built by upstream hydra, but which is required to run the tests on this commit).
I'll undraft it as soon as those tests are done.

@benaryorg benaryorg mentioned this pull request Mar 8, 2025
14 tasks
@github-actions github-actions bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 labels Mar 8, 2025
@benaryorg
Copy link
Contributor Author

The real machine works just fine, and the 4 NixOS Tests are passing too (again, only x86_64-linux tested).

@benaryorg benaryorg marked this pull request as ready for review March 8, 2025 00:54
@benaryorg benaryorg requested a review from nh2 March 8, 2025 00:54
@nix-owners nix-owners bot requested review from johanot, adevress and krav March 8, 2025 01:03
@benaryorg benaryorg force-pushed the backport-381075-to-release-24.11 branch from a3453fa to 60961a9 Compare March 18, 2025 18:08
@benaryorg
Copy link
Contributor Author

Rebase to make sure the checks are still fine and all that.…
Since I pulled a lot of stuff over from master one could review just the diff to that:

git fetch origin master pull/388087/head
git diff origin/master origin/pull/388087/head -- pkgs/tools/filesystems/ceph/

It's pretty much everything except for c314e80 (as per last commit message), plus it's been running on my machine for a while.

@nh2: whenever you have some time to spare, I'd appreciate the review ^^

@wolfgangwalther
Copy link
Contributor

Since I pulled a lot of stuff over from master one could review just the diff to that:

The proper way to do it, is to cherry-pick each commit from master with -x. Then the "check cherry-picks" job will confirm that the picks are the same as on master.

This is also documented: https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#manually-backporting-changes

Makes it a lot easier to review.

(needs a rebase anyway, to trigger eval once more)

nh2 and others added 8 commits March 23, 2025 23:49
Closes: NixOS#369777

(cherry picked from commit eef74a1)
Currently this is the only package keeping Boost 1.82 around; upstream
install 1.85 on the development branch and have merged patches for
1.86 compatibility, so let’s unpin it and deal with pinning again
if we run into compilation errors in the future.

All but one patch has already been merged upstream.

(cherry picked from commit 047670d)
(cherry picked from commit 6b77b73)
We surely don't need docs in the overridden pyopenssl.
https://hydra.nixos.org/build/287563868/nixlog/1/tail

(cherry picked from commit 7e259ce)
Honestly, I don't know.  All so ugly.
Hopefully we don't need much of pyopenssl here anyway.
https://hydra.nixos.org/build/290186587/nixlog/7/tail

(cherry picked from commit 61da3c0)
Had to downgrade the used boost186 -> boost183, see added comment.

(cherry picked from commit f2d3155)
Otherwise changes will keep causing merge conflicts.
Given that the master branch was already tested on 24.11 in the original PR it should be fine.

Includes partials from:

- cace2b6
- 5aba992

Explicitly not picked was c314e80 since the *trustme* version is still present in 24.11.

Signed-off-by: benaryorg <binary@benary.org>
@benaryorg benaryorg force-pushed the backport-381075-to-release-24.11 branch from 60961a9 to df76780 Compare March 23, 2025 23:52
@benaryorg
Copy link
Contributor Author

The proper way to do it, is to cherry-pick each commit from master with -x. Then the "check cherry-picks" job will confirm that the picks are the same as on master.

Aight, thank you very much, should be fine now, I hope, given I used interactive rebase with rewording to add the line myself, though I did diff every single commit to make sure nothing but the added line in the commit message as well as potentially line numbers changed.

(needs a rebase anyway, to trigger eval once more)

Done.

@wolfgangwalther
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 388087


x86_64-linux

✅ 17 packages built:
  • ceph (ceph-dev)
  • ceph-client
  • ceph-csi
  • ceph.dev (ceph-dev.dev)
  • ceph.doc (ceph-dev.doc)
  • libceph (ceph.lib, libceph.dev, libceph.doc, libceph.lib, libceph.man)
  • ceph.man (ceph-dev.man)
  • qemu_full
  • qemu_full.debug
  • qemu_full.ga
  • quickemu
  • quickgui
  • quickgui.debug
  • quickgui.pubcache
  • sambaFull (samba4Full)
  • sambaFull.dev (samba4Full.dev)
  • sambaFull.man (samba4Full.man)

aarch64-linux

✅ 10 packages built:
  • ceph (ceph-dev)
  • ceph-client
  • ceph-csi
  • ceph.dev (ceph-dev.dev)
  • ceph.doc (ceph-dev.doc)
  • libceph (ceph.lib, libceph.dev, libceph.doc, libceph.lib, libceph.man)
  • ceph.man (ceph-dev.man)
  • qemu_full
  • qemu_full.ga
  • quickemu

@wolfgangwalther wolfgangwalther merged commit 8ddb5bb into NixOS:release-24.11 Mar 24, 2025
25 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants