Skip to content

Conversation

@mpaulucci
Copy link
Collaborator

Motivation
To make it easier to debug if something goes badly

Description

  • Sets the job to be run every 3 hours
  • Makes a more strict timeout of 2hrs for Sepolia to it's easier to catch regressions
  • Sets the default log level to debug

@mpaulucci mpaulucci requested a review from a team as a code owner December 5, 2025 17:30
Copilot AI review requested due to automatic review settings December 5, 2025 17:30
@github-actions github-actions bot added the L1 Ethereum client label Dec 5, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR modifies the CI configuration to run snapsync jobs more frequently with enhanced debugging capabilities. The changes shift from a weekday-only daily schedule to running every 3 hours continuously, implement stricter timeouts for the Sepolia network to catch performance regressions earlier, and enable debug-level logging for better troubleshooting.

Key Changes:

  • Updated cron schedule from weekdays at 3AM UTC to every 3 hours, 24/7
  • Reduced Sepolia timeout from 10 hours to 2 hours for faster regression detection
  • Added --log.level=debug parameter and upgraded lighthouse from v8.0.0-rc.1 to v8.0.1

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/daily_snapsync.yaml Updates job schedule to run every 3 hours and reduces Sepolia sync timeout from 10h to 2h in both scheduled and manual dispatch scenarios
.github/actions/snapsync-run/action.yml Adds debug logging flag to execution layer parameters and upgrades lighthouse consensus client to stable v8.0.1 release

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-project-automation github-project-automation bot moved this to In Review in ethrex_l1 Dec 5, 2025
@MegaRedHand MegaRedHand added this pull request to the merge queue Dec 5, 2025
Merged via the queue into main with commit a57853b Dec 5, 2025
55 checks passed
@MegaRedHand MegaRedHand deleted the set-snapsync-log-debug branch December 5, 2025 18:33
@github-project-automation github-project-automation bot moved this from In Review to Done in ethrex_l1 Dec 5, 2025
rodrigo-o added a commit that referenced this pull request Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

L1 Ethereum client

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants