I think #289 introduced some severe flakiness in the ability to just run jobs, since monitor could time out and return context.DeadlineExceeded wrapping errors without the surrounding context actually being cancelled. I think. I'm not sure.
Also I changed my mind on closing the errors channel - reading controller.go, I don't think it needs to be closed at all.
I think #289 introduced some severe flakiness in the ability to just run jobs, since monitor could time out and return context.DeadlineExceeded wrapping errors without the surrounding context actually being cancelled. I think. I'm not sure.
Also I changed my mind on closing the errors channel - reading controller.go, I don't think it needs to be closed at all.