tentacle: test/neorados: Catch timeouts in Poll test

[adamemerson]

The test is fragile against timing issues which manifest from time to time on Teuthology. Catch the exception and print an error so it's not hidden completely, but stop signaling it as a test failure.

I can do something more later, for now just stop having it bother the RADOS team.

Fixes: https://tracker.ceph.com/issues/70916
(This time for real!)

(cherry picked from commit 48b4a75)

Conflicts:
src/test/neorados/watch_notify.cc

• Different timeout value in main from previous failed fix

Fixes: https://tracker.ceph.com/issues/70916

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)
  • Includes unit test(s)
  • Includes integration test(s)
  • Includes bug reproducer
  • No tests

Show available Jenkins commands

• jenkins test classic perf Jenkins Job | Jenkins Job Definition
• jenkins test crimson perf Jenkins Job | Jenkins Job Definition
• jenkins test signed Jenkins Job | Jenkins Job Definition
• jenkins test make check Jenkins Job | Jenkins Job Definition
• jenkins test make check arm64 Jenkins Job | Jenkins Job Definition
• jenkins test submodules Jenkins Job | Jenkins Job Definition
• jenkins test dashboard Jenkins Job | Jenkins Job Definition
• jenkins test dashboard cephadm Jenkins Job | Jenkins Job Definition
• jenkins test api Jenkins Job | Jenkins Job Definition
• jenkins test docs ReadTheDocs | Github Workflow Definition
• jenkins test ceph-volume all Jenkins Jobs | Jenkins Jobs Definition
• jenkins test windows Jenkins Job | Jenkins Job Definition
• jenkins test rook e2e Jenkins Job | Jenkins Job Definition

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

[rzarzynski]

Generally LGTM. The commit description is a bit weird with the duplicated Fixes and Conflicts. Is that because the patch is being reintroduced?

[adamemerson]

Generally LGTM. The commit description is a bit weird with the duplicated Fixes and Conflicts. Is that because the patch is being reintroduced?

The second Fixes: is for the backport fix. I thought we were supposed to leave the Conflicts: section in when doing a backport if there were merge conflicts?

[adamemerson]

@rzarzynski @yuriw Can we get this in a core QA run?

[ronen-fr]

@adamemerson
RADOS approved. See https://tracker.ceph.com/projects/rados/wiki/TENTACLE#httpstrackercephcomissues73856

Note: no 'tentacle' in the PR name