Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(k8s): skip flush and reshard test due to operator's bug #5423

Merged
merged 2 commits into from
Nov 16, 2022

Conversation

vponomaryov
Copy link
Contributor

@vponomaryov vponomaryov commented Nov 15, 2022

The 'test_nodetool_flush_and_reshard' test fails due to the following bug:
scylladb/scylla-operator#1077
So, skip it.

PR pre-checks (self review)

  • I followed KISS principle and best practices
  • I didn't leave commented-out/debugging code
  • I added the relevant backport labels
  • New configuration option are added and documented (in sdcm/sct_config.py)
  • I have added tests to cover my changes (Infrastructure only - under unit-test/ folder)
  • All new and existing unit tests passed (CI)
  • I have updated the Readme/doc folder accordingly (if needed)

@vponomaryov
Copy link
Contributor Author

The appropriate nemesis is already skipped due to this bug: #5375

Example of the usage:

    @pytest.mark.requires_backend("k8s-local-kind")
    def test_foo(db_cluster):
        ...

    @pytest.mark.requires_backend("k8s-eks", "k8s-gke")
    def test_bar(db_cluster):
        ...
The 'test_nodetool_flush_and_reshard' test is affected by the following
bug:

    scylladb/scylla-operator#1077

using non-fast K8S backends such as 'k8s-gke' and 'k8s-local-kind'.
So, skip it on the slow backends.
Copy link
Contributor

@fruch fruch left a comment

Choose a reason for hiding this comment

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

LGTM

@fruch fruch merged commit 47e9b25 into scylladb:master Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/none Backport is not required Ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants