chore: a quick fix for the AVD docs #116
Conversation
| - green means only virtual desktop green is deployed whilst virtual desktop blue is removed. Users are directed to group green. | ||
| - both-blue-primary means both virtual desktop groups are deployed, but ONLY the platform users can see group green. All other users will be directed to group blue. | ||
| - both-green-primary means both virtual desktop groups are deployed, but ONLY the platform users can see group blue. All other users will be directed to group green. | ||
| - both-with-blue-primary means both virtual desktop groups are deployed, but ONLY the platform users can see group green. All other users will be directed to group blue. |
There was a problem hiding this comment.
Can you check in the variables.tf file as well?
variable "virtual_desktop_group_active" {
description = <<-EOT
This can either be 'blue', 'green', 'both-with-blue-primary', 'both-with-green-primary', 'both-with-blue-primary-but-equal-vms' or 'both-with-green-primary-but-equal-vms'.
blue means only virtual desktop blue is deployed whilst virtual desktop green is removed.
green means only virtual desktop green is deployed whilst virtual desktop blue is removed. Users are directed to group green.
both-blue-primary means both virtual desktop groups are deployed, but ONLY the platform users can see group green. All other users will be directed to group blue.
both-green-primary means both virtual desktop groups are deployed, but ONLY the platform users can see group blue. All other users will be directed to group green.
both-with-blue-primary-but-equal-vms means both virtual desktop groups are deployed with equal VM counts, but ONLY the platform users can see group green. All other users will be directed to group blue.
both-with-green-primary-but-equal-vms means both virtual desktop groups are deployed with equal VM counts, but ONLY the platform users can see group blue. All other users will be directed to group green.
EOT
type = string
validation {
condition = contains(["blue", "green", "both-with-blue-primary", "both-with-green-primary", "both-with-blue-primary-but-equal-vms", "both-with-green-primary-but-equal-vms"], var.virtual_desktop_group_active)
error_message = "The virtual_desktop_group_active variable must be blue of: 'blue', 'green', 'both-with-blue-primary', 'both-with-green-primary', 'both-with-blue-primary-but-equal-vms', 'both-with-green-primary-but-equal-vms'."
}
}
There was a problem hiding this comment.
I have checked, the change I'm proposing is in line with the variables.tf and the variable validation rules.
Using the old version, before the fix, results in an error during the terraform plan.
2 participants
Description
A fix for the AVD update docs that currently has the wrong configuration for deploying both AVD host pools.
Old config:
New correct config:
Context
Type of changes
Checklist
Sensitive Information Declaration
To ensure the utmost confidentiality and protect your and others privacy, we kindly ask you to NOT including PII (Personal Identifiable Information) / PID (Personal Identifiable Data) or any other sensitive data in this PR (Pull Request) and the codebase changes. We will remove any PR that do contain any sensitive information. We really appreciate your cooperation in this matter.