Skip to content

Commit 90b962c

Browse files
committed
qa/rgw: Include rgw_restore_processor_period in s3tests
Signed-off-by: Soumya Koduri <[email protected]>
1 parent 71882f6 commit 90b962c

File tree

4 files changed

+9
-2
lines changed

4 files changed

+9
-2
lines changed

qa/suites/rgw/cloud-transition/overrides.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ overrides:
1212
rgw s3 auth use sts: true
1313
rgw lc debug interval: 10
1414
rgw_restore_debug_interval: 20
15+
rgw_restore_processor_period: 10
1516
rgw:
1617
storage classes:
1718
LUKEWARM:

qa/suites/rgw/cloud-transition/tasks/restore/cloud_restore_s3tests.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,5 +27,6 @@ tasks:
2727
extra_attrs: ["cloud_restore"]
2828
lc_debug_interval: 10
2929
rgw_restore_debug_interval: 20
30+
rgw_restore_processor_period: 10
3031
lifecycle_tests: True
3132
cloudtier_tests: True

qa/tasks/s3tests.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -372,6 +372,10 @@ def configure(ctx, config):
372372
if rgw_restore_debug_interval:
373373
s3tests_conf['s3 main']['rgw_restore_debug_interval'] = rgw_restore_debug_interval
374374

375+
rgw_restore_processor_period = properties.get('rgw_restore_processor_period')
376+
if rgw_restore_processor_period:
377+
s3tests_conf['s3 main']['rgw_restore_processor_period'] = rgw_restore_processor_period
378+
375379
if ctx.rgw_cloudtier is not None:
376380
log.info(' ctx.rgw_cloudtier config is %s ...', ctx.rgw_cloudtier.config)
377381
client_rgw_config = ctx.rgw_cloudtier.config.get(client)

src/rgw/rgw_restore.cc

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -341,13 +341,14 @@ int Restore::process(int index, int max_secs, optional_yield y)
341341
ret = process_restore_entry(entry, y);
342342

343343
if (!ret && entry.status == rgw::sal::RGWRestoreStatus::RestoreAlreadyInProgress) {
344-
r_entries.push_back(entry);
344+
r_entries.push_back(entry);
345345
ldpp_dout(this, 20) << __PRETTY_FUNCTION__ << ": re-pushing entry: '" << entry
346346
<< "' on shard:"
347347
<< obj_names[index] << dendl;
348348
}
349349

350-
if (ret < 0)
350+
// Skip the entry of object/bucket which no longer exists
351+
if (ret < 0 && (ret != -ENOENT))
351352
goto done;
352353

353354
///process all entries, trim and re-add

0 commit comments

Comments
 (0)