You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@marcusbooyah reported that he as an InStep service reporting being ready too early. We should verify that we're not making the same mistake for NEIONFORGE related services.
The text was updated successfully, but these errors were encountered:
I'm seeing NEONFORGE services calling this at the beginning of the their OnRunAsync() methods:
await SetStatusAsync(NeonServiceStatus.Starting);
This shouldn't be necessary. NeonService is initializing the status file to starting before calling OnRunAsync(). NeonService should also call SetStatusAsync(NeonServiceStatus.Starting) before calling OnRunAsync() and we should remove the service calls to this to simplify things.
Let's add a NeonService.Status property as well so services can be aware of their status.
@marcusbooyah reported that he as an InStep service reporting being ready too early. We should verify that we're not making the same mistake for NEIONFORGE related services.
The text was updated successfully, but these errors were encountered: