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
Related to #3775, we need to understand the purpose of the update checker better. Is it to tell that the client is outdated compared to the orchestrator's version? or outdated compared to latest releases from bacalhau?
As a user, I might expect the following:
2. If I am running a compute node, I would like the cli to compare my version with the orchestrator's version. I don't want the cli to ask me to update to a version that is beyond the orchestrator's
3. If I am running client commands, such as bacalhau job run, I would like the cli to also compare my version with the orchestrator's
4. If I a m running an orchestrator node, then I would like the cli to compare my version with the latest release from bacalhau
I'd propose we reduce the responsibility of the UpdateChecker to only retrieve the latest version of bacalhau from the update server. This will remove the dependency on a bacalhau client - and thus authentication - from the UpdateChecker service - which will fix this bug.
The UpdateChecker may then persist the value it retrieves from the update server to some "system configuration field" (will be very easy to do when #3959 lands). From there we can make the following changes:
Clients and Compute nodes can compare their version, the orchestrator's version, and the latest version notifying accordingly in stderr or log messages.
Orchestrator nodes can compare their version with the latest version notifying accordingly in log messages.
Alternatively, and this would be my preference:
Only Servers (Compute and Requester Nodes) use the update checker (as we already do now) to retrieve the latest version. If they are behind it they produce a log message stating so.
Clients are only notified when the server they are communicating with is on a version different than theirs. Clients commands do not run the UpdateChecked.
In this flow, clients are only notified they need to update when their server operator updates.
Related to #3775, we need to understand the purpose of the update checker better. Is it to tell that the client is outdated compared to the orchestrator's version? or outdated compared to latest releases from bacalhau?
As a user, I might expect the following:
2. If I am running a compute node, I would like the cli to compare my version with the orchestrator's version. I don't want the cli to ask me to update to a version that is beyond the orchestrator's
3. If I am running client commands, such as bacalhau job run, I would like the cli to also compare my version with the orchestrator's
4. If I a m running an orchestrator node, then I would like the cli to compare my version with the latest release from bacalhau
@frrist also has a valid point of view:
This requires feedback from product @aronchick @MichaelHoepler
The text was updated successfully, but these errors were encountered: