awslabs / fargatecli

CLI for AWS Fargate
Apache License 2.0
893 stars 114 forks source link

add new Fargate regions #54

Closed tisba closed 6 years ago

tisba commented 6 years ago

Fargate is now available in more regions, see https://aws.amazon.com/about-aws/global-infrastructure/regional-product-services/. It was announced a few days ago, see https://aws.amazon.com/about-aws/whats-new/2018/04/aws-fargate-now-available-in-ohio--oregon--and-ireland-regions/

tisba commented 6 years ago

I'm not sure if this is a problem with the freshly launched region, but there seems to be a problem with the default security group that fargate provides:

bin/fargate task run iperf-test --region=eu-west-1 --cpu 256 --memory 512
…
[!] Could not run ECS task
InvalidParameterException: security group cannot be blank.
    status code: 400, request id: c3ecc5a7-4be8-11e8-8aa8-e91705edd77d
tisba commented 6 years ago

I think the problem is related to our AWS account. Would be nice, if someone can sanity check this. Beside this, the changes work and I am able to use fargate in eu-west-1, e.g.

ramirantala commented 6 years ago

I tried this PR and actually got same error @tisba The region itself works, but command fails to this error

ramirantala commented 6 years ago

I need to specify the subnet and security group and then it seems to work. So detecting the default ones might not work. We have two vpcs on our account - have not really checked what is the problem.

tisba commented 6 years ago

It works fine though if you provide a explicit security group. Not sure what I am missing here...

ramirantala commented 6 years ago

Yes, this works for me..

./fargate task run nginx --image nginx --region=eu-west-1 --cpu 256 --memory 512 --cluster your-cluster --security-group-id sg-xxx --subnet-id subnet-xxx

tisba commented 6 years ago

The problems seems to be that the fargate-default security group is not created. I haven't figured out though, why this is the case :-/

jpignata commented 6 years ago

Thanks for opening this! We'll get the latest pull merged ASAP.