Skip to content

Updated solution to use AZD CLI with updated Bicep template for deployments#20

Open
kwkraus wants to merge 8 commits intoAzure-Samples:mainfrom
kwkraus:main
Open

Updated solution to use AZD CLI with updated Bicep template for deployments#20
kwkraus wants to merge 8 commits intoAzure-Samples:mainfrom
kwkraus:main

Conversation

@kwkraus
Copy link

@kwkraus kwkraus commented Aug 28, 2024

Purpose

This pull request includes significant updates to the CI/CD workflows, Azure Bicep files, and documentation to streamline the deployment process and improve the overall structure. The most important changes include the addition of a new GitHub Actions workflow for Azure deployment, removal of outdated workflows and Bicep files, and updates to the README to reflect these changes.

CI/CD Workflow Updates:

  • Added a new GitHub Actions workflow to deploy to Azure using azd CLI and federated credentials in .github/workflows/azure-dev.yml.
  • Removed the old deployment workflow in .github/workflows/deploy.yml.

Azure Bicep File Updates:

  • Removed Azure/container_app.bicep, Azure/environment.bicep, and Azure/main.bicep files, which are no longer needed with the new deployment process. [1] [2] [3]

Documentation Updates:

  • Updated the README to reflect the new deployment process using azd CLI and Docker Desktop, and updated the folder structure reference from "Azure" to "Infra". [1] [2]

Configuration Updates:

  • Added a new azure.yaml file to define the project structure and services for Azure deployment.

Does this introduce a breaking change?

[X] Yes
[ ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[X] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code and follow readme

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.

1 participant