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

Fix metrics namespace not always being present #596

Merged
merged 3 commits into from
Sep 20, 2022

Conversation

XAMPPRocky
Copy link
Collaborator

Some metrics were missing the quilkin prefix, this fixes it so that can't happen.

@quilkin-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: b789b087-7087-4ad5-9399-bd8bf37ed8da

To build this version:

git fetch git@github.com:googleforgames/quilkin.git pull/596/head:pr_596 && git checkout pr_596
cargo build

@markmandel markmandel merged commit 657dc41 into main Sep 20, 2022
@markmandel markmandel deleted the ep/fix-metrics-prefix branch September 20, 2022 01:11
XAMPPRocky added a commit that referenced this pull request Sep 20, 2022
* Fix metrics namespace not always being present

Co-authored-by: Mark Mandel <markmandel@google.com>
XAMPPRocky added a commit that referenced this pull request Oct 10, 2022
* Fix metrics namespace not always being present

Co-authored-by: Mark Mandel <markmandel@google.com>
XAMPPRocky added a commit that referenced this pull request Oct 10, 2022
* Fix metrics namespace not always being present

Co-authored-by: Mark Mandel <markmandel@google.com>
@markmandel markmandel added kind/bug Something isn't working area/operations Installation, updating, metrics etc labels Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/operations Installation, updating, metrics etc kind/bug Something isn't working size/xs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants