Bug #12436
PE 2.0 facter produces misleading facts
| Status: | Merged - Pending Release | Start date: | 02/04/2012 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Matthaus Litteken |
% Done: | 0% |
|
| Category: | packages | |||
| Target version: | Commercial edition - PE 2.0.3 | |||
| Keywords: | Affected URL: | |||
| Branch: | Affected PE version: | |||
| Votes: | 0 |
Description
# /opt/puppet/bin/facter virtual physical # yum intall -y pciutils dmidecode # /opt/puppet/bin/facter virtual vmware # rpm -qf /opt/puppet/bin/facter pe-facter-1.6.4-1.pe.el6.noarch
puppet-enterprise-installer should suggest that these two packages be installed so that facter behaves as expected. (Alt: pe-facter can Require: these two packages)
History
Updated by James Turnbull 4 months ago
- Project changed from Facter to Puppet Enterprise (Public)
- Target version deleted (
14)
Updated by James Turnbull 4 months ago
- Description updated (diff)
- Category set to packages
- Status changed from Unreviewed to Needs Decision
- Assignee set to Matthaus Litteken
Updated by Michael Stahnke 4 months ago
- Status changed from Needs Decision to Accepted
That’s a valid bug. We fixed that in FOSS and missed it in PE.
We need to add deps on dmidecode, and pciutils for Linux.
Updated by Matthaus Litteken 3 months ago
- Target version set to PE 2.0.3
Updated by Matthaus Litteken 3 months ago
Debian package update: pull request: https://github.com/puppetlabs/pe-deb-facter/pull/1, commit https://github.com/haus/pe-deb-facter/commit/7db9712d434466926755145a4414e3c4bed3600a
EL/SLES Package update: Pull https://github.com/puppetlabs/pe-packages/pull/100, commit: https://github.com/haus/pe-packages/commit/938a00043e2f0b07abecd8fb12041395c5ea5044
The installer and upgrader will need to enqueue pciutils for all but solaris while dmidecode will need to be enqueued on debian, ubuntu, and el (doesn’t exist on sles though).
Installer/Upgrader update: Pull https://github.com/puppetlabs/enterprise-dist/pull/398/files, commit https://github.com/haus/enterprise-dist/commit/18899183bc4129ef23e635396c5531f18c19350f
Updated by Matthaus Litteken 3 months ago
- Status changed from Accepted to In Topic Branch Pending Review
Updated by Michael Stahnke 3 months ago
- Status changed from In Topic Branch Pending Review to Merged - Pending Release
merged into pe-packages at https://github.com/puppetlabs/pe-packages/commit/c2d91d225186f280e13a87a72433d4305a68697b