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

Add metrics-server for Kubernetes HPA / stats #3

Merged
merged 2 commits into from
Sep 11, 2019
Merged

Add metrics-server for Kubernetes HPA / stats #3

merged 2 commits into from
Sep 11, 2019

Conversation

alexellis
Copy link
Contributor

Tested with "install.sh" against Civo k3s

Signed-off-by: Alex Ellis (OpenFaaS Ltd) [email protected]

Tested with "install.sh" against Civo k3s

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <[email protected]>
@@ -0,0 +1,9 @@
---
name: metrics-server
Copy link
Contributor

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure

Copy link
Contributor

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.


### Test it out

* Check that data is being received from nodes:
Copy link
Contributor

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.

Copy link
Contributor Author

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]>
@andyjeffries andyjeffries merged commit 5829f8c into civo:master Sep 11, 2019
@ssmiller25 ssmiller25 mentioned this pull request Oct 7, 2020
3 tasks
DMajrekar pushed a commit that referenced this pull request Nov 2, 2023
pull latest changes from upstream civo repo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants