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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

ceph: 18.2.0 -> 18.2.1 #281474

Merged
merged 1 commit into from
Feb 6, 2024
Merged

ceph: 18.2.0 -> 18.2.1 #281474

merged 1 commit into from
Feb 6, 2024

Conversation

dotlambda
Copy link
Member

@dotlambda dotlambda commented Jan 17, 2024

Description of changes

Changelog: https://docs.ceph.com/en/latest/releases/reef/#v18-2-1-reef

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 馃憤 reaction to pull requests you find important.

@dotlambda
Copy link
Member Author

Result of nixpkgs-review pr 281474 run on x86_64-linux 1

13 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
  • sambaFull (samba4Full)
  • sambaFull.dev (samba4Full.dev)
  • sambaFull.man (samba4Full.man)

@dotlambda
Copy link
Member Author

@ofborg build ceph ceph.passthru.tests

Copy link
Contributor

@nh2 nh2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changelog looks good, ofborg is happy (ran NixOS tests).

What's not working is aarch64-linux:

The Python library watchfiles-0.21.0 seems to timeout in a test on this architecture:

error: builder for '/nix/store/kgy19r4gl6bcg1jjla9rvwlpgcfw77qj-python3.10-watchfiles-0.21.0.drv' failed with exit code 1;
       last 10 log lines:
       >         path = os.getenv('WATCHFILES_TEST_PERMISSION_DENIED_PATH') or '/'
       >
       > >       RustNotify([path], False, False, 0, True, True)
       > E       Failed: Timeout >10.0s
       >
       > tests/test_rust_notify.py:317: Failed
       > =========================== short test summary info ============================
       > FAILED tests/test_rust_notify.py::test_ignore_permission_denied - Failed: Timeout >10.0s
       > ====== 1 failed, 150 passed, 2 skipped, 1 deselected in 80.39s (0:01:20) =======
       > /nix/store/in5b9lna5p97wak30r8397645akl5092-stdenv-linux/setup: line 1582: pop_var_context: head of shell_variables not a function context

It does seem to build on Hydra with Python 3.11: https://hydra.nixos.org/build/247427896

I don't have an aarch64 builder so I can't easily investigate this.

I'm approving this PR under the assumption that this failure has nothing to do with Ceph.

@nh2
Copy link
Contributor

nh2 commented Jan 29, 2024

@GrahamcOfBorg build ceph ceph.passthru.tests

Just in case that test is flaky and works upon rebuild.

@fpletz
Copy link
Member

fpletz commented Jan 30, 2024

Strangely, ceph builds for me on aarch64-linux but the NixOS tests fail due to missing python modules on some services.

@nh2
Copy link
Contributor

nh2 commented Jan 31, 2024

but the NixOS tests fail due to missing python modules on some services.

@fpletz Can you post the output?

@nh2
Copy link
Contributor

nh2 commented Feb 5, 2024

I force-pushed the PR to rebase on the merged PR #281858 with my Ceph GCC fixes.

@nh2
Copy link
Contributor

nh2 commented Feb 5, 2024

@GrahamcOfBorg build ceph ceph.passthru.tests

@srhb
Copy link
Contributor

srhb commented Feb 6, 2024

Looks good to me. I can try to get at an aarch64-linux if we think there's truly a problem?

@srhb
Copy link
Contributor

srhb commented Feb 6, 2024

By the way, I also see (racy) failures of ceph-mgr modules (also in prod, sadly) which seem related to the internal subinterpreter struggle, but worsened by pyo3 updates, but I've yet to see a complete test failure because of it. That may just be nondeterminism in the mgr module ordering though, I suppose.

@nh2
Copy link
Contributor

nh2 commented Feb 6, 2024

Ofborg now succeeds on aarch64-linux:

https://github.com/NixOS/nixpkgs/pull/281474/checks?check_run_id=21247371804

I upgraded a test cluster from 16.2.7 to 18.2.1 based on this commit, so this is good enough to merge now.

By the way, I also see (racy) failures of ceph-mgr modules (also in prod, sadly) which seem related to the internal subinterpreter struggle, but worsened by pyo3 updates, but I've yet to see a complete test failure because of it. That may just be nondeterminism in the mgr module ordering though, I suppose.

@srhb Thanks for the info!

I hope Ceph fixes this issue.

Personally I enjoy that I saw the Ceph dashboard for the first time yesterday on my cluster on 18.2.1 :)

@nh2 nh2 merged commit 2d927a6 into NixOS:master Feb 6, 2024
22 of 23 checks passed
@srhb
Copy link
Contributor

srhb commented Feb 7, 2024

We should backport this, right?

@dotlambda dotlambda deleted the ceph-18.2.1 branch February 7, 2024 07:35
@nh2 nh2 added the backport release-23.11 Backport PR automatically label Feb 7, 2024
@nh2
Copy link
Contributor

nh2 commented Feb 7, 2024

We should backport this, right?

I think so; I added the tag.

Copy link
Contributor

github-actions bot commented Feb 7, 2024

Successfully created backport PR for release-23.11:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants