Skip to content

Conversation

jayantk
Copy link
Contributor

@jayantk jayantk commented May 1, 2025

Summary

We want to alert on this log message

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

@vercel
Copy link

vercel bot commented May 1, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api-reference ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2025 1:46pm
component-library ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2025 1:46pm
entropy-debugger ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2025 1:46pm
insights ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2025 1:46pm
proposals ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2025 1:46pm
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2025 1:46pm

let outstanding_requests =
provider_info.sequence_number - provider_info.current_commitment_sequence_number;
if outstanding_requests > threshold {
// NOTE: This log message triggers a a grafana alert. If you want to change the text, please change the alert also.
Copy link
Contributor

Choose a reason for hiding this comment

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

a a grafana alert -> a grafana alert

let contract_call = contract.withdraw_as_fee_manager(provider_address, fees);
send_and_confirm(contract_call).await?;
} else if keeper_balance < min_balance {
// NOTE: This log message triggers a a grafana alert. If you want to change the text, please change the alert also.
Copy link
Contributor

Choose a reason for hiding this comment

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

a a grafana alert -> a grafana alert

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