Added Service Fabric Reverse Proxy Certificate and Endpoint in Nodetype

[joranm]

Added reverse_proxy_certificate and reverse_proxy_endpoint_port to the azurerm_service_fabric_cluster resource.

resource "azurerm_service_fabric_cluster" "sf" {
  name                = "test-app-sf"
  resource_group_name = "${azurerm_resource_group.test.name}"
  location            = "${azurerm_resource_group.test.location}"
  reliability_level   = "Bronze"
  upgrade_mode        = "Automatic"
  vm_image            = "Windows"

  management_endpoint = "https://${azurerm_lb.test.private_ip_address}:19080"
  add_on_features     = ["DnsService"]

  certificate {
    thumbprint      = "${azurerm_key_vault_certificate.test.thumbprint}"
    x509_store_name = "My"
  }

  reverse_proxy_certificate {
    thumbprint      = "${azurerm_key_vault_certificate.test.thumbprint}"
    x509_store_name = "My"
  }

  diagnostics_config {
    storage_account_name       = "${azurerm_storage_account.sflogs.name}"
    protected_account_key_name = "StorageAccountKey1"
    blob_endpoint              = "${azurerm_storage_account.sflogs.primary_blob_endpoint}"
    queue_endpoint             = "${azurerm_storage_account.sflogs.primary_queue_endpoint}"
    table_endpoint             = "${azurerm_storage_account.sflogs.primary_table_endpoint}"
  }
  fabric_settings {
    name = "Security"

    parameters = {
      ClusterProtectionLevel = "EncryptAndSign"
    }
  }
  node_type {
    name                 = "backend"
    durability_level     = "Bronze"
    instance_count       = 3
    is_primary           = true
    client_endpoint_port = 19000
    http_endpoint_port   = 19080

    reverse_proxy_endpoint_port = 19081

    ephemeral_ports {
      start_port = 49152
      end_port   = 65534
    }
    application_ports {
      start_port = 20000
      end_port   = 30000
    }
  }
  tags {
    resourceType = "Service Fabric"
    clusterName  = "jms-test-app-sf"
  }
}

[katbyte]

Hi @joranm,

Thank you for this contribution! The tests pass and it LGTM aside from the lack of documentation. Once we get that updated it should be good to merge 🙂

[joranm]

Hi @katbyte, updated the website documentation according to the additions of the reverse proxy certificate and endpoint.

[katbyte]

Thanks @joranm, LGTM now 🙂

[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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!