Skip to content

Commit

Permalink
Add more context to s3 listing error.
Browse files Browse the repository at this point in the history
  • Loading branch information
arssher committed Apr 30, 2024
1 parent 4ac4b21 commit 3a2f107
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion s3_scrubber/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -368,7 +368,10 @@ async fn list_objects_with_retries(
{
Ok(response) => return Ok(response),
Err(e) => {
error!("list_objects_v2 query failed: {e}");
error!(
"list_objects_v2 query failed: {e}, bucket_name={}, prefix={}, delimiter={}",
s3_target.bucket_name, s3_target.prefix_in_bucket, s3_target.delimiter
);
tokio::time::sleep(Duration::from_secs(1)).await;
}
}
Expand Down

1 comment on commit 3a2f107

@github-actions
Copy link

Choose a reason for hiding this comment

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

2930 tests run: 2796 passed, 0 failed, 134 skipped (full report)


Flaky tests (1)

Postgres 14

  • test_partial_evict_tenant[relative_spare]: release

Code coverage* (full report)

  • functions: 28.1% (6589 of 23418 functions)
  • lines: 46.8% (46844 of 100006 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
3a2f107 at 2024-04-30T16:43:18.205Z :recycle:

Please sign in to comment.