rkt: add long command descriptions #2098
rkt: add long command descriptions #2098
Conversation
Is this worth a CHANGELOG entry? I say yeah. |
6bd2ff1
to
6aa461e
Compare
|
||
Long: `UUID should be the UUID of a running pod. | ||
|
||
By default the CMD run is bash, providing the user with shell access to the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bash -> /bin/bash
6aa461e
to
4d0a66b
Compare
Updated |
Short: "Garbage collect rkt pods no longer in use", | ||
Long: `This is intended to be run periodically from a timer or cron job. | ||
|
||
Garbage collection is a 2 step process. First, stopped pods are moved to the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/2 step/2-step/
?
4d0a66b
to
38b6b22
Compare
LGTM |
@@ -34,7 +34,10 @@ var ( | |||
cmdFetch = &cobra.Command{ | |||
Use: "fetch IMAGE_URL...", | |||
Short: "Fetch image(s) and store them in the local store", | |||
Run: runWrapper(runFetch), | |||
Long: `Locates and downloads remote ACIs and their attached signatures. | |||
If the ACI image is available in the local store, the image will not be fetched |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACI image is redundant
Paging @joshix |
This needs to be rebased, it has some conflicts. |
@@ -42,7 +42,11 @@ var ( | |||
cmdAPIService = &cobra.Command{ | |||
Use: `api-service [--listen="localhost:15441"]`, | |||
Short: "Run API service (experimental, DO NOT USE IT)", | |||
Run: runWrapper(runAPIService), | |||
Long: `The API service listens for gRPC requests on the address and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quoting style: Should Run:
value be quoted as it is here, with backticks, or as the value for Short:
is quoted, in double quotes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is fine as it is: double quotes for single-line strings and backticks for multi-line strings.
38b6b22
to
d5a45f1
Compare
updated and rebased |
Thanks |
…ptions rkt: add long command descriptions
Most of these changes were inspired by the rkt subcommand documentation.
Closes #2080