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

ci: Set python version to 3.11 #269

Merged
merged 1 commit into from
Dec 25, 2023
Merged

Conversation

roydahan
Copy link
Collaborator

Python 3.12 has some compatability issues with scylla-ccm. Setting the python version to 3.11 should workaround the issue for now.

@fruch
Copy link

fruch commented Dec 24, 2023

@roydahan there two more appearances of the 3.x in the file you changed

Python 3.12 has some compatability issues with scylla-ccm.
Setting the python version to 3.11 should workaround the issue for
now.
@roydahan
Copy link
Collaborator Author

This solves the CI issue with CCM of tests that didn't run at all.
There are still some tests failures, but are unrelated.

e.g.:

'''
Error: Failures:
Error: MapperUDTTest.should_throw_error_when_table_is_altered_and_is_not_compatible_anymore:563
Expecting:
"Unrecognized name mainaddress"
to match pattern:
"(Undefined name mainaddress in selection clause|Undefined column name mainaddress.*)"
[INFO]
Error: Tests run: 120, Failures: 1, Errors: 0, Skipped: 4
'''

@roydahan
Copy link
Collaborator Author

@fruch / @Bouncheck please review and let's merge this one so we will have at least a working CI.

@fruch
Copy link

fruch commented Dec 25, 2023

@fruch / @Bouncheck please review and let's merge this one so we will have at least a working CI.

CI seems to work o.k.,

5.4 - failing on some error message that seems to have change in some UDT failure.
2024.1.0~rc2 seems to be failing to start a cluster in 16 tests.

the rest of the version worked o.k.

@roydahan roydahan merged commit df622ab into scylladb:scylla-3.x Dec 25, 2023
11 of 14 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