hammer: Testing: SIGABRT in TrackedOp::dump() via dump_ops_in_flight() #1081
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* add a wrapper to log uncaught exception to self.logger, greenlet also prints the backtrace and exception to stderr, but teuthology.log does not capture stderr. so we need to catch them by ourselves to reveal more info to root-cause this issue. * log uncaught exception thrown by Thrasher.do_thrash() to self.log. See: #10630 Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit 64de3cd)
Signed-off-by: David Zafman <dzafman@redhat.com> (cherry picked from commit 4ad3b86)
Add dump_ops_enable which continuously dumps ops using 3 commands Add optrack_toggle_delay to alternate op tracking enablement Signed-off-by: David Zafman <dzafman@redhat.com> (cherry picked from commit 7a52876) Conflicts: tasks/ceph_manager.py (trivial) tasks/thrashosds.py (trivial)
Signed-off-by: David Zafman <dzafman@redhat.com> (cherry picked from commit 447bf87)
Signed-off-by: David Zafman <dzafman@redhat.com> (cherry picked from commit a595651)
dzafman
changed the title
|
Only merge with ceph/ceph#10255 |
|
For some reason I got 11 crashes of python during a rados:thrash run. Using strings on the cores revealed a common strings: /usr/bin/python /usr/bin/ceph tell osd.* injectargs --osd_enable_op_tracker=... |
dzafman
changed the title
dzafman
changed the title
Signed-off-by: David Zafman <dzafman@redhat.com>
Needed to add stdout option to pass down to run() Signed-off-by: David Zafman <dzafman@redhat.com> (cherry picked from commit d762fa5)
Signed-off-by: David Zafman <dzafman@redhat.com> (cherry picked from commit 9d52d10) Conflicts: tasks/ceph_manager.py (trivial)
|
The last 2 commits come from #1122 which hasn't merged yet |
|
Passed with latest version: |
|
@dzafman Can we remove DNM now? I have prepared a |
dzafman
changed the title
|
Superseded by #1278 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport 3 pull requests and add fix specifically for Hammer not to use osd.* since it crashes the command.
#344
#911
#1122