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

Simpler cqlsh installation in base-admin-tools image #262

Merged
merged 1 commit into from
Feb 14, 2025

Conversation

rodrigozhou
Copy link
Collaborator

@rodrigozhou rodrigozhou commented Feb 13, 2025

What was changed

Install cqlsh using pipx and dependencies to be build from source works with Python 3.12.

Why?

cqlsh with Python3.12 doesn't work out-of-box because asyncore module was removed from Python3.12, and the default built of cqlsh doesn't include the alternative libev.

However, it works if we can install it by building from source with libev (https://datastax-oss.atlassian.net/browse/PYTHON-1376).

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

@rodrigozhou rodrigozhou requested a review from a team as a code owner February 13, 2025 21:49
@rodrigozhou rodrigozhou force-pushed the rodrigozhou/update-base-admin-tools branch from 76d2da5 to 92a6b19 Compare February 13, 2025 21:51
@rodrigozhou rodrigozhou force-pushed the rodrigozhou/update-base-admin-tools branch from 92a6b19 to a3c4ee4 Compare February 14, 2025 00:24
Copy link
Member

@alexshtin alexshtin left a comment

Choose a reason for hiding this comment

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

I want to be the last person who can review python dependency hell :-). But this looks a way nicer. I hope it works too.

@rodrigozhou rodrigozhou merged commit fc07f33 into main Feb 14, 2025
12 checks passed
@rodrigozhou rodrigozhou deleted the rodrigozhou/update-base-admin-tools branch February 14, 2025 00:59
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