Skip to content

chore: consistent service tagging for Datadog #222

Merged
scottgerring merged 2 commits into
mainfrom
chore/align-metadata
Mar 4, 2026
Merged

chore: consistent service tagging for Datadog #222
scottgerring merged 2 commits into
mainfrom
chore/align-metadata

Conversation

@scottgerring

@scottgerring scottgerring commented Mar 4, 2026

Copy link
Copy Markdown
Member

Summary

  • Standardize DD_SERVICE names to match directory names: sticker-servicesticker-catalogue, user-serviceuser-management, award-service
    sticker-award
  • Fix shared CDK migration-task construct to use the base service name instead of appending -migration suffix, aligning with Datadog Universal Service. This is the app-specific migration task
    Tagging
  • Set database-init Lambdas to a single database-init service name, since they're shared infrastructure rather than part of each application service. This is the thing that creates the schema and schema user for each service.
  • Fix OTel ActivitySource names in print-service to match DD_SERVICE (Stickerlandia.PrintServiceprint-service)
  • Fix copy-paste bug in print-service migration Dockerfile (was labeled user-service)
  • Fix sticker-award GORM trace plugin to use the logical database name from config instead of a hardcoded Docker hostname

@scottgerring scottgerring marked this pull request as ready for review March 4, 2026 08:06
@scottgerring scottgerring force-pushed the chore/align-metadata branch from fe12373 to 3090aa0 Compare March 4, 2026 08:14

@jeastham1993 jeastham1993 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@scottgerring scottgerring merged commit 8476e5f into main Mar 4, 2026
29 checks passed
@scottgerring scottgerring deleted the chore/align-metadata branch March 4, 2026 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants