-
Notifications
You must be signed in to change notification settings - Fork 23
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
Example of using .spec.affinity #133
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.
@navidsh Looks great. Couple of comments for enhancements. Thanks.
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.
Hi Navid,
I added my comments. If you have any questions about my suggested updates, let me know.
examples/affinity/README.adoc
Outdated
|
||
You will deploy instances of two applications, `coffeeshop-frontend` and `coffeeshop-backend`, co-located on the same nodes with SSD storage type. | ||
|
||
This scenario is inspired by examples from link:++https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node++[Assigning Pods to Nodes] Kubernetes tutorial. |
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.
Change to "examples from the Assigning Pods to Nodes Kubernetes tutorial.". (Add "the" in front of "Assigning".)
What this PR does / why we need it?:
.spec.affinity
Does this PR introduce a user-facing change?
CHANGELOG.md
Which issue(s) this PR fixes:
Fixes #126