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

py3-parallel-ssh: remove package & deps (wolfi-dev#23756) #26270

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

justinvreeland
Copy link
Member

In b7de8cb py3-ssh-python py3-ssh2-python and py3-parallel-ssh were imported for use in py3-cassandra-medusa.

Sice then py3-cassandra-medusa switched to a venv
49c9723. These packages are not used or useable in their current state and upstream is not in a position to be releasing python3.12 compatible versions right now.

Fixes:

Related:

Pre-review Checklist

There's no Pre-review checklist for removing things but I see no packages or images that specify they need these contents and the original need for them is gone.

@mritunjaysharma394 can you confirm that we don't need these anymore?

In b7de8cb py3-ssh-python
py3-ssh2-python and py3-parallel-ssh were imported for use in
py3-cassandra-medusa.

Sice then py3-cassandra-medusa switched to a venv
49c9723. These packages are not
used or useable in their current state and upstream is not in a
position to be releasing python3.12 compatible versions right now.
@mritunjaysharma394
Copy link
Contributor

Thanks @justinvreeland , i checked and yes since we switched to venv method, I don't think we need them anymore, I also checked if any images uses them and seems like none use it directly so we should be safe to remove it

Copy link
Member

@smoser smoser left a comment

Choose a reason for hiding this comment

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

thanks.

@smoser smoser merged commit a5e4968 into wolfi-dev:main Aug 15, 2024
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.

3 participants