Configurable provisionning data source

[sestegra]

Hello,

In order to support both http and cd data source for the provisioning, I suggest following updates.

Let's assume the update of Ubuntu 20.04 for following instructions

1/ Add common_data_source in variables.pkr.hcl

variable "common_data_source" {
  type        = string
  description = "The provisioning data source ('http' or 'cd')."
}

2/ Rename the folder http to `data``

3/ Add data_source_content and data_source_command local variables

locals {
  ...
  data_source_content = {
    "/meta-data" = file("data/meta-data")
    "/user-data" = templatefile("data/user-data.pkrtpl.hcl", { build_username = var.build_username, build_password_encrypted = var.build_password_encrypted, vm_guest_os_language = var.vm_guest_os_language, vm_guest_os_keyboard = var.vm_guest_os_keyboard, vm_guest_os_timezone = var.vm_guest_os_timezone })
  }
  data_source_command = var.common_data_source == "http" ? "ds=nocloud-net;s=http://{{.HTTPIP}}:{{.HTTPPort}}/" : "ds=nocloud"
}

4/ Update `Boot and Provisioning Settings part``

  // Boot and Provisioning Settings
  http_port_min = var.common_data_source == "http" ? var.common_http_port_min : null
  http_port_max = var.common_data_source == "http" ? var.common_http_port_max : null
  http_content  = var.common_data_source == "http" ? local.data_source_content : null

  cd_label   = var.common_data_source == "cd" ? "cidata" : null
  cd_content = var.common_data_source == "cd" ? local.data_source_content : null

  boot_order       = var.vm_boot_order
  boot_wait        = var.vm_boot_wait
  boot_command     = ["<enter><enter><f6><esc><wait> ", "autoinstall ", "ip=dhcp ipv6.disable=1 ${local.data_source_command} ", "<enter><wait>"]

What do you think about it?

Kind regards,
Stéphane

[tenthirtyam]

I was planning to add this as an enhancement. If you'd like to make a pull request for this one for compatible Linux images, please clone to an enhancement issue and submit a pull. It's a clean approach.

Ryan

[tenthirtyam]

Merged with #59. Thanks, @sestegra!!!

Ryan