Skip to content

fix: adding .metrics property and circuit breaker metrics#14

Merged
leftieFriele merged 1 commit intobetafrom
adding-metrics
Nov 4, 2024
Merged

fix: adding .metrics property and circuit breaker metrics#14
leftieFriele merged 1 commit intobetafrom
adding-metrics

Conversation

@leftieFriele
Copy link
Contributor

First attempt at adding some metrics on the circuit breaking behaviour.
Currently it adds a counter for all events coming from Opossum, we might want to expand on this later.

Copy link
Member

@digitalsadhu digitalsadhu left a comment

Choose a reason for hiding this comment

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

Looking good

@leftieFriele leftieFriele force-pushed the adding-metrics branch 6 times, most recently from e8f969a to 93f699a Compare November 4, 2024 11:17
@leftieFriele leftieFriele marked this pull request as ready for review November 4, 2024 11:34
@leftieFriele leftieFriele merged commit 09d9e47 into beta Nov 4, 2024
@leftieFriele leftieFriele deleted the adding-metrics branch November 4, 2024 12:26
@github-actions
Copy link

github-actions bot commented Nov 4, 2024

🎉 This PR is included in version 1.0.0-beta.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants