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

Install Python 3.11.9 on base-admin-tools so cqlsh can run #233

Merged
merged 3 commits into from
Jul 17, 2024

Conversation

ShahabT
Copy link
Contributor

@ShahabT ShahabT commented Jul 17, 2024

What was changed

Alpine v3.20 comes with Python 3.12. But cqlsh won't work with Python 3.12 until the following issue is resolved.
https://issues.apache.org/jira/browse/CASSANDRA-19206
Compiling Python 3.11.9 from source and installing it on the side. Then creating a venv using that python version.
Revert this change once the issue mentioned above is resolved.

Why?

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

@ShahabT ShahabT requested review from a team as code owners July 17, 2024 19:43
@ShahabT ShahabT changed the title Shahab/python311 Install Python 3.11.9 on base-admin-tools so cqlsh can run Jul 17, 2024
Comment on lines +25 to +27
make clean ; \
cd .. ; \
rm -rf Python-3.11.9*
Copy link
Member

Choose a reason for hiding this comment

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

You could technically skip the cleanup since this is all in the build container but not critical either way.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, added this just because I'm copying the parent folder to the other image.

@ShahabT ShahabT merged commit 5568ddc into main Jul 17, 2024
11 checks passed
@ShahabT ShahabT deleted the shahab/python311 branch July 17, 2024 20:35
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.

3 participants