Skip to content

flags: use time.Duration in duration flags #803

@jsign

Description

@jsign

We currently have some arbitrary magnitude durations seconds/minutes as Int64 in flag/envs

Switch to time.Duration since it's much clearer and the user can specify them with 10m or 10h. This avoids confusion about the right magnitude: seconds or minutes?

This is a breaking change, since the operators should fix whatever flag/env that they might be overridden, so it would be good to batch into some planned major release.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions