Skip to content
This repository has been archived by the owner on Jun 19, 2020. It is now read-only.

(FACT-2636) Set external as fact_type for environment variable facts. #537

Merged
merged 1 commit into from
May 29, 2020

Conversation

BogdanIrimie
Copy link
Contributor

@BogdanIrimie BogdanIrimie commented May 27, 2020

Environment variable external facts were missing fact_type: :external option, and because of this they were ignored when resolving facts. This is required since Facter 4.0.22 as external facts are identified based on fact_type option.

@BogdanIrimie BogdanIrimie added the bugfix Something isn't working label May 27, 2020
@BogdanIrimie BogdanIrimie requested review from a team May 27, 2020 12:43
@Magisus
Copy link

Magisus commented May 28, 2020

Would it be possible to get this released soon? PDB tests are still failing because of it. If not, please let me know, so I can revert our switch to testing with Facter 4 in PDB and puppetserver.

Copy link

@Magisus Magisus left a comment

Choose a reason for hiding this comment

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

I verified that this fix resolves the issue, using the repro case in the ticket.

@BogdanIrimie BogdanIrimie merged commit 4e48155 into master May 29, 2020
@BogdanIrimie BogdanIrimie deleted the FACT-2636 branch May 29, 2020 07:02
@BogdanIrimie
Copy link
Contributor Author

@Magisus released Facter 4.0.25 today, please give it a try.

netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Jun 7, 2020
Update ruby-facter to 4.0.25.


## [4.0.25](https://github.com/puppetlabs/facter-ng/tree/4.0.25) (2020-05-29)

### Fixed

- \(FACT-2636\) Set external as fact\_type for environment variable facts. [\#537](puppetlabs/facter-ng#537) ([IrimieBogdan](https://github.com/IrimieBogdan))


## [4.0.24] (2020-05-26)

### Added

- \(FACT-2605\) Add vmware resolver [\#525](puppetlabs/facter-ng#525) ([oanatmaria](https://github.com/oanatmaria))
- \(FACT-2604\) Add virt-what resolver [\#523](puppetlabs/facter-ng#523) ([oanatmaria](https://github.com/oanatmaria))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bugfix Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants