Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove failpoints from the executor #15721

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

msmouse
Copy link
Contributor

@msmouse msmouse commented Jan 13, 2025

Description

it's no longer assumed that consensus can recover from an error in the executor

How Has This Been Tested?

forge

Key Areas to Review

Type of Change

  • Tests

Which Components or Systems Does This Change Impact?

  • Developer Infrastructure

it's no longer assumed that consensus can recover from an error in the
executor
Copy link

trunk-io bot commented Jan 13, 2025

⏱️ 17m total CI duration on this PR
Job Cumulative Duration Recent Runs
determine-test-metadata 6m 🟩🟥
test-target-determinator 5m 🟩
check-dynamic-deps 2m 🟩🟩
rust-cargo-deny 2m 🟩
semgrep/ci 47s 🟩🟩
general-lints 27s 🟩
file_change_determinator 13s 🟩
file_change_determinator 12s 🟩
generate-matrix 7s 🟩🟩🟩
permission-check 6s 🟩🟩
permission-check 4s 🟩🟩
permission-check 3s 🟩
determine-docker-build-metadata 2s 🟩

settingsfeedbackdocs ⋅ learn more about trunk.io

@msmouse msmouse added the CICD:build-images when this label is present github actions will start build+push rust images from the PR. label Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CICD:build-images when this label is present github actions will start build+push rust images from the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants