Skip to content

Commit

Permalink
The eastern region is us-east-1, not us-east-2
Browse files Browse the repository at this point in the history
  • Loading branch information
errordeveloper committed Jun 6, 2018
1 parent 548b6ce commit 6ef09b3
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions cmd/eksctl/create.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ func createCmd() *cobra.Command {

const (
EKS_REGION_US_WEST_2 = "us-west-2"
EKS_REGION_US_EAST_2 = "us-east-2"
EKS_REGION_US_EAST_1 = "us-east-1"
DEFAULT_EKS_REGION = EKS_REGION_US_WEST_2
DEFAULT_NODE_COUNT = 2
DEFAULT_NODE_TYPE = "m5.large"
Expand Down Expand Up @@ -96,8 +96,8 @@ func doCreateCluster(cfg *eks.ClusterConfig) error {
return fmt.Errorf("--ssh-public-key must be non-empty string")
}

if cfg.Region != EKS_REGION_US_WEST_2 && cfg.Region != EKS_REGION_US_EAST_2 {
return fmt.Errorf("--region=%s is not supported only %s and %s are supported", cfg.Region, EKS_REGION_US_WEST_2, EKS_REGION_US_EAST_2)
if cfg.Region != EKS_REGION_US_WEST_2 && cfg.Region != EKS_REGION_US_EAST_1 {
return fmt.Errorf("--region=%s is not supported only %s and %s are supported", cfg.Region, EKS_REGION_US_WEST_2, EKS_REGION_US_EAST_1)
}

if err := ctl.LoadSSHPublicKey(); err != nil {
Expand Down
2 changes: 1 addition & 1 deletion pkg/eks/cfn.go
Original file line number Diff line number Diff line change
Expand Up @@ -277,7 +277,7 @@ func (c *ClusterProvider) stackParamsDefaultNodeGroup() map[string]string {
// currently source of truth for these is here:
// https://docs.aws.amazon.com/eks/latest/userguide/launch-workers.html
"us-west-2": "ami-73a6e20b",
"us-east-2": "ami-dea4d5a1",
"us-east-1": "ami-dea4d5a1",
}

if c.cfg.NodeAMI == "" {
Expand Down

0 comments on commit 6ef09b3

Please sign in to comment.