Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix CLI reference for the tsh --ttl flag #13489

Merged
merged 1 commit into from
Jun 14, 2022

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Jun 14, 2022

The value of the --ttl flag for tsh commands was incorrectly stated to
be a duration. Instead, it is an integer indicating a number of minutes.
The default is 12 hours.

Also does some minor copy-editing on the tsh global flags.

@zmb3
Copy link
Collaborator

zmb3 commented Jun 14, 2022

Ugh, this stinks. It's a duration for tctl and and number of minutes for tsh.

Hopefully work towards a CLI style guide can prevent things like this going forward.

The value of the --ttl flag for tsh commands was incorrectly stated to
be a duration. Instead, it is an integer indicating a number of minutes.
The default is 12 hours.

Also does some minor copy-editing on the tsh global flags.
@ptgott ptgott force-pushed the paul.gottschling/2022-06-14-tsh-ttl branch from c8252ea to 248b696 Compare June 14, 2022 20:52
@ptgott ptgott merged commit aafe27c into master Jun 14, 2022
@github-actions
Copy link

@ptgott See the table below for backport results.

Branch Result
branch/v7 Create PR
branch/v8 Create PR
branch/v9 Failed

ptgott added a commit that referenced this pull request Jun 16, 2022
Backports #13489

The value of the --ttl flag for tsh commands was incorrectly stated to
be a duration. Instead, it is an integer indicating a number of minutes.
The default is 12 hours.

Also does some minor copy-editing on the tsh global flags.
ptgott added a commit that referenced this pull request Jun 21, 2022
Backports #13489

The value of the --ttl flag for tsh commands was incorrectly stated to
be a duration. Instead, it is an integer indicating a number of minutes.
The default is 12 hours.

Also does some minor copy-editing on the tsh global flags.
ptgott added a commit that referenced this pull request Jul 7, 2022
Backports #13489

The value of the --ttl flag for tsh commands was incorrectly stated to
be a duration. Instead, it is an integer indicating a number of minutes.
The default is 12 hours.

Also does some minor copy-editing on the tsh global flags.
ptgott added a commit that referenced this pull request Jul 12, 2022
Backports #13489

The value of the --ttl flag for tsh commands was incorrectly stated to
be a duration. Instead, it is an integer indicating a number of minutes.
The default is 12 hours.

Also does some minor copy-editing on the tsh global flags.
ptgott added a commit that referenced this pull request Jul 12, 2022
Fix CLI reference for the tsh --ttl flag

Backports #13489

The value of the --ttl flag for tsh commands was incorrectly stated to
be a duration. Instead, it is an integer indicating a number of minutes.
The default is 12 hours.

Also does some minor copy-editing on the tsh global flags.
@ptgott ptgott deleted the paul.gottschling/2022-06-14-tsh-ttl branch March 22, 2023 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants