Skip to content
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

Update log format for release service on Fastly #567

Merged
merged 5 commits into from
Sep 23, 2024
Merged

Conversation

jdno
Copy link
Member

@jdno jdno commented Sep 11, 2024

The log format of the Fastly service for releases has been slightly changed to set Datadog's unified tags1. This makes it easier to work with the logs in Datadog's UI.

Footnotes

  1. https://docs.datadoghq.com/getting_started/tagging/unified_service_tagging

@jdno jdno force-pushed the log-release-traffic branch from 435d297 to 5725947 Compare September 11, 2024 20:14
The log format of the Fastly service for releases has been slightly
changed to set Datadog's unified tags[^1]. This makes it easier to work
with the logs in Datadog's UI.

[^1]: https://docs.datadoghq.com/getting_started/tagging/unified_service_tagging
We use a hierarchy of tags on Datadog to provide different means for
filtering metrics and logs. All logs from a particular piece of
infrastructure share the same `service` tag and then have different tags
for the `env`. In this specific case, we want to tag everything as
coming from the `static.rust-lang.org` service and then differentiate
between the environments using the `dev` and `prod` valus for the
environment.
The `host` attribute in the logs was not set properly, but is now being
populated properly based in different information available in Fastly.
@jdno jdno force-pushed the log-release-traffic branch from d873700 to 132eb89 Compare September 23, 2024 11:06
@jdno
Copy link
Member Author

jdno commented Sep 23, 2024

Rebased this to resolve a merge conflict with #569.

@jdno jdno merged commit c171bbe into master Sep 23, 2024
3 checks passed
@jdno jdno deleted the log-release-traffic branch September 23, 2024 11:10
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.

2 participants