From fb3186266ee52b352ebdf4ce394d3144a23db8f2 Mon Sep 17 00:00:00 2001 From: Shilpa Jagannath Date: Mon, 15 Apr 2024 10:54:57 -0400 Subject: [PATCH] qa/multisite: disable two-zonegroup and three-zone configs. Signed-off-by: Shilpa Jagannath --- ...e-zones.yaml => three-zones.yaml.disabled} | 0 ...group.yaml => two-zonegroup.yaml.disabled} | 0 qa/suites/rgw/multisite/realms/two-zones.yaml | 4 +- src/test/rgw/rgw_multi/tests.py | 37 +++++++++++++++++-- 4 files changed, 35 insertions(+), 6 deletions(-) rename qa/suites/rgw/multisite/realms/{three-zones.yaml => three-zones.yaml.disabled} (100%) rename qa/suites/rgw/multisite/realms/{two-zonegroup.yaml => two-zonegroup.yaml.disabled} (100%) diff --git a/qa/suites/rgw/multisite/realms/three-zones.yaml b/qa/suites/rgw/multisite/realms/three-zones.yaml.disabled similarity index 100% rename from qa/suites/rgw/multisite/realms/three-zones.yaml rename to qa/suites/rgw/multisite/realms/three-zones.yaml.disabled diff --git a/qa/suites/rgw/multisite/realms/two-zonegroup.yaml b/qa/suites/rgw/multisite/realms/two-zonegroup.yaml.disabled similarity index 100% rename from qa/suites/rgw/multisite/realms/two-zonegroup.yaml rename to qa/suites/rgw/multisite/realms/two-zonegroup.yaml.disabled diff --git a/qa/suites/rgw/multisite/realms/two-zones.yaml b/qa/suites/rgw/multisite/realms/two-zones.yaml index 3564c9619b7cb..1bea381077c00 100644 --- a/qa/suites/rgw/multisite/realms/two-zones.yaml +++ b/qa/suites/rgw/multisite/realms/two-zones.yaml @@ -8,7 +8,7 @@ overrides: is_master: true is_default: true endpoints: [c1.client.0] - enabled_features: ['resharding'] + enabled_features: ['resharding', 'notification_v2'] zones: - name: test-zone1 is_master: true @@ -18,4 +18,4 @@ overrides: is_default: true endpoints: [c2.client.0] rgw-multisite-tests: - args: [tests.py] + args: [tests.py, -a, '!fails_with_rgw'] diff --git a/src/test/rgw/rgw_multi/tests.py b/src/test/rgw/rgw_multi/tests.py index 33c885d239bfb..685e0bcb2febd 100644 --- a/src/test/rgw/rgw_multi/tests.py +++ b/src/test/rgw/rgw_multi/tests.py @@ -1919,9 +1919,10 @@ def test_account_metadata_sync(): check_oidc_providers_eq(source_conn, target_conn) -# TODO: move sync policy and endpoint tests into separate test file -# fix data sync init tests -''' +# TODO: disable failing tests temporarily +# until they are fixed + +@attr('fails_with_rgw') def test_bucket_reshard_index_log_trim(): zonegroup = realm.master_zonegroup() zonegroup_conns = ZonegroupConns(zonegroup) @@ -2006,6 +2007,7 @@ def test_bucket_reshard_index_log_trim(): test_bilog = bilog_list(zone.zone, test_bucket.name) assert(len(test_bilog) > 0) +@attr('fails_with_rgw') @attr('bucket_reshard') def test_bucket_sync_run_basic_incremental(): """ @@ -2071,6 +2073,7 @@ def trash_bucket(zone, bucket_name): cmd += ['--bucket', bucket_name] zone.cluster.admin(cmd) +@attr('fails_with_rgw') @attr('bucket_reshard') def test_zap_init_bucket_sync_run(): """ @@ -2223,6 +2226,7 @@ def test_bucket_full_sync_after_data_sync_init(): zonegroup_bucket_checkpoint(zonegroup_conns, bucket.name) zonegroup_data_checkpoint(zonegroup_conns) +@attr('fails_with_rgw') @attr('data_sync_init') @attr('bucket_reshard') def test_resharded_bucket_full_sync_after_data_sync_init(): @@ -2263,6 +2267,7 @@ def test_resharded_bucket_full_sync_after_data_sync_init(): zonegroup_bucket_checkpoint(zonegroup_conns, bucket.name) zonegroup_data_checkpoint(zonegroup_conns) +@attr('fails_with_rgw') @attr('data_sync_init') def test_bucket_incremental_sync_after_data_sync_init(): zonegroup = realm.master_zonegroup() @@ -2300,6 +2305,7 @@ def test_bucket_incremental_sync_after_data_sync_init(): zonegroup_bucket_checkpoint(zonegroup_conns, bucket.name) zonegroup_data_checkpoint(zonegroup_conns) +@attr('fails_with_rgw') @attr('data_sync_init') @attr('bucket_reshard') def test_resharded_bucket_incremental_sync_latest_after_data_sync_init(): @@ -2348,6 +2354,7 @@ def test_resharded_bucket_incremental_sync_latest_after_data_sync_init(): zonegroup_bucket_checkpoint(zonegroup_conns, bucket.name) zonegroup_data_checkpoint(zonegroup_conns) +@attr('fails_with_rgw') @attr('data_sync_init') @attr('bucket_reshard') def test_resharded_bucket_incremental_sync_oldest_after_data_sync_init(): @@ -2509,6 +2516,7 @@ def remove_sync_group_pipe(cluster, group, pipe_id, bucket = None, args = None): assert False, 'failed to remove sync group pipe groupid=%s, pipe_id=%s, src_zones=%s, dest_zones=%s, bucket=%s' % (group, pipe_id, src_zones, dest_zones, bucket) return json.loads(result_json) +@attr('fails_with_rgw') @attr('sync_policy') def test_sync_policy_config_zonegroup(): """ @@ -2580,6 +2588,7 @@ def test_sync_policy_config_zonegroup(): return +@attr('fails_with_rgw') @attr('sync_policy') def test_sync_flow_symmetrical_zonegroup_all(): """ @@ -2637,6 +2646,7 @@ def test_sync_flow_symmetrical_zonegroup_all(): remove_sync_policy_group(c1, "sync-group") return +@attr('fails_with_rgw') @attr('sync_policy') def test_sync_flow_symmetrical_zonegroup_select(): """ @@ -2705,6 +2715,7 @@ def test_sync_flow_symmetrical_zonegroup_select(): remove_sync_policy_group(c1, "sync-group") return +@attr('fails_with_rgw') @attr('sync_policy') def test_sync_flow_directional_zonegroup_select(): """ @@ -2822,6 +2833,7 @@ def test_sync_flow_directional_zonegroup_select(): remove_sync_policy_group(c1, "sync-group") return +@attr('fails_with_rgw') @attr('sync_policy') def test_sync_single_bucket(): """ @@ -2934,6 +2946,7 @@ def test_sync_single_bucket(): remove_sync_policy_group(c1, "sync-group") return +@attr('fails_with_rgw') @attr('sync_policy') def test_sync_different_buckets(): """ @@ -3083,6 +3096,7 @@ def test_sync_different_buckets(): remove_sync_policy_group(c1, "sync-group") return +@attr('fails_with_rgw') @attr('sync_policy') def test_sync_multiple_buckets_to_single(): """ @@ -3204,6 +3218,7 @@ def test_sync_multiple_buckets_to_single(): remove_sync_policy_group(c1, "sync-group") return +@attr('fails_with_rgw') @attr('sync_policy') def test_sync_single_bucket_to_multiple(): """ @@ -3332,6 +3347,7 @@ def start_2nd_rgw(zonegroup): z.gateways[1].start() log.info('gateway started zone=%s gateway=%s', z.name, z.gateways[1].endpoint()) +@attr('fails_with_rgw') @attr('rgw_down') def test_bucket_create_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3349,6 +3365,7 @@ def test_bucket_create_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_bucket_remove_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3374,6 +3391,7 @@ def test_bucket_remove_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_object_sync_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3385,6 +3403,7 @@ def test_object_sync_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_object_delete_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3396,6 +3415,7 @@ def test_object_delete_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_concurrent_versioned_object_incremental_sync_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3407,6 +3427,7 @@ def test_concurrent_versioned_object_incremental_sync_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_suspended_delete_marker_full_sync_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3418,6 +3439,7 @@ def test_suspended_delete_marker_full_sync_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_bucket_acl_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3429,6 +3451,7 @@ def test_bucket_acl_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_bucket_sync_enable_right_after_disable_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3440,6 +3463,7 @@ def test_bucket_sync_enable_right_after_disable_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_multipart_object_sync_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3451,6 +3475,7 @@ def test_multipart_object_sync_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_bucket_sync_run_basic_incremental_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3462,6 +3487,7 @@ def test_bucket_sync_run_basic_incremental_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_role_sync_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3473,6 +3499,7 @@ def test_role_sync_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_bucket_full_sync_after_data_sync_init_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3484,6 +3511,7 @@ def test_bucket_full_sync_after_data_sync_init_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_sync_policy_config_zonegroup_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3495,6 +3523,7 @@ def test_sync_policy_config_zonegroup_rgw_down(): finally: start_2nd_rgw(zonegroup) +@attr('fails_with_rgw') @attr('rgw_down') def test_sync_flow_symmetrical_zonegroup_all_rgw_down(): zonegroup = realm.master_zonegroup() @@ -3505,5 +3534,5 @@ def test_sync_flow_symmetrical_zonegroup_all_rgw_down(): test_sync_flow_symmetrical_zonegroup_all() finally: start_2nd_rgw(zonegroup) -''' + -- 2.39.5