Skip to content

pybind/mgr: call new _ceph_exit for killpoints#67632

Merged
batrick merged 2 commits intoceph:mainfrom
batrick:i74605
Mar 18, 2026
Merged

pybind/mgr: call new _ceph_exit for killpoints#67632
batrick merged 2 commits intoceph:mainfrom
batrick:i74605

Conversation

@batrick
Copy link
Member

@batrick batrick commented Mar 3, 2026

Contribution Guidelines

  • To sign and title your commits, please refer to Submitting Patches to Ceph.

  • If you are submitting a fix for a stable branch (e.g. "quincy"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.

  • When filling out the below checklist, you may click boxes directly in the GitHub web UI. When entering or editing the entire PR message in the GitHub web UI editor, you may also select a checklist item by adding an x between the brackets: [x]. Spaces and capitalization matter when checking off items this way.

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands

You must only issue one Jenkins command per-comment. Jenkins does not understand
comments with more than one command.

@tchaikov
Copy link
Contributor

tchaikov commented Mar 4, 2026

Cloning into '/ceph/src/c-ares'...
remote: Internal Server Error
fatal: unable to access 'https://github.com/ceph/c-ares.git/': The requested URL returned error: 500
fatal: clone of 'https://github.com/ceph/c-ares.git' into submodule path '/ceph/src/c-ares' failed
Failed to clone 'src/c-ares'. Retry scheduled
Cloning into '/ceph/src/crypto/isa-l/isa-l_crypto'...
Cloning into '/ceph/src/erasure-code/jerasure/gf-complete'...
Cloning into '/ceph/src/erasure-code/jerasure/jerasure'...
Cloning into '/ceph/src/fmt'...
Cloning into '/ceph/src/googletest'...
Cloning into '/ceph/src/isa-l'...
remote: Internal Server Error
fatal: unable to access 'https://github.com/ceph/isa-l/': The requested URL returned error: 500
fatal: clone of 'https://github.com/ceph/isa-l' into submodule path '/ceph/src/isa-l' failed

github returned 500..

@tchaikov
Copy link
Contributor

tchaikov commented Mar 4, 2026

jenkins test make check

@tchaikov
Copy link
Contributor

tchaikov commented Mar 4, 2026

jenkins test make check arm64

@tchaikov
Copy link
Contributor

tchaikov commented Mar 4, 2026

jenkins test windows

Copy link
Contributor

@NitzanMordhai NitzanMordhai left a comment

Choose a reason for hiding this comment

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

test passed on teuthology!

@batrick
Copy link
Member Author

batrick commented Mar 4, 2026

rebased to fix tox failures

batrick added 2 commits March 5, 2026 09:10
So we can work around modules disabling calls to sys._exit

Fixes: https://tracker.ceph.com/issues/74605
Fixes: 78983ad
Fixes: ceph/ceph.git#66244
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
Fixes: https://tracker.ceph.com/issues/74605
Fixes: 78983ad
Fixes: ceph/ceph.git#66244
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
@batrick
Copy link
Member Author

batrick commented Mar 5, 2026

another tox failure to fix

@batrick
Copy link
Member Author

batrick commented Mar 6, 2026

This PR is under test in https://tracker.ceph.com/issues/75377.

@batrick batrick merged commit 2de737f into ceph:main Mar 18, 2026
13 checks passed
@batrick batrick deleted the i74605 branch March 18, 2026 17:56
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.

3 participants