-
Notifications
You must be signed in to change notification settings - Fork 98
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
fix(manager): update default Scylla version to 2024.2 for Manager tests #9622
fix(manager): update default Scylla version to 2024.2 for Manager tests #9622
Conversation
Default scylla_version is set to 2024.2 instead of 2024.1. At the same time, to have some coverage for 2024.1 as well, all Debian11 jobs adjusted to use version 2024.1. Related test_get_manager_scylla_backend_returns_repo_address test is adjusted too.
42f83d2
to
4bd3f86
Compare
@@ -9,6 +9,8 @@ managerPipeline( | |||
test_name: 'mgmt_cli_test.ManagerInstallationTests.test_manager_installed_and_functional', | |||
test_config: '''["test-cases/manager/manager-installation-set-distro.yaml", "configurations/manager/debian11.yaml"]''', | |||
|
|||
scylla_version: '2024.1', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't be 2024.2?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I put 2024.1
there by purpose for debian jobs to keep some coverage for 2024.1
too since it's still used in Production.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Default scylla_version is set to 2024.2 instead of 2024.1.
At the same time, to have some coverage for 2024.1 as well, all Debian 11 jobs adjusted to use version 2024.1.
Testing
PR pre-checks (self review)
backport
labels