Skip to content

chore: improve logs TLD functions#1212

Merged
davidgamez merged 4 commits intomainfrom
chore/improve_logs_tld
May 27, 2025
Merged

chore: improve logs TLD functions#1212
davidgamez merged 4 commits intomainfrom
chore/improve_logs_tld

Conversation

@davidgamez
Copy link
Copy Markdown
Member

Summary:

Part of #1027
This PR updates the logging calls to the new standardized pattern on the tld functions.

Expected behavior:

The Logger class is no longer used in favour of the newly added logging functions.

Testing tips:

This can only be visible in the life environment.

  • Deploy to DEV environment
  • Tigger the functions
  • Verify that the logs are properly formatted with trace and span IDs.

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run the unit tests with ./scripts/api-tests.sh to make sure you didn't break anything
  • Add or update any needed documentation to the repo
  • Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
  • Linked all relevant issues
  • Include screenshot(s) showing how this pull request works and fixes the issue(s)

@davidgamez davidgamez changed the title Chore/improve logs tld chore: improve logs TLD functions May 23, 2025
@davidgamez
Copy link
Copy Markdown
Member Author

Log records:

feed-sync-dispatcher-transitland-batch
Screenshot 2025-05-26 at 4 17 23 PM

feed-sync-process-transitland-pubsub
Screenshot 2025-05-26 at 4 25 44 PM

@davidgamez davidgamez merged commit fe4ea3e into main May 27, 2025
8 of 9 checks passed
@davidgamez davidgamez deleted the chore/improve_logs_tld branch May 27, 2025 20:08
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