Skip to content

fix: added post-deployment script/command visible in the terminal after deployment.#483

Merged
Roopan-Microsoft merged 3 commits into
devfrom
psl-postacrpushscript-vs
Jul 23, 2026
Merged

fix: added post-deployment script/command visible in the terminal after deployment.#483
Roopan-Microsoft merged 3 commits into
devfrom
psl-postacrpushscript-vs

Conversation

@VishalSh-Microsoft

Copy link
Copy Markdown
Contributor

Purpose

This pull request adds a new postprovision hook to the azure.yaml configuration, enhancing the developer experience after provisioning is complete. The hook provides clear instructions for the next steps and displays the web app URL, with tailored messages for both Windows and POSIX environments.

Enhancements to provisioning workflow:

  • Added a postprovision hook to azure.yaml that outputs a completion message, next-step instructions for building and pushing container images, and the web app URL after provisioning. The hook provides platform-specific guidance for both Windows (PowerShell) and POSIX (sh) environments.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds an azd postprovision hook to improve the post-provisioning developer experience by printing next-step guidance and the provisioned web app URL in the terminal.

Changes:

  • Added a postprovision hook to azure.yaml.
  • Implemented platform-specific messaging for Windows (PowerShell) and POSIX (sh).
  • Displays the WEB_APP_URL environment value after provisioning.
Comments suppressed due to low confidence (1)

azure.yaml:64

  • For consistency with the documented instructions (docs/DeploymentGuide.md), prefer bash scripts/build_and_push_images.sh (no leading ./) in the POSIX message so users can copy/paste the same command shown in the guide.
        echo "=> Next step: build and push container images to the dedicated ACR (remote build):"
        echo "  bash ./scripts/build_and_push_images.sh"

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread azure.yaml
Copilot AI review requested due to automatic review settings July 23, 2026 10:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

azure.yaml:66

  • $WEB_APP_URL may not be exported into the hook’s shell environment even though it exists in the azd environment. To avoid printing an empty URL, fall back to azd env get-value WEB_APP_URL when the variable is unset/empty (this matches the pattern used in scripts/build_and_push_images.sh).
        echo ""
        echo "Web app URL: $WEB_APP_URL"

Comment thread azure.yaml Outdated
Copilot AI review requested due to automatic review settings July 23, 2026 11:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@Roopan-Microsoft
Roopan-Microsoft merged commit 41ef09f into dev Jul 23, 2026
3 checks passed
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.

3 participants