Merge pull request #48 from Ontotext-AD/GDB-10293/Change-AMI-Owner-ID…

…-and-add-functionality-to-specify-custom-owner-ID

[GDB-10293] Added support for custom owner ID for the AMIs and added new default owner ID

README.md

@@ -90,6 +90,7 @@ Before you begin using this Terraform module, ensure you meet the following prer
 | common\_tags | (Optional) Map of common tags for all taggable AWS resources. | `map(string)` | `{}` | no |
 | resource\_name\_prefix | Resource name prefix used for tagging and naming AWS resources | `string` | n/a | yes |
 | aws\_region | AWS region to deploy resources into | `string` | n/a | yes |
+| override\_owner\_id | Override the default owner ID used for the AMI images | `string` | `null` | no |
 | deploy\_backup | Deploy backup module | `bool` | `true` | no |
 | backup\_schedule | Cron expression for the backup job. | `string` | `"0 0 * * *"` | no |
 | backup\_retention\_count | Number of backups to keep. | `number` | `7` | no |

main.tf

@@ -175,8 +175,9 @@ module "graphdb" {
 
   # VM Image
 
-  ami_id          = var.ami_id
-  graphdb_version = var.graphdb_version
+  ami_id            = var.ami_id
+  graphdb_version   = var.graphdb_version
+  override_owner_id = var.override_owner_id
 
   # Managed Disks
 

modules/graphdb/main.tf

@@ -11,7 +11,7 @@ data "aws_caller_identity" "current" {}
 data "aws_ami" "graphdb" {
   count = var.ami_id != null ? 0 : 1
 
-  owners      = ["770034820396"] # Ontotext
+  owners      = var.override_owner_id != null ? [var.override_owner_id] : ["679593333241"]
   most_recent = true
 
   filter {

modules/graphdb/variables.tf

@@ -20,6 +20,11 @@ variable "aws_subscription_id" {
   type        = string
 }
 
+variable "override_owner_id" {
+  description = "Override the default owner ID used for the AMI images"
+  type        = string
+}
+
 # GraphDB Parameters
 
 variable "graphdb_admin_password" {

variables.tf

@@ -16,6 +16,12 @@ variable "aws_region" {
   type        = string
 }
 
+variable "override_owner_id" {
+  description = "Override the default owner ID used for the AMI images"
+  type        = string
+  default     = null
+}
+
 # Backup configurations
 variable "deploy_backup" {
   description = "Deploy backup module"