-
Notifications
You must be signed in to change notification settings - Fork 188
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
Add metrics-server for Kubernetes HPA / stats #3
Add metrics-server for Kubernetes HPA / stats #3
Conversation
Tested with "install.sh" against Civo k3s Signed-off-by: Alex Ellis (OpenFaaS Ltd) <[email protected]>
@@ -0,0 +1,9 @@ | |||
--- | |||
name: metrics-server |
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.
Please could you name this as Metrics Server, the name is used as a title in display on the site.
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.
Sure
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.
Actually, don't worry, I've made a change to the YAML format to allow for name to have a hyphen but a title
for a nicer display version. I'll fix this post-merge.
metrics-server/post_install.md
Outdated
|
||
### Test it out | ||
|
||
* Check that data is being received from nodes: |
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.
Would rather drop these bullet points, normal paragraphs work well when interspersed with code snippets. If you like I can change these after merging though.
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.
Resolved
Signed-off-by: Alex Ellis (OpenFaaS Ltd) <[email protected]>
pull latest changes from upstream civo repo
Tested with "install.sh" against Civo k3s
Signed-off-by: Alex Ellis (OpenFaaS Ltd) [email protected]