Skip to content

Conversation

@JuArce
Copy link
Collaborator

@JuArce JuArce commented Apr 29, 2025

Add alerts for aggregation mode

Description

How to Test

  1. make anvil_start
  2. make batcher_start_local
  3. make batcher_send_risc0_burst BURST_SIZE=4
  4. Create and complete .env file in the following location ./alerts/.env
    CONTRACT_ADDRESS=0xB0D4afd8879eD9F52b28595d31B441D079B2Ca07
    AGGREGATED_PROOF_VERIFIED_TOPIC=0xfe3e9e971000ab9c80c7e06aba2933aae5419d0e44693e3046913e9e58053f62
    RPC_URL=http://localhost:8545
    LOGS_BLOCK_RANGE=50
    SLEEP_TIME=30
    PAGER_DUTY_KEY=YOUR_PAGER_DUTY_KEY
    PAGER_DUTY_EMAIL=YOUR_PAGER_DUTY_EMAIL
    PAGER_DUTY_SERVICE_ID=YOUR_PAGER_DUTY_SERVICE_ID
    SLACK_WEBHOOK_URL=YOUR_SLACK_WEBHOOK_URL
    TELEGRAM_BOT_TOKEN=YOUR_TELEGRAM_BOT_TOKEN
    TELEGRAM_CHAT_ID=YOUR_TELEGRAM_CHAT_ID
    
  5. Wait for at least to the block 50 in anvil
  6. Now run the alert script with ./alerts/aggregation_mode_alerts.sh ./alerts/.env
    You should see a message like
    🚨  ALERT Aggregation Mode: No new aggregated proofs since block 53
    
  7. AGGREGATOR=risc0 make start_proof_aggregator_dev
  8. After the AggregationMode finishes, you should see the following logs in the alerts script
    🟩  INFO Aggregation Mode: Aggregated proofs creation resumed since block 299
    

Note: You will see error logs when trying to send the alerts to slack/telegram, you can set your own values

Type of change

  • Infra

Checklist

  • “Hotfix” to testnet, everything else to staging
  • Linked to Github Issue
  • This change depends on code or research by an external entity
    • Acknowledgements were updated to give credit
  • Unit tests added
  • This change requires new documentation.
    • Documentation has been added/updated.
  • This change is an Optimization
    • Benchmarks added/run
  • Has a known issue
  • If your PR changes the Operator compatibility (Ex: Upgrade prover versions)
    • This PR adds compatibility for operator for both versions and do not change batcher/docs/examples
    • This PR updates batcher and docs/examples to the newer version. This requires the operator are already updated to be compatible

@JuArce JuArce self-assigned this Apr 29, 2025
@JuArce JuArce linked an issue Apr 29, 2025 that may be closed by this pull request
Copy link
Member

@MarcosNicolau MarcosNicolau left a comment

Choose a reason for hiding this comment

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

Worked for me. Note: I did not the telegram, slack. pagerduty integrations.

@MauroToscano MauroToscano added this pull request to the merge queue May 6, 2025
Merged via the queue into staging with commit d2c297b May 6, 2025
1 check passed
@MauroToscano MauroToscano deleted the 1902-infraalerts-add-alerts-for-aggregation_mode branch May 6, 2025 19:06
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.

infra(alerts): add alerts for aggregation_mode

4 participants