Skip to content

Disable checkpointer (not needed in single mode)#970

Merged
tdrz merged 4 commits intomainfrom
tdrz/fe-disableCheckpointer
Apr 9, 2026
Merged

Disable checkpointer (not needed in single mode)#970
tdrz merged 4 commits intomainfrom
tdrz/fe-disableCheckpointer

Conversation

@tdrz
Copy link
Copy Markdown
Collaborator

@tdrz tdrz commented Apr 9, 2026

Disable checkpointer (not needed in single mode anyway).

See related PR in backend.

@tdrz tdrz changed the title Tdrz/fe disable checkpointer Disable checkpointer (not needed in single mode) Apr 9, 2026
Copy link
Copy Markdown
Collaborator

@samwillis samwillis left a comment

Choose a reason for hiding this comment

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

Let's keep an eye on the run time of this test. If its slow, we can pull it out into the parallel task in CI

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

@tdrz tdrz merged commit b88c5c3 into main Apr 9, 2026
23 of 24 checks passed
@tdrz tdrz deleted the tdrz/fe-disableCheckpointer branch April 9, 2026 11:22
@tdrz
Copy link
Copy Markdown
Collaborator Author

tdrz commented Apr 9, 2026

Let's keep an eye on the run time of this test. If its slow, we can pull it out into the parallel task in CI

It's actually under a flag atm, takes very long even on bare metal to run.

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