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

Update Bicep form with pre-deployment input validation #482

Closed
glennmusa opened this issue Oct 25, 2021 · 1 comment · Fixed by #577
Closed

Update Bicep form with pre-deployment input validation #482

glennmusa opened this issue Oct 25, 2021 · 1 comment · Fixed by #577
Assignees
Labels
bug Something isn't working

Comments

@glennmusa
Copy link
Contributor

Description

Deploying using the marketplace UI template fails with this error:

{
    "status": "Failed",
    "error": {
        "code": "InvalidResourceLocation",
        "message": "The specified location '[parameters('sharedServicesLocation')]' is invalid. A location must consist of characters, whitespace, digit, or following symbols '(,)'."
    }
}

Steps to Reproduce

Try to deploy from the Marketplace UI at src/bicep/ui/ like so:

Deploy To Azure

Steps to reproduce the behavior:

  1. Click the Deploy to Azure button
  2. Receive an error about shared services location

Expected behavior

Deployment succeeds without modifying any input

Actual behavior

Deployment fails on InvalidResourceLocation

Screenshots

image

Additional context

n/a

@glennmusa glennmusa added bug Something isn't working needs triage labels Oct 25, 2021
@brooke-hamilton
Copy link
Contributor

Add a checkbox to the PR template that asks if the UI template has been updated when a parameter is added or changed.

@glennmusa glennmusa self-assigned this Nov 4, 2021
@glennmusa glennmusa changed the title UI deployment from marketplace template fails with InvalidResourceLocation on Log Analytics Workspace creation Update Bicep form with pre-deployment input validation Dec 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants