Lookup of AMIs by region in atlas_artifact ?

[steve-jansen]

Hi Hashicorp,

Following up on #2731, how does one lookup an atlas_artifact AMI by AWS region?

@ravbaba posted in #732:

ami = "${lookup(atlas_artifact.mesos-master.metadata_full, concat("region-", var.region))}"

But this raises an error like:

Error running plan: 1 error(s) occurred:

* lookup: lookup in 'B780EEEC-B661-4EB8-8236-A01737AD98B6' failed to find 'region.us-east-1' in:

${lookup(atlas_artifact.mesos-master.metadata_full, concat("region.", var.region))}

Is Atlas/AMI lookup possible via some other means now?

Cheers,
Steve

[pstengel]

This is the workaround I'm currently using:

resource "aws_instance" "controller" {
  ami = "${element(split(",", atlas_artifact.controller.metadata_full.ami_id), index(split(",", atlas_artifact.controller.metadata_full.region), var.region))}"
  ...

It's long and ugly and we definitely need a better way, but it works for now. 😄

[stack72]

Hi @steve-jansen

Sorry this has taken so long to reply to - there is a backlog of issues that I am working my way through right now. Can you have a look and tell me if the new data_sources work will help here?

https://github.com/hashicorp/terraform/blob/master/website/source/docs/providers/atlas/d/artifact.html.markdown

This is already available in 0.7.0-RC3

thanks

Paul

[steve-jansen]

Hi @stack72

No worries, we found ways to workaround the challenge here, which includes #7365.

The data_sources are great. Not sure they solve this specific issue here, which is that metadata_full doesn't seem to behave like a proper map, which breaks the lookup function when trying to lookup an AMI by region.

Cheers,
Steve

[mootpt]

@steve-jansen

I have some success with:

variable "region" { default = "us-east-1" }

data "atlas_artifact" "foo" {
        name          = "ptr/foo"
        type          = "amazon.ami"
}

output "ami_id" {
        value = "${lookup(data.atlas_artifact.foo.metadata_full, "region-${var.region}")}"
}

but maybe I am missing the requirements? Why the use of concat? Concat no longer works with strings, just lists.

[steve-jansen]

@mootpt looks great! Thanks for sharing. Closing this issue.

🍻

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.