Skip to content

Conversation

@webknjaz
Copy link
Member

@webknjaz webknjaz commented Sep 3, 2025

This reverts commit d7e5b51.

SUMMARY

The upstream issue might have been fixed, according to #721 (comment).

ISSUE TYPE
  • Maintenance Pull Request

@webknjaz webknjaz self-assigned this Sep 3, 2025
Copilot AI review requested due to automatic review settings September 3, 2025 17:17
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request reverts a previous commit that excluded Python 3.12+ sdist installations from GitHub Actions workflows due to a tox bug. The revert is being made because the upstream issue may have been resolved.

  • Restores Python 3.12, 3.13, and 3.14 sdist installation jobs in CI/CD workflows
  • Removes exclusion matrix entries that were preventing these combinations from running
  • Re-enables comprehensive testing coverage across all supported Python versions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@packit-as-a-service
Copy link

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/ansible-pylibssh-764
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

Tox's self-provisioning has a bug when it is invoked with
`--installpkg` [[1]] that makes Tox hang on teardown.

The proposed workaround is making all these requirements available in
the same env Tox is installed. This patch gets rid of them instead.

[1]: tox-dev/tox#3600
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Sep 4, 2025
@webknjaz webknjaz enabled auto-merge September 4, 2025 15:51
@webknjaz webknjaz disabled auto-merge September 4, 2025 17:03
@webknjaz webknjaz merged commit 5b4f425 into ansible:devel Sep 4, 2025
58 of 62 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:chronographer:provided There is a change note present in this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant