Modify dapi_err
command and exception handling
#15298
Merged
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.
Description
This PR changes how the
WorkerHandler
and theMasterHandler
manage thedapi_err
command. Its management is passed to theHandler
class, from which both inherit, incommon.py
, more specifically in itsprocess_request
method. It takes into account the case when the command has to be sent to the peer but the same has disconnected before receiving it.Logs/Alerts example
When the peer disconnects before receiving the
dapi_err
command, the logs show:Master
cluster.log
Worker
cluster.log
Tests
Unit tests
New test cases have been added in order to cover the modifications in
common.py
Cluster endpoints Integration Tests