Skip to content
This repository has been archived by the owner on Feb 11, 2022. It is now read-only.

parameterize f5_owner_id to account for other regions #75

Open
Mikej81 opened this issue Aug 18, 2020 · 0 comments
Open

parameterize f5_owner_id to account for other regions #75

Mikej81 opened this issue Aug 18, 2020 · 0 comments

Comments

@Mikej81
Copy link

Mikej81 commented Aug 18, 2020

Non US Commercial regions have different OwnerID.

Add

variable f5_owner_ids {
  description = "AWS Owner ID's across regions change"
  type        = list(string)
  default     = ["679593333241", "345084742485", "self", "aws-marketplace"]
}

to https://github.com/f5devcentral/terraform-aws-bigip/blob/develop/variables.tf#L12

and change

data "aws_ami" "f5_ami" {
  most_recent = true
  owners      = var.f5_owners_ids

  filter {
    name   = "name"
    values = ["${var.f5_ami_search_name}"]
  }
}

https://github.com/f5devcentral/terraform-aws-bigip/blob/develop/main.tf#L65

Would PR from my fork but I messed up master branch, and lazy.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant