Skip to content
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

octopus: librados/watch_notify: reconnect after socket injection #46500

Open
wants to merge 2 commits into
base: octopus
Choose a base branch
from

Conversation

NitzanMordhai
Copy link
Contributor

backport tracker: https://tracker.ceph.com/issues/55767


backport of #45825
parent tracker: https://tracker.ceph.com/issues/45868

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

For some tests, if socket failure injection cause watch to fail, we can re-watch using the error callback

Fixes: https://tracker.ceph.com/issues/45868
Signed-off-by: Nitzan Mordechai <nmordec@redhat.com>
(cherry picked from commit 5e5e7c5)
When socket failure injection or any delay that time out watch we need reconnect the watch
using the error callback to reconnect

Fixes: https://tracker.ceph.com/issues/47025
Signed-off-by: Nitzan Mordechai <nmordec@redhat.com>
(cherry picked from commit d2c5d7a)
@NitzanMordhai NitzanMordhai added this to the octopus milestone Jun 2, 2022
@NitzanMordhai
Copy link
Contributor Author

jenkins test make

@NitzanMordhai
Copy link
Contributor Author

jenkins test make check

@neha-ojha neha-ojha requested a review from badone June 22, 2022 21:21
@neha-ojha neha-ojha added the core label Jun 22, 2022
@neha-ojha
Copy link
Member

removing needs-qa, octopus is EOL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants