Skip to content
This repository has been archived by the owner on Mar 30, 2023. It is now read-only.

Prefix loadbalancers, application gateway and ip addresses with vmHostNamePrefix #285

Merged
merged 8 commits into from
Jun 11, 2019

Conversation

russcam
Copy link
Contributor

@russcam russcam commented Jun 7, 2019

This PR updates the load balancer, application gateway and ip address resources to be prefixed with
the vmHostNamePrefix value. This would allow a cluster to be deployed into an existing resource group where another cluster has been deployed with a different vmHostNamePrefix value.

The vnet is not prefixed with vmHostNamePrefix because its name is an input parameter, so can be passed prefixed.

Fixes #278

russcam added 7 commits June 7, 2019 16:20
…tNamePrefix

This commit updates the load balancer, application gateway and ip address resources to be prefixed with
the vmHostNamePrefix value. This would allow a cluster to be deployed into an existing resource group where
another cluster has been deployed with a different vmHostNamePrefix value.

Fixes #278
Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, agreed with the decision to not autoprefix the vnet name

@russcam russcam merged commit 3019df6 into 7.1 Jun 11, 2019
russcam added a commit that referenced this pull request Jun 11, 2019
…tNamePrefix (#285)

This commit updates the load balancer, application gateway and ip address resources to be prefixed with
the vmHostNamePrefix value. This would allow a cluster to be deployed into an existing resource group where
another cluster has been deployed with a different vmHostNamePrefix value.

Fixes #278
russcam added a commit that referenced this pull request Jun 11, 2019
…tNamePrefix (#285)

This commit updates the load balancer, application gateway and ip address resources to be prefixed with
the vmHostNamePrefix value. This would allow a cluster to be deployed into an existing resource group where
another cluster has been deployed with a different vmHostNamePrefix value.

Fixes #278
russcam added a commit that referenced this pull request Jun 11, 2019
This commit updates the app-gateway resource name used to validate a deployment, to align with changes in #285
russcam added a commit that referenced this pull request Jun 14, 2019
This commit updates the app-gateway resource name used to validate a deployment, to align with changes in #285
@russcam russcam deleted the fix/prefix-resources branch June 26, 2019 21:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants