Skip to content

fix: one click deployment github action issue#100

Merged
Roopan-Microsoft merged 2 commits intomainfrom
psl-task-18125
May 16, 2025
Merged

fix: one click deployment github action issue#100
Roopan-Microsoft merged 2 commits intomainfrom
psl-task-18125

Conversation

@Prashant-Microsoft
Copy link
Copy Markdown
Contributor

@Prashant-Microsoft Prashant-Microsoft commented May 14, 2025

Purpose

This pull request enhances the deployment workflow by introducing dynamic resource handling, including the generation of unique solution prefixes, integration of OpenAI resources, and improved resource cleanup processes. Below is a breakdown of the most important changes:

Resource Management Enhancements:

  • Added a step to generate a unique solution prefix using a timestamp, which is dynamically assigned to the Prefix parameter during deployment. (.github/workflows/deploy.yml, .github/workflows/deploy.ymlR54-R73)
  • Updated the resource-fetching step to include the retrieval of OpenAI resources from the resource group, ensuring their availability for subsequent operations. (.github/workflows/deploy.yml, [1] [2]

Resource Cleanup Improvements:

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.

@Roopan-Microsoft Roopan-Microsoft merged commit 1460d6c into main May 16, 2025
5 of 6 checks passed
@Roopan-Microsoft Roopan-Microsoft deleted the psl-task-18125 branch May 16, 2025 03:33
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Ritesh-Microsoft pushed a commit that referenced this pull request Oct 10, 2025
fix: one click deployment github action issue
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