Skip to content

fix: correct FailedDatabase counter for upsert_disbursement, expose api-server metrics#2548

Merged
mergify[bot] merged 3 commits intoethereum-optimism:developfrom
cfromknecht:fix-teleportr-failed-database-metric
May 6, 2022
Merged

fix: correct FailedDatabase counter for upsert_disbursement, expose api-server metrics#2548
mergify[bot] merged 3 commits intoethereum-optimism:developfrom
cfromknecht:fix-teleportr-failed-database-metric

Conversation

@cfromknecht
Copy link
Contributor

Description
Previously we would always increment the counter instead of adding the
number of failed upserts observed.

Metadata

  • Fixes ENG-2198

Previously we would always increment the counter instead of adding the
number of failed upserts observed.
@changeset-bot
Copy link

changeset-bot bot commented May 5, 2022

🦋 Changeset detected

Latest commit: ca7b766

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/teleportr Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mergify mergify bot requested review from mslipper and tuxcanfly May 5, 2022 22:02
@cfromknecht cfromknecht changed the title fix: correct FailedDatabase counter for upsert_disbursement fix: correct FailedDatabase counter for upsert_disbursement, expose metrics server May 5, 2022
@cfromknecht cfromknecht changed the title fix: correct FailedDatabase counter for upsert_disbursement, expose metrics server fix: correct FailedDatabase counter for upsert_disbursement, expose api-server metrics May 5, 2022
Copy link
Contributor

@Inphi Inphi left a comment

Choose a reason for hiding this comment

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

There are two changesets. Shouldn't there be only one summarizing both changes?

@mslipper
Copy link
Collaborator

mslipper commented May 6, 2022

Both changesets will be squashed by the changesets tool into one.

@mergify
Copy link
Contributor

mergify bot commented May 6, 2022

This PR has been added to the merge queue, and will be merged soon.

@mergify
Copy link
Contributor

mergify bot commented May 6, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify mergify bot merged commit 3263960 into ethereum-optimism:develop May 6, 2022
This was referenced May 10, 2022
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