Increase Garage tests robustness #526
No reviewers
Labels
No Label
AdminAPI
Bug
Check AWS
CI
Correctness
Critical
Documentation
Ideas
Improvement
Low priority
Newcomer
Performance
S3 Compatibility
Testing
Usability
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Deuxfleurs/garage#526
Loading…
Reference in New Issue
No description provided.
Delete Branch "tests/increase-robustness"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
We are noticing tests failing in a non deterministic way, for 2 reasons:
This PR address these issues as follow:
This PR is not ready for review
Fix #528
WIP: Increase Garage tests robustnessto Increase Garage tests robustnessfa7248c0a1
to70b5424b99
LGTM, maybe we can get a review by @trinity-1686a or @KokaKiwi as they contributed to the testing code? Otherwise we can merge as is.
lgtm
@ -22,3 +22,3 @@
process: process::Child,
pub path: PathBuf,
pub key: Key,
pub default_key: Key,
is this read anywhere?
@ -170,3 +183,3 @@
}
pub fn new_key(&self, name: &str) -> Key {
pub fn key(&self, maybe_name: Option<&str>) -> Key {
I'm not fond of that name, it sounds too much like a getter imo (not blocking)