-
Notifications
You must be signed in to change notification settings - Fork 18
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
Some README improvements #16
Conversation
22ff1af
to
0bf0ec2
Compare
@@ -46,8 +46,10 @@ Example: | |||
### Daemonset | |||
Checks if a specified daemonset is already running on the same host, this dependency requires a `POD_NAME` | |||
env which can be easily passed through the [downward api](http://kubernetes.io/docs/user-guide/downward-api/). | |||
The `POD_NAME` variable is used to determine right host to check dependencies. |
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 would change it to :
The POD_NAME
variable is mandatory and is used to resolve dependencies.
Example: | ||
|
||
`POD_NAME=nova` |
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 would delete this POD_NAME has to passed via downward api.
@@ -73,7 +75,12 @@ Example: | |||
|
|||
`DEPENDENCY_SOCKET=/var/run/openvswitch/ovs.socket` | |||
|
|||
## Image | |||
|
|||
Image is build after every push to kubernetes-entrypoint repository. |
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 would change it to :
Build process for image is trigged after each commit.
0bf0ec2
to
655b28a
Compare
@@ -46,6 +46,7 @@ Example: | |||
### Daemonset | |||
Checks if a specified daemonset is already running on the same host, this dependency requires a `POD_NAME` | |||
env which can be easily passed through the [downward api](http://kubernetes.io/docs/user-guide/downward-api/). | |||
The `POD_NAME` variable is mandatory and is used to resolve dependencies. | |||
Example: | |||
|
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.
Add example of using downward api.
655b28a
to
eb4ca84
Compare
Added link to example how to use downward api |
Information about kubernetes-entrypoint docker image Improved daemonset usage
eb4ca84
to
129413f
Compare
LGTM |
Closes #3
Related with #15
cc @DTadrzak @PiotrProkop