Skip to content

testmap: add Fedora-36 to subscription-manager#3016

Merged
martinpitt merged 1 commit intocockpit-project:mainfrom
jelly:candlepin-f36
Mar 4, 2022
Merged

testmap: add Fedora-36 to subscription-manager#3016
martinpitt merged 1 commit intocockpit-project:mainfrom
jelly:candlepin-f36

Conversation

@jelly
Copy link
Copy Markdown
Member

@jelly jelly commented Mar 1, 2022

No description provided.

@jelly jelly added the no-test label Mar 1, 2022
@jelly
Copy link
Copy Markdown
Member Author

jelly commented Mar 1, 2022

+ python3 ./setup.py install --prefix /usr --root /
running install
/usr/lib/python3.10/site-packages/setuptools/command/install.py:34: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
  warnings.warn(
running build
running build_py
running build_ext
running build_trans
running build_template
running install_lib
creating /usr/local/lib64/python3.10
creating /usr/local/lib64/python3.10/site-packages
creating /usr/local/lib64/python3.10/site-packages/cloud_what
copying build/lib.linux-x86_64-3.10/cloud_what/__init__.py -> /usr/local/lib64/python3.10/site-packages/cloud_what
copying build/lib.linux-x86_64-3.10/cloud_what/_base_provider.py -> /usr/local/lib64/python3.10/site-packages/cloud_what
copying build/lib.linux-x86_64-3.10/cloud_what/fact_collector.py -> /usr/local/lib64/python3.10/site-packages/cloud_what

The issue is that it's installed into /local. setting --root to /tmp makes it install to /tmp/usr/local/lib64/python3.10/site-packages/subscription_manager-1.29.26-py3.10.egg-info. So the problem is that local is coming from somewhere.

Setuptools was updated to 59 from (57)

BTW, we could use --exec-prefix to get rid of the manual mv?

@jelly
Copy link
Copy Markdown
Member Author

jelly commented Mar 1, 2022

Conclusion, broken by pypa/distutils@c8fcf4d

@ptoscano
Copy link
Copy Markdown
Contributor

ptoscano commented Mar 1, 2022

Thanks @jelly for the report!

candlepin/subscription-manager#2989 fixes the cockpit CI for sub-man on fedora-36.

@martinpitt
Copy link
Copy Markdown
Member

Note that earlier Fedoras had a similar patch downstream, just like the upstream patch suggested. So it's not immediately clear why this changed now, except if the patches are doing slightly different things (I didn't compare them in detail)

@hroncok
Copy link
Copy Markdown

hroncok commented Mar 1, 2022

Conclusion, broken by pypa/distutils@c8fcf4d

I strongly believe this is a wrong conclusion. This is far more likely caused by https://src.fedoraproject.org/rpms/python3.10/pull-request/63 -- see also https://bugzilla.redhat.com/show_bug.cgi?id=2026979

@ptoscano
Copy link
Copy Markdown
Contributor

ptoscano commented Mar 4, 2022

subscription-manager/main was fixed, can you please check this again?

Thanks in advance!

@jelly jelly removed the blocked label Mar 4, 2022
@jelly jelly marked this pull request as ready for review March 4, 2022 12:56
@jelly jelly removed the no-test label Mar 4, 2022
@ptoscano
Copy link
Copy Markdown
Contributor

ptoscano commented Mar 4, 2022

From a subscription-manager POV, 👍 - thanks!

@jelly jelly requested a review from mvollmer March 4, 2022 13:07
@martinpitt martinpitt merged commit ce10796 into cockpit-project:main Mar 4, 2022
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.

4 participants