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

(FACT-2522) Fix for tests/external_facts/fact_directory_precedence.rb #517

Merged
merged 2 commits into from
May 15, 2020
Merged

(FACT-2522) Fix for tests/external_facts/fact_directory_precedence.rb #517

merged 2 commits into from
May 15, 2020

Conversation

BogdanIrimie
Copy link
Contributor

Default directories for external facts were missing, the correct ones are

'/etc/puppetlabs/facter/facts.d'
'/etc/facter/facts.d/'
'/opt/puppetlabs/facter/facts.d'

The order is also important, as the first ones will be loaded first and will have higher priority.

@BogdanIrimie BogdanIrimie added the bugfix Something isn't working label May 15, 2020
@BogdanIrimie BogdanIrimie marked this pull request as ready for review May 15, 2020 10:45
@BogdanIrimie BogdanIrimie requested review from a team May 15, 2020 10:45
@florindragos florindragos merged commit 486127a into master May 15, 2020
@florindragos florindragos deleted the FACT-2522 branch May 15, 2020 11:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@florindragos florindragos florindragos approved these changes

Assignees
No one assigned
Labels
bugfix Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@BogdanIrimie @florindragos