Skip to content

Commit

Permalink
Fix test failure (pretty nice that this bug was caught by the tests)
Browse files Browse the repository at this point in the history
  • Loading branch information
arpad-m committed Dec 9, 2024
1 parent 1083f18 commit ba19374
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion storage_scrubber/src/checks.rs
Original file line number Diff line number Diff line change
Expand Up @@ -621,6 +621,8 @@ pub(crate) async fn list_tenant_manifests(
.map(|(g, obj)| (*g, obj.clone()))
.unwrap();

manifests.retain(|(gen, _obj)| gen != &latest_generation);

let manifest_bytes =
match download_object_with_retries(remote_client, &latest_listing_object.key).await {
Ok(bytes) => bytes,
Expand All @@ -646,7 +648,7 @@ pub(crate) async fn list_tenant_manifests(
manifest,
listing_object: latest_listing_object,
}),
manifests: vec![],
manifests,
});
}
Err(parse_error) => errors.push((
Expand Down

0 comments on commit ba19374

Please sign in to comment.