Skip to content

AG-885: Add service metadata to logs#457

Merged
LindvedKrvang merged 5 commits intomainfrom
AG-885/add-service-metadata-to-logs
Dec 22, 2025
Merged

AG-885: Add service metadata to logs#457
LindvedKrvang merged 5 commits intomainfrom
AG-885/add-service-metadata-to-logs

Conversation

@KvelaGorrrrnio
Copy link
Contributor

@KvelaGorrrrnio KvelaGorrrrnio commented Dec 16, 2025

Warning

This PR has changes from #456.

Context

In order to get our logs into DataDog via the Observability teams pipeline, we need to set the entries service.name and service.version in our logs.

Change description

The PR updates the create revision build to look for the next CalVer version, such that the release version can be built into the Docker image. The Dockerfile is changed to expose the release version build argument as an environment variable to the application. And lastly, an ApplicationMetadataGenerator is added that extracts the service metadata and composes it into a metadata object that is registered on the top-most logger instance.

Warning

Running the Create Revision Build workflow from a feature branch will set the RELEASE_VERSION to an empty string.

Checklist

  • The changes are documented.
  • Core logic is covered by tests.

rana-tv2 and others added 4 commits December 12, 2025 14:08
The workflow might have an error if we run it from a feature branch.
It extracts ENV variable information and creates a metadata object.
The metadata additions added in this commit are service.name and service.version.
@github-actions
Copy link

github-actions bot commented Dec 16, 2025

Coverage Report for tv2/alba-server

Status Category Percentage Covered / Total
🔵 Lines 74.5% 2899 / 3891
🔵 Statements 74.32% 3028 / 4074
🔵 Functions 69.62% 832 / 1195
🔵 Branches 67.67% 741 / 1095
File CoverageNo changed files found.
Generated in workflow #1386 for commit c294335 by the Vitest Coverage Report Action

@KvelaGorrrrnio KvelaGorrrrnio changed the title Ag 885/add service metadata to logs AG-885: Add service metadata to logs Dec 16, 2025
@LindvedKrvang LindvedKrvang merged commit f040ae1 into main Dec 22, 2025
5 checks passed
@LindvedKrvang LindvedKrvang deleted the AG-885/add-service-metadata-to-logs branch December 22, 2025 08:24
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.

3 participants