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

Revert "Adding support for inf2 and trn1n instances" #6597

Merged
merged 2 commits into from
May 10, 2023

Conversation

ljosyula
Copy link
Contributor

@ljosyula ljosyula commented May 9, 2023

This reverts commit 8ab37f653eb184b06ea63fea38e634beec576840.

Description

We are seeing integration test failures due to neuron daemonset not getting installed. Reverting the changes to dive deeper into the issue

Checklist

  • Added tests that cover your change (if possible)
  • Added/modified documentation as required (such as the README.md, or the userdocs directory)
  • Manually tested
  • Made sure the title of the PR is a good description that can go into the release notes
  • (Core team) Added labels for change area (e.g. area/nodegroup) and kind (e.g. kind/improvement)

BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯

  • Backfilled missing tests for code in same general area 🎉
  • Refactored something and made the world a better place 🌟

This reverts commit 8ab37f653eb184b06ea63fea38e634beec576840.
@cPu1 cPu1 added the skip-release-notes Causes PR not to show in release notes label May 10, 2023
@cPu1 cPu1 enabled auto-merge (squash) May 10, 2023 06:52
@cPu1 cPu1 disabled auto-merge May 10, 2023 07:40
@cPu1 cPu1 merged commit e96652d into eksctl-io:main May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-release-notes Causes PR not to show in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants