Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Comment out data values #29

Open
tstrupp opened this issue Aug 17, 2023 · 0 comments
Open

Comment out data values #29

tstrupp opened this issue Aug 17, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@tstrupp
Copy link

tstrupp commented Aug 17, 2023

Describe the bug

As the AMI is currently not available in the marketplace, can we comment this section out in data.tf temporarily until resolved?

Deployment will fail unable to find the correct AMI-ID.

data "aws_ami" "avi" {
  most_recent = true
  owners      = ["aws-marketplace"]

  filter {
    name   = "name"
    values = ["Avi*Controller-${var.avi_version}-*"]
  }
}

Reproduction steps

Run TF module with a custom_ami variable defined. This will result in a failure to find any objects. Debugging in Terraform confirms it says the AMI-ID is incorrect. Commenting this section out resolves the issue for now.

Expected behavior

The Terraform module should be able to locate purely based off of the AMI id itself, versus any other AMI properties. If an AMI is specified, a version should also not be needed. Something like:

count = var.custom_ami != null : 1 : 0

Additional context

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant