Skip to content

feat(pipeline): handle pipeline prunes#21250

Open
Maddiaa0 wants to merge 1 commit intomd/pipeline-timetablefrom
md/pipeline-recovery-2
Open

feat(pipeline): handle pipeline prunes#21250
Maddiaa0 wants to merge 1 commit intomd/pipeline-timetablefrom
md/pipeline-recovery-2

Conversation

@Maddiaa0
Copy link
Member

@Maddiaa0 Maddiaa0 commented Mar 9, 2026

Overview

Revert the pending chain whenever a pipeline does not get checkpointed onto L1

@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from 8259a2c to adaa2e9 Compare March 9, 2026 15:35
@Maddiaa0 Maddiaa0 marked this pull request as ready for review March 9, 2026 15:36
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from adaa2e9 to 4a45f77 Compare March 9, 2026 15:40
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-timetable branch from 2a965f2 to c2df8a2 Compare March 9, 2026 16:01
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from 4a45f77 to 2337bcc Compare March 9, 2026 16:01
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-timetable branch from c2df8a2 to 6ccc88b Compare March 9, 2026 16:07
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from 2337bcc to 4c7e8e6 Compare March 9, 2026 16:07
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-timetable branch from 6ccc88b to d349239 Compare March 9, 2026 16:41
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from 4c7e8e6 to 58cfbc5 Compare March 9, 2026 16:41
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch 2 times, most recently from 36f99e9 to 9511446 Compare March 12, 2026 15:52
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-timetable branch from d349239 to 24f6555 Compare March 12, 2026 15:52
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from 9511446 to f3e3e68 Compare March 16, 2026 18:08
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-timetable branch from 24f6555 to cb21eb9 Compare March 16, 2026 18:08
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from f3e3e68 to 829607e Compare March 18, 2026 11:33
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-timetable branch from cb21eb9 to 45ff14f Compare March 18, 2026 11:33
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from 829607e to 87da649 Compare March 18, 2026 11:59
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-timetable branch from 45ff14f to 456e3e9 Compare March 18, 2026 11:59
@Maddiaa0 Maddiaa0 changed the base branch from md/pipeline-timetable to graphite-base/21250 March 19, 2026 01:04
@Maddiaa0 Maddiaa0 force-pushed the graphite-base/21250 branch from 456e3e9 to f9ca5bf Compare March 19, 2026 01:07
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from 87da649 to 884b61b Compare March 19, 2026 01:07
@Maddiaa0 Maddiaa0 changed the base branch from graphite-base/21250 to md/pipeline-timetable March 19, 2026 01:07
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from 884b61b to 1936112 Compare March 19, 2026 01:20
@Maddiaa0 Maddiaa0 changed the base branch from md/pipeline-timetable to graphite-base/21250 March 19, 2026 14:24
@Maddiaa0 Maddiaa0 force-pushed the graphite-base/21250 branch from f9ca5bf to e50ad91 Compare March 19, 2026 18:16
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from 1936112 to 7d3123e Compare March 19, 2026 18:16
@Maddiaa0 Maddiaa0 changed the base branch from graphite-base/21250 to md/pipeline-timetable March 19, 2026 18:16
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch 2 times, most recently from fdebf1f to 9462d6c Compare March 19, 2026 21:03
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-timetable branch from e50ad91 to 890ce40 Compare March 19, 2026 21:03
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-recovery-2 branch from 9462d6c to eb4d7fc Compare March 20, 2026 13:21
@Maddiaa0 Maddiaa0 force-pushed the md/pipeline-timetable branch from 890ce40 to ce7da45 Compare March 20, 2026 13:21
} catch (err) {
this.log.warn(`Pre-send validation failed for ${request.action}, discarding enqueued requests`, err);
this.requests = [];
return undefined;
Copy link
Member Author

Choose a reason for hiding this comment

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

change this such that it just removes the invalidated request, NOT all requests

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.

1 participant