Skip to content

Conversation

@baum
Copy link
Collaborator

@baum baum commented Nov 20, 2025

tests: add connect panic test and update late registration test

(cherry picked from commit acc7f9f)

ceph shaman: https://shaman.ceph.com/builds/ceph/squid-nvmeof/49859cdd4088a3d59a81919864b3ecabf441f4f0/

- related to monitor client connect panic from ceph/ceph#64713,  late registrationi test, cap random sleep at 25 seconds to stay safely below monitor connect timeout
- Add new connect_panic.sh test to validate monitor client timeout behavior

Signed-off-by: Alexander Indenbaum <[email protected]>
(cherry picked from commit acc7f9f)
@baum baum force-pushed the connect_panic_squid_8.1 branch from 05ef81b to 19d575d Compare November 20, 2025 15:56
Copy link
Collaborator

@gadididi gadididi left a comment

Choose a reason for hiding this comment

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

LGTM!

@github-project-automation github-project-automation bot moved this from 🆕 New to 👍 Ready to merge in NVMe-oF Nov 23, 2025
@baum baum merged commit d3824c0 into ceph:squid_8.1 Nov 23, 2025
116 of 120 checks passed
@github-project-automation github-project-automation bot moved this from 👍 Ready to merge to ✅ Done in NVMe-oF Nov 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants