Skip to content

Value of --wait flag is being used as a timeout by linkerd multicluster check command #11266

@klingerf

Description

@klingerf

What problem are you trying to solve?

All check commands in the codebase support a --wait flag. That flag is used as the value of "RetryDeadline" when executing individual checks. For the linkerd multicluster check command only, the value of the flag is also being used as a timeout when checking if gateway mirrors have endpoints. This is confusing, since the --wait duration controls whether or not to retry failed checks in almost all cases, but it also controls how long it takes to run an individual check in just this one case.

How should the problem be solved?

We could introduce a separate --timeout flag that can be used to time out individual checks. That way the --wait flag can be used to control retry behavior exclusively, and not affect timeouts.

Any alternatives you've considered?

We could just pick a sane timeout for checking if gateway mirrors have endpoints, and not allow that duration to be configurable via CLI flags.

How would users interact with this feature?

No response

Would you like to work on this feature?

maybe

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions