Skip to content

Conversation

@lerouxb
Copy link
Contributor

@lerouxb lerouxb commented Jan 31, 2025

This is with me hardcoding a throw new Error('This is a test'):

{"t":{"$date":"2025-01-31T10:48:37.186Z"},"s":"E","c":"COMPASS-TELEMETRY","id":1001000339,"ctx":"getPublicCloudInfo","msg":"Error fetching cloud info","attr":"This is a test"}

NOTE: the one logging id got deduped in #6677

@lerouxb lerouxb added no release notes Fix or feature not for release notes no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) labels Jan 31, 2025
@lerouxb
Copy link
Contributor Author

lerouxb commented Jan 31, 2025

@lerouxb lerouxb merged commit 54f1ccc into main Feb 4, 2025
28 of 32 checks passed
@lerouxb lerouxb deleted the log-cloud-info-errors branch February 4, 2025 11:20
@lerouxb lerouxb changed the title chore(telemetry): log when getting public cloud info fails chore(telemetry): log when getting public cloud info or resolving srv fails Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no release notes Fix or feature not for release notes no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants