Adding signal handler module #227
Conversation
|
@nishith-vihar why are we defining signal handlers in every module ? Redant runs as a single process and signal handlers are to be defined on the process level. |
|
@srijan-sivakumar Ack. I think you are right . I checked it out again. I believe the signal handling has to be done in the main function of the redant alone. I will make the required changes. |
There was a problem hiding this comment.
A crude way. But a start nonetheless. The next stage is to add cleanups in individual test cases whenever they see this exception being raised. @nishith-vihar can you verify if this exception being raised is seen by the test runs too ? If it is the case, then we can do a cleanup directly in the except block in the parent_tests, else we have to come with a signalling mechanism within the main and the child processes ( which are running the TC ).
|
@aujjwal-redhat please review the code, I've removed the merge conflicts. |
|
@nishith-vihar ...lint check.. |
Signal handler is added for the graceful exit of the test framework. Fixes: gluster#208 Signed-off-by: Nishith Vihar Sakinala <nsakinal@redhat.com>
Updates: gluster#58 Signed-off-by: Nishith Vihar Sakinala <nsakinal@redhat.com>
Signal handler function is added for graceful exiting of
the test framework.
Fixes: #208
Signed-off-by: Nishith Vihar Sakinala nsakinal@redhat.com