ManageIQ / manageiq-appliance-build

Scripts to build ManageIQ appliances
Apache License 2.0
10 stars 55 forks source link

Ensure psutil is installed for all pythons #528

Closed Fryguy closed 1 year ago

Fryguy commented 1 year ago

@agrare Please review.

miq-bot commented 1 year ago

Checked commit https://github.com/Fryguy/manageiq-appliance-build/commit/3a3dd4a7dea30ebc73e8440df6e5e7b1c49b7295 with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint 0 files checked, 0 offenses detected Everything looks fine. :+1:

Fryguy commented 1 year ago

Not my finest work, but it works 🤷 .

We're gonna have to do this very different with a proper rpm if we want upgrades to work properly, but this will get us over the hump.

agrare commented 1 year ago

@Fryguy so removing the weak dependencies didn't remove python39?

Fryguy commented 1 year ago

No. Turns out 3.9 is a dep on that version of ansible. Still not sure why appliances are a different version of ansible than podified

agrare commented 1 year ago

Interesting, and ansible/ansible-core are slightly different versions on podified and appliances right?

Fryguy commented 1 year ago

Interesting, and ansible/ansible-core are slightly different versions on podified and appliances right?

yeah

Fryguy commented 1 year ago

on appliance, ansible is coming from @epel-next, ansible-core is coming from @appstream

epel-next,Extra Packages for Enterprise Linux 8 - Next - x86_64,/etc/yum.repos.d/epel-next.repo,rsync://ftp-osl.osuosl.org/fedora-epel/next/8/Everything/x86_64/ (143 more),,https://mirrors.fedoraproject.org/metalink?repo=epel-next-8&arch=x86_64&infra=stock&content=centos

but on podified, ansible and ansible-core are coming from @System and @epel-next doesn't exist.