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
quincy: mgr: Add one finisher thread per module #51044
Conversation
Adds a finisher thread for each module. Each module command is executed via corresponding finisher thread. There is generic finisher thread via which all other commands like notify, config is run. Fixes: https://tracker.ceph.com/issues/51177 Signed-off-by: Kotresh HR <khiremat@redhat.com> (cherry picked from commit 46de643)
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com> (cherry picked from commit 8114bdd)
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com> (cherry picked from commit d8ec1bd)
Fixes: https://tracker.ceph.com/issues/51177 Signed-off-by: Kotresh HR <khiremat@redhat.com> (cherry picked from commit 2c2ef6d)
jenkins test make check |
jenkins test api |
@vshankar I think you may want to add quincy-batch-1 as well so that @yuriw can pick this up for quincy testing? |
I used to do that, but Yuri picked up PRs that have needs-qa tag, so, I stopped using that (additional) label. |
Reviewed-by: Kotresh HR khiremat@redhat.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed-by: Kotresh HR khiremat@redhat.com
backport tracker: https://tracker.ceph.com/issues/59416
Added following dependent qa commits from main branch:
backport of #47893
parent tracker: https://tracker.ceph.com/issues/51177
this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh