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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pacific: Wip nitzan fixing few rados/test.sh #49943
Merged
Merged
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
Adding debug message in case fork failed and output the error message. Signed-off-by: Nitzan Mordechai <nmordec@redhat.com> (cherry picked from commit 9b1d1b1)
…elete reconnect During test LibRadosWatchNotify.AioWatchDelete rados_watch_check can return error -102 if reconnect happened, in that case Broken pipe reconnect and -102 returned Signed-off-by: Nitzan Mordechai <nmordech@redhat.com> (cherry picked from commit 0184c4b)
Some of the tests in Rados.sh can fail when trying to test watch_list return size if we hit watch timeout. increase the watch timeout for rados test Fixes: https://tracker.ceph.com/issues/47025 Signed-off-by: Nitzan Mordechai <nmordec@redhat.com> (cherry picked from commit 8e76c17)
Waiting to watch_notify callback can take some time during heavy tests that run under rados/test.sh for example. Increasing the alarm time will allow the task to complete under those conditions. Fixing: https://tracker.ceph.com/issues/58130 Signed-off-by: Nitzan Mordechai <nmordec@redhat.com> (cherry picked from commit 9f0dd47)
Adding and changing temporary pools names prefix to match the TestCase and test name to make debugging easier Signed-off-by: Nitzan Mordechai <nmordec@redhat.com> (cherry picked from commit 983a9ca)
NitzanMordhai
force-pushed
the
wip-58611-pacific
branch
from
January 31, 2023 06:15
25df4f3
to
4e036b3
Compare
@ljflores Can you please review? |
@ljflores please review |
1 similar comment
@ljflores please review |
ljflores
approved these changes
Jul 5, 2023
LGTM @NitzanMordhai |
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 tracker: https://tracker.ceph.com/issues/58611
backport of #49109
parent tracker: https://tracker.ceph.com/issues/45615
this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh