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

Add support for Asia Pacific (Osaka) region #3347

Merged
merged 3 commits into from
Feb 26, 2021
Merged

Conversation

Callisto13
Copy link
Contributor

@Callisto13 Callisto13 commented Feb 24, 2021

Description

Closes #3031.

The ap-northeast-3 region is not opt-in, so it can use the standard account ID, which means adding support for it is very simple (a little too simple, let me know if there is more to it 😅 ).

Fargate support in this region will lag behind for a couple of days, so there is now a check on create to log this. I have also added a warning in the docs, but since this will be removed in like a week, I wonder if it is necessary? Maybe simply pinning an issue in this repo will be enough to cover us for that time 🤷‍♀️ .

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 🌟

Added to DoCreateFargateProfiles so that this is checked as part of
cluster create and fargateprofile create
@Callisto13 Callisto13 added the kind/feature New feature or request label Feb 24, 2021
@Callisto13 Callisto13 requested a review from a team February 24, 2021 11:38
Copy link
Contributor

@aclevername aclevername left a comment

Choose a reason for hiding this comment

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

LGTM

@Callisto13 Callisto13 enabled auto-merge (squash) February 26, 2021 08:59
@Callisto13 Callisto13 merged commit 0e1c6e5 into eksctl-io:main Feb 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for Asia Pacific (Osaka) region
2 participants