#atlantis - June 2019 | Slack Archive

atlantisArchived

19 messages

Discuss the Atlantis (runatlantis.io)
Archive: https://archive.sweetops.com/atlantis/

May 2019

June 2019

Aug 2019

nutellinoitover 7 years ago

i made a pull request that parhaps fixes the problem

Igorover 7 years ago(edited)

I updated to 0.8.1 running on fargate, and authentication fails for me on the app user in BitBucket. @nutellinoit Have you given the new version a shot yet? Are things working for you?

nutellinoitover 7 years ago

I tried one pull request this morning and everything was working , I did not try without approving the pull tho

Igorover 7 years ago

@nutellinoit You didn't have to update anything with the bitbucket app user?

nutellinoitover 7 years ago

No, nothing

nutellinoitover 7 years ago

I will try tomorrow morning with other pull requests

nutellinoitover 7 years ago

it's 22:52 in Italy

Igorover 7 years ago

Thanks, must be something with my setup, I jumped a few versions

Igorover 7 years ago

Okay, figured it out, I now need to use the username of the actual BitBucket user, instead of the app name as it previously was configured

marcover 7 years ago(edited)

is there a way to pass the region for the aws provider directly to whatever modules the ecs-atlantis depends on?
I've noticed that it asks twice for provider.aws.region so I am assuming region is not passed alongside to dependencies.

marcover 7 years ago

Acquiring state lock. This may take a few moments...
provider.aws.region
  The region where AWS operations will take place. Examples
  are us-east-1, us-west-2, etc.

  Default: us-east-1
  Enter a value: us-east-2

provider.aws.region
  The region where AWS operations will take place. Examples
  are us-east-1, us-west-2, etc.

  Default: us-east-1
  Enter a value: us-east-2

marcover 7 years ago

not a big deal, but I wonder if this is expected

marcover 7 years ago(edited)

basically it seems that var.region ain't passed here: https://github.com/cloudposse/terraform-aws-ecs-atlantis/blob/master/main.tf#L79

Andriy Knysh (Cloud Posse)over 7 years ago

@marc we did not test it in this scenario. We have ENV var for region defined in the Dockerfile(s) https://github.com/cloudposse/testing.cloudposse.co/blob/master/Dockerfile#L20

Andriy Knysh (Cloud Posse)over 7 years ago

so it always has the region

marcover 7 years ago

ah, good to know

marcover 7 years ago

thank you

Andriy Knysh (Cloud Posse)over 7 years ago

but we will be converting it to TF 0.12, and fix that too

marcover 7 years ago

🙇‍♂️

May 2019 Browse by date August 2019