clustershell icon indicating copy to clipboard operation
clustershell copied to clipboard

Task: add default event handler in task.shell()

Open brianpkelly opened this issue 9 years ago • 2 comments

Fixed the bug I ran into as detailed in this thread from the mailing list.

brianpkelly avatar May 26 '15 16:05 brianpkelly

Thanks for this pull request but this bug should not be fixed that way. It is perfectly normal for a worker to not have an event handler.

The bug only appears when using the new gateway code. The proper fix is more removing event handler checks in PropagationChannel.recv_ctl() but I need to look at this more closely to be sure...

degremont avatar May 27 '15 08:05 degremont

Ah OK, I wasn't sure whether an event handler was necessary or not.

brianpkelly avatar May 27 '15 14:05 brianpkelly