Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CaseSensitiveDeploymentParameterNamesFound #457

Merged
merged 4 commits into from
Feb 24, 2023
Merged

Conversation

sangling
Copy link
Contributor

The deployment parameters are using case sensitive names. The error parameter name(s): parAllowHubVpnGatewayTransit,parSpokeToHubRouteTableName

Overview/Summary

Fixed parameter cases in Orchestration - hubPeeredSpoke which was causing CaseSensitiveDeploymentParameterNamesFound error

This PR fixes/adds/changes/removes

  1. Corrected case of parSpokeToHubRouteTableName in hubPeeredSpoke.parameters.all.json
  2. Corrected case of parAllowHubVpnGatewayTransit in hubPeeredSpoke.parameters.all.json
  3. Corrected case of parSpokeToHubRouteTableName in hubPeeredSpoke.vwan.parameters.all.json
  4. Corrected case of parAllowHubVpnGatewayTransit in hubPeeredSpoke.vwan.parameters.all.json

Breaking Changes

  1. CaseSensitiveDeploymentParameterNamesFound - The deployment parameters are using case sensitive names. The error parameter name(s): parSpokeToHubRouteTableName,parAllowHubVpnGatewayTransit

Testing Evidence

Error:
image

Tested:
image

As part of this Pull Request I have

The deployment parameters are using case sensitive names. The error parameter name(s): parAllowHubVpnGatewayTransit, parSpokeToHubRouteTableName
@ghost ghost added the Needs: Triage 🔍 Needs triaging by the team label Feb 22, 2023
@sangling
Copy link
Contributor Author

@microsoft-github-policy-service agree

@jtracey93 jtracey93 added bug and removed Needs: Triage 🔍 Needs triaging by the team labels Feb 23, 2023
Copy link
Collaborator

@jtracey93 jtracey93 left a comment

Choose a reason for hiding this comment

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

Thanks for this @sangling LGTM. Will test and merge if all okay

@jtracey93
Copy link
Collaborator

/azp run validateazcloud

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jtracey93 jtracey93 merged commit e3dd910 into Azure:main Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants