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

feat(torture): unify rollback and rollback crash #792

Merged
merged 1 commit into from
Feb 15, 2025

Conversation

gabriele-0201
Copy link
Contributor

@gabriele-0201 gabriele-0201 commented Feb 7, 2025

The unification of the two has the side effect of handling
ENOSPC errors also for rollbacks that are expected to crash.

Closes THR-110

Copy link
Contributor Author

gabriele-0201 commented Feb 7, 2025

self.ensure_snapshot_validity(rr).await?;
Ok(())
}
// Udpate RequestResponse, the agent could have been respawned.
Copy link
Contributor

Choose a reason for hiding this comment

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

ditto

@gabriele-0201 gabriele-0201 force-pushed the gm_torture_rollback_support_enospc branch 2 times, most recently from bc7ff0c to 262e8da Compare February 12, 2025 10:27
@gabriele-0201 gabriele-0201 force-pushed the gm_torture_unify_rollback_and_rollback_crash branch from 1885e40 to 0385977 Compare February 12, 2025 10:27
@gabriele-0201 gabriele-0201 force-pushed the gm_torture_rollback_support_enospc branch from 262e8da to 294c0a0 Compare February 13, 2025 12:14
@gabriele-0201 gabriele-0201 force-pushed the gm_torture_unify_rollback_and_rollback_crash branch from 0385977 to 0227c53 Compare February 13, 2025 12:14
@gabriele-0201 gabriele-0201 force-pushed the gm_torture_rollback_support_enospc branch from 294c0a0 to e4b2710 Compare February 14, 2025 09:07
@gabriele-0201 gabriele-0201 force-pushed the gm_torture_unify_rollback_and_rollback_crash branch from 0227c53 to 0e2635c Compare February 14, 2025 09:07
Copy link
Contributor

pepyakin commented Feb 15, 2025

Merge activity

  • Feb 15, 2:32 AM EST: A user started a stack merge that includes this pull request via Graphite.
  • Feb 15, 2:37 AM EST: Graphite rebased this pull request as part of a merge.
  • Feb 15, 2:39 AM EST: A user merged this pull request with Graphite.

@pepyakin pepyakin changed the base branch from gm_torture_rollback_support_enospc to graphite-base/792 February 15, 2025 07:34
@pepyakin pepyakin changed the base branch from graphite-base/792 to master February 15, 2025 07:36
The unification of the two has the side effect of handling
ENOSPC errors also for rollbacks that are expected to crash.
@pepyakin pepyakin force-pushed the gm_torture_unify_rollback_and_rollback_crash branch from 0e2635c to 2d0e980 Compare February 15, 2025 07:37
@pepyakin pepyakin merged commit b896c9c into master Feb 15, 2025
8 checks passed
@pepyakin pepyakin deleted the gm_torture_unify_rollback_and_rollback_crash branch February 15, 2025 07:39
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