Issue #20989 has been updated by Josh Cooper. Status changed from In Topic Branch Pending Review to Merged - Pending Release Target version set to 1.7.3 Affected Facter version changed from development to 1.7.1
Originally merged into master in commit 164f5c3. Backported into 5d2bd57 into stable, and merged up to master in 5c402b7. ---------------------------------------- Bug #20989: Facter 1.7.1 on 32 bit windows reports architecture fact as "i1586", should be "x86" https://projects.puppetlabs.com/issues/20989#change-94206 * Author: Shane Madden * Status: Merged - Pending Release * Priority: Normal * Assignee: Rob Reynolds * Category: windows * Target version: 1.7.3 * Keywords: windows * Branch: https://github.com/puppetlabs/facter/pull/473 * Affected Facter version: 1.7.1 ---------------------------------------- 9c38f19's fix to the `hardwaremodel` fact seems to have changed the value of the `architecture` fact as well. The WMI for the CPUs on this system (a VMware virtual machine running 32 bit Windows 2008) returns an Address Width of 32, an Architecture of 9, and a Level of 15. Based on the changed logic in hardwaremodel.rb, this gives it a `hardwaremodel` of `i1586`, which seems odd.. but it's what the OS is reporting, so we'll go with it. However, this misses the regex in architecture.rb, `/(i[3456]86|pentium)/` - so it falls back to `architecture` being set to the value of `hardwaremodel`, instead of being properly set to `x86`. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/puppet-bugs. For more options, visit https://groups.google.com/groups/opt_out.
