Bug 2142064

Summary:	elements fails to build with pytest 7.2.0 - problem with dependency on `py` package
Product:	[Fedora] Fedora	Reporter:	Lumír Balhar <lbalhar>
Component:	elements	Assignee:	Alejandro Alvarez <a.alvarezayllon>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	a.alvarezayllon
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2022-11-23 19:20:15 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	2137514

Description Lumír Balhar 2022-11-11 13:34:45 UTC

I'm testing an update of pytest to 7.2.0 and I've discovered that this package fails to build with it. The reason is a change related to `py` package.

`py` is an independent package developed by the same team as pytest and it has been bundled in pytest for quite some time. The `py` package itself is now in maintenance mode and pytest developers decided to bundle only a subset of its functionalities in pytest since version 7.2.0.

It seems that elements needs some parts of `py` that are no longer available in pytest (including deprecated `py.test` itself). There are two possible solutions:

1. Drop dependency on `py`. That requires some coordination with upstream but it's generally a good idea long-term.
2. Add python3-py to build dependencies. `py` package installed from python3-py will have a higher priority than py.py module bundled in pytest. This is a quick workaround but given the status of `py` project, it's only temporary.

If you want to, you can find the failed build in https://copr.fedorainfracloud.org/coprs/lbalhar/pytest/

Comment 1 Lumír Balhar 2022-11-14 11:14:22 UTC

The issue is already fixed upstream: https://github.com/degauden/Elements/commit/f339c678fbcadbcc011e099a620725d47dc105ff

Comment 2 Alejandro Alvarez 2022-11-14 21:27:27 UTC

(In reply to Lumír Balhar from comment #1)
> The issue is already fixed upstream:
> https://github.com/degauden/Elements/commit/
> f339c678fbcadbcc011e099a620725d47dc105ff

Thanks for the tip!

Comment 3 Lumír Balhar 2022-11-23 12:20:40 UTC

Our plan is to ship the updated pytest during the next week. If you need any assistance from us, feel free to say so.

Comment 4 Alejandro Alvarez 2022-11-23 19:20:15 UTC

Sorry, I didn't mark this as closed, but I had pushed the changes to rawhide.