#terraform-tool

hey there, anybody here aware why ForceNew is set to true for security_groups in builtin/providers/aws/resource_aws_instance.go ?

( I do not want TF to recreate an instance just to update (a) security group(s) )

ojongerius: I said the same today

I think you can override that

but I forget the details I am thinking of

luminous: I did some digging, and is currently needed, but AFAICS it is no longer needed. Removing it does require to call ModifyInstanceAttribute, instead of relying on the resource recreation to take care of things. Should not be hard to change, I might have a crack at it, if time permits.

'I did some digging, and is currently needed, but AFAICS it is no longer needed.' I meant to say, one can change security groups on running instances, but removing 'ForceNew' will not automagically work.

-it does after applying my patch, but I'll have to polish it a bit, before it's PR ready

luminous: guess your right about that one… For AWS this (setting the name) is kind of nasty, but that is because of how the AWS API works…

Most other resources (e.g. CloudStack and Google Compute instances, Heroku apps, DigitalOcean droplets) just have a `name` argument to set the name…

svanharmelen: nasty is a great name for the AWS api..

I am grateful terraform is taming it for me

Greetings, i see eu-central-1 in aws/config.go but i still see the "Not a valid region: eu-central-1" error, anyone have an idea why this is?

rmenn: are you using master or release?

the release

rmenn: maybe if you run the latest code from git you will see that work

oh really? i was thinking since it was in master it was on the 0.3.5 uild

ah

i see its there only on 0.3.6

my bad

@luminous thanks man, i got something working with the build from master