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
Expose endpoint for triggering sync. Operator should be able to execute sync manually when he notices differences between Marathon and Consul states to resolve the issue faster.
Differences may occur e.g. when marathon-consul doesn't receive events from Marathon.
Things to consider:
usability: when running multiple instances of marathon-consul as a cluster how to know which instance to call? Currently sync may be executed on the leader only.
safety: would it be safe if manual sync and scheduled sync were running simultaneously? If not, how to introduce mutual exclusion?
The text was updated successfully, but these errors were encountered:
We can add force parameter. Without force endpoint will perform sync only if configuration allows syncs (sync is enabled and shouldPerformSync() returns true). This will complicate the code so I prefer to always perform syncing on manual trigger.
I think we should make it as simple as possible. This is maintenance feature to recover after outage (e..g., Marathon stops sending events and we need quickly refresh state). Same effect could be achieved by starting marathon-consul with forced sync enabled on local machine.
Expose endpoint for triggering sync. Operator should be able to execute sync manually when he notices differences between Marathon and Consul states to resolve the issue faster.
Differences may occur e.g. when marathon-consul doesn't receive events from Marathon.
Things to consider:
The text was updated successfully, but these errors were encountered: