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

new task for config pod density #1237

Merged
merged 4 commits into from
Feb 25, 2022
Merged

Conversation

geoffcline
Copy link
Contributor

1. Issue, if available:
#1216

2. Description of changes:
add task for configuring pod density

3. How was this change tested?
n/a

4. Does this change impact docs?

  • Yes, PR includes docs updates
  • Yes, issue opened: link to issue
  • No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@geoffcline geoffcline added the documentation Improvements or additions to documentation label Jan 28, 2022
@geoffcline geoffcline self-assigned this Jan 28, 2022
@netlify
Copy link

netlify bot commented Jan 28, 2022

✔️ Deploy Preview for karpenter-docs-prod ready!

🔨 Explore the source changes: 1386165

🔍 Inspect the deploy log: https://app.netlify.com/sites/karpenter-docs-prod/deploys/62183e30f3c6550008d6021b

😎 Browse the preview: https://deploy-preview-1237--karpenter-docs-prod.netlify.app

@geoffcline geoffcline requested a review from chrisnegus January 28, 2022 20:33
Copy link
Contributor

@mikesir87 mikesir87 left a comment

Choose a reason for hiding this comment

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

Added a few thoughts. Feel free to provide additional feedback/thoughts.

website/content/en/preview/tasks/pod-density.md Outdated Show resolved Hide resolved
@geoffcline
Copy link
Contributor Author

Thanks for this detailed review @mikesir87 I understand it better now :)

@@ -0,0 +1,59 @@
---
title: "Control Pod Density"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
title: "Control Pod Density"
title: "Pod Density"

Copy link
Contributor

Choose a reason for hiding this comment

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

Control sends the wrong message, I think. Ultimately karpenter is going to have the say here. Limit might be more accurate, but I'd probably just drop any verb from the title.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think of pages under the task section fitting into a "I want to X" pattern. perhaps control is implied?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants