search

alastria / monitor

Monitor for Alastria platform.
Apache License 2.0
3 stars 3 forks source link

Possible timing issues when restarting the node after updating the configuration files #5

Closed jesus-alastria closed 6 years ago

jesus-alastria commented 6 years ago

https://github.com/alastria/monitor/blob/09e33ebc9fd00ed31d4d0910ed8129611f3c8a9f/lib/node.go#L139-L145 There is an arbitrary sleep time of 1 second between the call to Stop() and Start(). There may be timing issues causing errors on the Start() call if the process has not yet died. Additionally, because of issue #3 those errors are not detected and are not reported to the caller.

adlrocha commented 6 years ago

@jesus-alastria, this random sleep time was for development purposes. We will fix this issue. But again, the problem with #3 needs further analysis.