Skip to content

Conversation

@Jakuje
Copy link
Contributor

@Jakuje Jakuje commented Aug 19, 2025

SUMMARY

ci: python3-pytest was moved to AppStream in CentOS 9

Built on top of #757, which fixes another breakage.

ISSUE TYPE
  • Bugfix Pull Request
ADDITIONAL INFORMATION
curl: (22) The requested URL returned error: 404
error: skipping https://rpmfind.net/linux/centos-stream/9-stream/CRB/x86_64/os/Packages/python3-pytest-6.2.2-4.el9.noarch.rpm - transfer failed

@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-758
  • And now you can install the packages.

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

@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Aug 19, 2025
@webknjaz
Copy link
Member

@Jakuje let's scope this to just UBI fixes, I'll merge the setuptools-scm bump separately.

@Jakuje
Copy link
Contributor Author

Jakuje commented Aug 19, 2025

@Jakuje let's scope this to just UBI fixes, I'll merge the setuptools-scm bump separately.

without the setuptools-scm update it explodes much easier than it gets to the ubi failures.

But even after fixing the pytest, there is something bringing in the toml, which I do not understand where it comes from and how to fix so not sure if it is something I can help much more from now on ...

@webknjaz
Copy link
Member

I force-merged #757 and clicking "rebase" in this one.

@webknjaz
Copy link
Member

But even after fixing the pytest, there is something bringing in the toml, which I do not understand where it comes from and how to fix so not sure if it is something I can help much more from now on ...

Okay, I think I'll force-merge this one too, once the CI completes. And figure that out separately.

The exception is caused by https://github.com/pypa/build/blob/0.3.1.post1/src/build/__init__.py#L19. It's an ancient version of pypa/build that's installed manually: https://github.com/ansible/pylibssh/blob/6dcfbb6/packaging/rpm/ansible-pylibssh.spec#L42.

We should probably try looking into the possibility to run newer versions or check if the RPM repos already have it (this was a hack for when it wasn't available in RHEL).

My guess is that maybe the toml project was somehow available in the images in the past through some other dep we don't manage and not it's not. Newever versions of build don't use it: they rely on tomllib from stdlib on Python 3.11+ with a fallback to the API-compatible tomli project.

@webknjaz webknjaz merged commit 2959959 into ansible:devel Aug 20, 2025
41 of 52 checks passed
@webknjaz
Copy link
Member

UPD: simplifying the RPM spec works locally, I'll send a PR once I polish its overall shape.

@webknjaz
Copy link
Member

#759

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.

2 participants