Skip to content

Commit

Permalink
fix(rolling-upgrade): ignore RPC unknown exception
Browse files Browse the repository at this point in the history
ignoring those unsuppressed exceptions,
till both ends of the upgrade would have scylladb/scylladb#14681

Ref: scylladb/scylladb#14681
Ref: scylladb/scylladb#14882
  • Loading branch information
fruch committed Nov 8, 2023
1 parent 370c065 commit ea13014
Showing 1 changed file with 20 additions and 2 deletions.
22 changes: 20 additions & 2 deletions upgrade_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
import re
from functools import wraps, cache
from typing import List
import contextlib

from argus.client.sct.types import Package

Expand All @@ -33,13 +34,16 @@
from sdcm.stress_thread import CassandraStressThread
from sdcm.utils.version_utils import get_node_supported_sstable_versions
from sdcm.sct_events.system import InfoEvent
from sdcm.sct_events.database import IndexSpecialColumnErrorEvent
from sdcm.sct_events.database import (
IndexSpecialColumnErrorEvent,
DatabaseLogEvent,
)
from sdcm.sct_events.filters import EventsSeverityChangerFilter
from sdcm.sct_events.group_common_events import ignore_upgrade_schema_errors, ignore_ycsb_connection_refused, \
ignore_abort_requested_errors, decorate_with_context
from sdcm.utils import loader_utils
from test_lib.sla import create_sla_auth


NUMBER_OF_ROWS_FOR_TRUNCATE_TEST = 10


Expand Down Expand Up @@ -109,6 +113,20 @@ class UpgradeTest(FillDatabaseData, loader_utils.LoaderUtilsMixin):
"""
Test a Scylla cluster upgrade.
"""

def __init__(self, *args):
super().__init__(*args)

self.stack = contextlib.ExitStack()
# ignoring those unsuppressed exceptions, till both ends of the upgrade would have https://github.com/scylladb/scylladb/pull/14681
# see https://github.com/scylladb/scylladb/issues/14882 for details
self.stack.enter_context(EventsSeverityChangerFilter(
new_severity=Severity.WARNING,
event_class=DatabaseLogEvent,
regex=r".*std::runtime_error \(unknown exception\).*",
extra_time_to_expiration=30,
))

orig_ver = None
new_ver = None

Expand Down

0 comments on commit ea13014

Please sign in to comment.