-
Notifications
You must be signed in to change notification settings - Fork 10
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
Updated docs based on latest Score release #60
Conversation
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.
A few small changes. Otherwise good.
@@ -244,6 +246,8 @@ readinessProbe: | |||
- `target`: specifies a path and name. | |||
- `mode`: specifies access mode. | |||
- `content`: specifies inline content and supports templates. |
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.
Exactly one of content
or source
must be specified.
Suggest:
content
: specifies inline content and supports templates. Exactly one ofcontent
orsource
must be defined.
@@ -244,6 +246,8 @@ readinessProbe: | |||
- `target`: specifies a path and name. | |||
- `mode`: specifies access mode. | |||
- `content`: specifies inline content and supports templates. | |||
- `source`: specifies a path to a file to make available. |
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.
Exactly one of content
or source
must be specified.
Suggest:
source
: specifies a path to a file to make available. Exactly one ofcontent
orsource
must be defined.
@@ -244,6 +246,8 @@ readinessProbe: | |||
- `target`: specifies a path and name. | |||
- `mode`: specifies access mode. | |||
- `content`: specifies inline content and supports templates. | |||
- `source`: specifies a path to a file to make available. | |||
- `noExpand`: If true, placeholder expansion will not occur in the `content`. |
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.
Is not super clear. Would suggest the more clear:
noExpand
: If true, thecontent
or file referenced insource
will be used literally with no placeholder expansion in the running container.
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.
LGTM
Updated docs based on latest release v.0.8.0:
workload-source-url
to specify URL of file that is managing the humanitec workload.Not included:
--message
parameter (deployment
'scomment
with--message
score-humanitec#57) is already documented for the delta and message command