Skip to content

Commit

Permalink
Merge pull request #11435 from VladLazar/fix-test-produce-consume-cloud
Browse files Browse the repository at this point in the history
cloud_storage: tweak test to use default archiver
  • Loading branch information
piyushredpanda authored Jun 15, 2023
2 parents f92380f + 2c46fd8 commit a40e9b9
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions src/v/cloud_storage/tests/cloud_storage_e2e_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -79,10 +79,7 @@ FIXTURE_TEST(test_produce_consume_from_cloud, e2e_fixture) {

// Upload the closed segment to object storage.
tests::cooperative_spin_wait_with_timeout(3s, [&archiver] {
return archiver.upload_next_candidates().then(
[](archival::ntp_archiver::batch_result res) {
return res.non_compacted_upload_result.num_succeeded == 1;
});
return archiver.manifest().size() == 1;
}).get();
auto manifest_res = archiver.upload_manifest("test").get();
BOOST_REQUIRE_EQUAL(manifest_res, cloud_storage::upload_result::success);
Expand Down

0 comments on commit a40e9b9

Please sign in to comment.