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

Consume Content Store logs from new location #7

Merged
merged 1 commit into from
Dec 21, 2023

Conversation

mike29736
Copy link
Contributor

The app name "content-store" was being squatted by the Content Store Proxy for that past few months, so we've been consuming Content Store log data using its temporary name, "content-store-mongo-main". The proxy was removed from production today*, leaving the "content-store" app name referring to the real Content Store, so we need to update how we query Logit.

*alphagov/govuk-helm-charts#1598

https://trello.com/c/jDSDGNyn/814-measure-and-record-our-publishing-latency-sli

The app name "content-store" was being squatted by the Content Store
Proxy for that past few months, so we've been consuming Content Store
log data using its temporary name, "content-store-mongo-main". The proxy
was removed from production today*, leaving the "content-store" app name
referring to the real Content Store, so we need to update how we query
Logit.

*alphagov/govuk-helm-charts#1598
Copy link
Contributor

@JonathanHallam JonathanHallam left a comment

Choose a reason for hiding this comment

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

lgtm

@mike29736 mike29736 merged commit 9275569 into main Dec 21, 2023
3 checks passed
@mike29736 mike29736 deleted the consume-new-content-store-logs branch December 21, 2023 15:20
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