Skip to content

Commit 84681cc

Browse files
committedNov 18, 2024
failpoints: Add context to tool error message
We have a separate binary for setting failpoints, which works by connecting to a Readyset controller instance. If it fails, we now print the status of what went wrong. Also include instructions in the message telling the user how to compile Readyset with support for failpoints. Change-Id: Ibf5d9c7c689bae134e2b3a2f5b24fb051ab9db8a Reviewed-on: https://gerrit.readyset.name/c/readyset/+/8326 Reviewed-by: Jason Brown <[email protected]> Tested-by: Buildkite CI
1 parent da262e0 commit 84681cc

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed
 

‎readyset-tools/src/failpoint.rs

+5-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,11 @@ impl Failpoint {
4949
let client = Client::new();
5050
let res = client.request(r).await.unwrap();
5151
let status = res.status();
52-
assert!(status == hyper::StatusCode::OK);
52+
assert!(
53+
status == hyper::StatusCode::OK,
54+
"status was: {} (don't forget to compile Readyset with --features failure_injection)",
55+
status
56+
);
5357
Ok(())
5458
}
5559
}

0 commit comments

Comments
 (0)
Please sign in to comment.