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

Example of using .spec.affinity #133

Merged
merged 4 commits into from
Jul 7, 2020
Merged

Example of using .spec.affinity #133

merged 4 commits into from
Jul 7, 2020

Conversation

navidsh
Copy link
Member

@navidsh navidsh commented Jun 2, 2020

What this PR does / why we need it?:

  • Adds examples for using .spec.affinity

Does this PR introduce a user-facing change?

  • User guide
  • CHANGELOG.md

Which issue(s) this PR fixes:

Fixes #126

@navidsh navidsh marked this pull request as ready for review June 4, 2020 14:51
@navidsh navidsh requested review from leochr and arturdzm June 4, 2020 14:51
Copy link
Member

@leochr leochr left a 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.

Copy link

@chirp1 chirp1 left a 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.


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.
Copy link

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".)

@navidsh navidsh merged commit 9474a54 into master Jul 7, 2020
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.

Adding a sample for node and pod (anti)affinity
3 participants