Skip to content

chore(test): just ensure it had more than 0 pipeline runs#676

Merged
joshieDo merged 3 commits intomainfrom
joshie/backfill-e2e-2
Nov 4, 2025
Merged

chore(test): just ensure it had more than 0 pipeline runs#676
joshieDo merged 3 commits intomainfrom
joshie/backfill-e2e-2

Conversation

@joshieDo
Copy link
Copy Markdown
Contributor

@joshieDo joshieDo commented Nov 4, 2025

Backfill might be triggered more than once if chain progresses fast or the node is slow during tests.

https://github.com/tempoxyz/tempo/actions/runs/19080423872/job/54507240269

thread 'tests::backfill::validator_can_join_later_with_pipeline_sync' (50171) panicked at crates/e2e/src/tests/backfill.rs:62:5:
Expected 1 backfill runs, got 2

@joshieDo joshieDo requested a review from SuperFluffy November 4, 2025 19:45
@joshieDo joshieDo changed the title chore(test): just ensure it had more than 0 backfill runs chore(test): just ensure it had more than 0 pipeline runs Nov 4, 2025
Comment thread crates/e2e/src/tests/backfill.rs Outdated
Comment thread crates/e2e/src/tests/backfill.rs Outdated
joshieDo and others added 2 commits November 4, 2025 19:54
Co-authored-by: Richard Janis Goldschmidt <github@aberrat.io>
@joshieDo joshieDo enabled auto-merge (squash) November 4, 2025 19:55
@joshieDo joshieDo merged commit 6a66e76 into main Nov 4, 2025
15 checks passed
@joshieDo joshieDo deleted the joshie/backfill-e2e-2 branch November 4, 2025 20:01
unbalancedparentheses pushed a commit to unbalancedparentheses/tempo that referenced this pull request Feb 23, 2026
Backfill might be triggered more than once if chain progresses fast or
the node is slow during tests.


https://github.com/tempoxyz/tempo/actions/runs/19080423872/job/54507240269
> thread 'tests::backfill::validator_can_join_later_with_pipeline_sync'
(50171) panicked at crates/e2e/src/tests/backfill.rs:62:5:
    Expected 1 backfill runs, got 2

---------

Co-authored-by: Richard Janis Goldschmidt <github@aberrat.io>
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