Skip to content

Remove oss/ent_scylla_version params from xfail_scylla_version_lt#868

Merged
sylwiaszunejko merged 1 commit intoscylladb:masterfrom
sylwiaszunejko:xfail-remove-enterprise
May 6, 2026
Merged

Remove oss/ent_scylla_version params from xfail_scylla_version_lt#868
sylwiaszunejko merged 1 commit intoscylladb:masterfrom
sylwiaszunejko:xfail-remove-enterprise

Conversation

@sylwiaszunejko
Copy link
Copy Markdown
Collaborator

xfail_scylla_version_lt now takes a single scylla_version parameter instead of separate oss_scylla_version and ent_scylla_version params. The enterprise/OSS version branching logic is removed; the decorator simply compares the current version against the single provided version.

Update all call sites accordingly.

Pre-review checklist

  • I have split my patch into logically separate commits.
  • All commit messages clearly explain what they change and why.
  • I added relevant tests for new features and bug fixes.
  • All commits compile, pass static checks and pass test.
  • PR description sums up the changes and reasons why they should be introduced.
  • I have provided docstrings for the public items that I want to introduce.
  • I have adjusted the documentation in ./docs/source/.
  • I added appropriate Fixes: annotations to PR description.

xfail_scylla_version_lt now takes a single scylla_version parameter
instead of separate oss_scylla_version and ent_scylla_version params.
The enterprise/OSS version branching logic is removed; the decorator
simply compares the current version against the single provided version.

Update all call sites accordingly.
@sylwiaszunejko sylwiaszunejko requested a review from Lorak-mmk May 6, 2026 06:42
@sylwiaszunejko sylwiaszunejko self-assigned this May 6, 2026
@sylwiaszunejko sylwiaszunejko merged commit e6f9e9f into scylladb:master May 6, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants