Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed missing software inventory on RedHat based systems #2908

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

craigcomstock
Copy link
Contributor

A previous fix to add package inventory for AIX and adding ability to influence default package manager and inventory via augments introduced this bug.

The commit was: 8729217

There was a typo that used the class cfe_yum_package_module_supported instead of cfe_python_for_package_modules_supported.

The first class is undefined anywhere in policy. The second class it the correct one.

Ticket: ENT-11850
Changelog: none

A previous fix to add package inventory for AIX and adding ability to influence default package manager and inventory via augments introduced this bug.

The commit was: 8729217

There was a typo that used the class cfe_yum_package_module_supported instead of cfe_python_for_package_modules_supported.

The first class is undefined anywhere in policy. The second class it the correct one.

Ticket: ENT-11850
Changelog: none
Copy link
Contributor

@vpodzime vpodzime left a comment

Choose a reason for hiding this comment

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

ACK

@craigcomstock
Copy link
Contributor Author

craigcomstock commented Jun 6, 2024

acceptance_tests relies on nightlies and apparently ubuntu 22 testing-pr failed to produce a package.

$ cf-remote --version master list ubuntu
http://buildcache.cfengine.com/packages/testing-pr/jenkins-master-nightly-pipeline-1568/PACKAGES_HUB_x86_64_linux_ubuntu_20/cfengine-nova-hub_3.24.0a.fc037ae3b~33563.ubuntu20_amd64.deb

@craigcomstock craigcomstock merged commit ff42e72 into cfengine:master Jun 6, 2024
3 of 4 checks passed
@craigcomstock craigcomstock deleted the ENT-11850/master branch June 6, 2024 13:16
@craigcomstock
Copy link
Contributor Author

cherry picks
#2910
#2909

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants