Skip to content

Amines/21 #22

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 4 commits into from
Apr 22, 2025
Merged

Amines/21 #22

merged 4 commits into from
Apr 22, 2025

Conversation

aminespinoza10
Copy link
Collaborator

This pull request introduces several updates across multiple files, focusing on improving the ContactosAPI project with new features, telemetry integration, and updates to class materials. Below is a summary of the most important changes grouped by theme:

Telemetry and Monitoring Enhancements

  • Integrated OpenTelemetry for metrics and tracing in ContactosAPI by adding packages (Azure.Monitor.OpenTelemetry.Exporter, OpenTelemetry.Extensions.Hosting, OpenTelemetry.Instrumentation.AspNetCore, and OpenTelemetry.Instrumentation.Http) and updating Program.cs to configure telemetry services. [1] [2]

New Features in ContactosAPI

  • Added a new /obtenercontactos endpoint to retrieve a list of contacts and included an associated test case (ContactosTest) to validate the functionality.

Updates to Class Materials

  • Class 13: Fixed a syntax issue in the Terraform init command by correcting the backend-config parameter.
  • Class 14: Added detailed class material, including a GitHub Actions workflow for deploying a Docker-based API to Azure Container Apps.
  • Class 15: Provided instructions for adding OpenTelemetry NuGet packages, creating appsettings.json for Azure Monitor, and updating Program.cs for telemetry integration.
  • Class 16: Included sample KQL queries for monitoring Azure resources.
  • Class 18: Added a new method for the /obtenercontactos endpoint and instructions for writing a test case.

These changes improve the functionality, observability, and educational content of the project while ensuring better deployment practices and monitoring capabilities.

Close #21

@aminespinoza10 aminespinoza10 requested a review from gndx April 22, 2025 15:35
@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 50c3a2e into main Apr 22, 2025
1 check 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.

Agregar monitoreo usando Open Telemetry
2 participants