Skip to content

fix: Tolerances for EKS nodes need to be ON_DEMAND #2

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

Merged
merged 3 commits into from
Sep 16, 2024
Merged

Conversation

brent-anyscale
Copy link
Collaborator

Switches the ALL tolerance to ON_DEMAND, as taints must be unique to a tuple of (key, effect) in EKS, which wasn't possible with the ALL convention

Changes to be committed:
modified: eks-private/main.tf
modified: eks-public/main.tf

Pull request checklist

Please check if your PR fulfills the following requirements:

  • pre-commit has been run
  • Tests for the changes have been added (for bug fixes / features)
  • All tests passing
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Pull Request Type

  • Bugfix
  • New feature
  • Refactoring (no functional changes)
  • Documentation change
  • Other (please describe):

Does this introduce a breaking change?

  • Yes
  • No

Changes to be committed:
	modified:   examples/aws/eks-private/main.tf
Switches the ALL tolerance to ON_DEMAND, as taints must be unique to a tuple of (key, effect) in EKS, which wasn't possible with the ALL convention

Changes to be committed:
	modified:   eks-private/main.tf
	modified:   eks-public/main.tf
@brent-anyscale brent-anyscale requested a review from a team as a code owner September 16, 2024 15:53
@github-actions github-actions bot added documentation Improvements or additions to documentation examples labels Sep 16, 2024
Changes to be committed:
	modified:   CHANGELOG.md
@brent-anyscale brent-anyscale added the bug Something isn't working label Sep 16, 2024
@brent-anyscale brent-anyscale merged commit c0e3de0 into main Sep 16, 2024
3 checks passed
@brent-anyscale brent-anyscale deleted the brent/k8s branch September 16, 2024 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants