Skip to content

Update containerDeployment.yml #20

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

Merged
merged 1 commit into from
Apr 22, 2025
Merged

Update containerDeployment.yml #20

merged 1 commit into from
Apr 22, 2025

Conversation

aminespinoza10
Copy link
Collaborator

Closes: #19

This pull request updates the .github/workflows/containerDeployment.yml file to enhance the deployment workflow for a containerized application. The changes include the addition of environment variables, integration with Azure for deployment, and updates to Docker image handling.

Workflow Enhancements:

  • Updated the IMAGE_BASE_NAME environment variable to include the :latest tag and added new variables RESOURCE_GROUP and ENVIRONMENT_NAME to support Azure deployment. ([.github/workflows/containerDeployment.ymlL8-R10](https://github.com/platzi/curso-devops/pull/20/files#diff-202cd0851de712b3eccd6653d0355a786699e33de55c3aa455af5ea58aaa047bL8-R10))

Azure Integration:

  • Added a step for Azure login using the Azure/[email protected] action and configured it with credentials from GitHub secrets. ([.github/workflows/containerDeployment.ymlR22-R30](https://github.com/platzi/curso-devops/pull/20/files#diff-202cd0851de712b3eccd6653d0355a786699e33de55c3aa455af5ea58aaa047bR22-R30))
  • Included a step to install the Azure CLI containerapp extension with dynamic installation enabled. ([.github/workflows/containerDeployment.ymlR22-R30](https://github.com/platzi/curso-devops/pull/20/files#diff-202cd0851de712b3eccd6653d0355a786699e33de55c3aa455af5ea58aaa047bR22-R30))

Docker and Azure Deployment:

  • Modified the Docker push command to remove the explicit :latest tag, aligning it with the updated IMAGE_BASE_NAME. ([.github/workflows/containerDeployment.ymlL32-R47](https://github.com/platzi/curso-devops/pull/20/files#diff-202cd0851de712b3eccd6653d0355a786699e33de55c3aa455af5ea58aaa047bL32-R47))
  • Added a new step to deploy the containerized application to Azure Container Apps using the az containerapp up command. ([.github/workflows/containerDeployment.ymlL32-R47](https://github.com/platzi/curso-devops/pull/20/files#diff-202cd0851de712b3eccd6653d0355a786699e33de55c3aa455af5ea58aaa047bL32-R47))

@aminespinoza10 aminespinoza10 requested a review from gndx April 22, 2025 15:02
@aminespinoza10 aminespinoza10 self-assigned this Apr 22, 2025
Copy link
Member

@gndx gndx left a comment

Choose a reason for hiding this comment

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

LGTM

@aminespinoza10 aminespinoza10 merged commit 9ea254c into main Apr 22, 2025
1 check passed
@aminespinoza10 aminespinoza10 deleted the amines/19 branch April 22, 2025 15:03
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.

Desplegar Container App
2 participants