Skip to content

Infraestructura agregada #15

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

Infraestructura agregada #15

merged 1 commit into from
Apr 22, 2025

Conversation

aminespinoza10
Copy link
Collaborator

This pull request introduces several infrastructure changes to set up and configure an Azure environment using Terraform. Key updates include the addition of backend configuration, provider setup, resource definitions, and variable declarations. The changes also update documentation to reflect the new infrastructure setup.

Infrastructure Setup

  • Backend Configuration: Added a Terraform backend configuration in infra/backend.tf to store the state file in an Azure Storage Account.
  • Provider Configuration: Defined the required Terraform version and Azure provider in infra/provider.tf, specifying the provider version and subscription ID.

Resource Definitions

  • Resource Group and Dependencies: Added resources in infra/main.tf to create a resource group, log analytics workspace, application insights, and a container app environment. These resources are interconnected using variables and references.

Variable Declarations

  • Variables for Resource Configuration: Declared variables in infra/variables.tf for resource group name prefix, location, Azure Container Registry (ACR) name, and container app environment name. Default values are provided for each variable.

Documentation Update

  • Updated Example Code: Modified docs/clase12.md to reflect the new infrastructure setup, replacing the Azure Container Registry resource with Application Insights and updating the example code accordingly. [1] [2]

@aminespinoza10 aminespinoza10 requested a review from gndx April 22, 2025 14:30
@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 4012198 into main Apr 22, 2025
1 check passed
@aminespinoza10 aminespinoza10 deleted the amines/14 branch April 22, 2025 14:31
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