Skip to content

Conversation

@avilagaston9
Copy link
Contributor

@avilagaston9 avilagaston9 commented Nov 11, 2025

Motivation

The l2-dev workflow is flaky and we don't have all logs available for debugging.

Description

Print logs from the first line instead.

Closes None

@avilagaston9 avilagaston9 marked this pull request as ready for review November 11, 2025 21:32
@avilagaston9 avilagaston9 requested a review from a team as a code owner November 11, 2025 21:32
Copilot AI review requested due to automatic review settings November 11, 2025 21:32
@avilagaston9 avilagaston9 moved this to In Review in ethrex_l2 Nov 11, 2025
Copilot finished reviewing on behalf of avilagaston9 November 11, 2025 21:33
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 enhances debugging capabilities for the flaky l2-dev workflow by printing all logs from the beginning instead of just the last 100 lines.

  • Changes tail -n 100 to tail -n +1 to print logs from the first line instead of the last 100 lines
  • Simplifies the env variable export command by replacing shell cat with direct redirection

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

@avilagaston9 avilagaston9 changed the title ci(l2): print all logs in l2 dev workflow ci(l2): show all logs in l2 dev workflow Nov 11, 2025
@avilagaston9 avilagaston9 added this pull request to the merge queue Nov 12, 2025
Merged via the queue into main with commit 5c3a902 Nov 12, 2025
40 checks passed
@avilagaston9 avilagaston9 deleted the ci/l2/l2_dev_print_logs branch November 12, 2025 14:15
@github-project-automation github-project-automation bot moved this from In Review to Done in ethrex_l2 Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

L2 Rollup client

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants