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

(FACT-2617) Fix for tests/external_facts/external_fact_stderr_messages_output_to_stderr.rb #522

Merged
merged 11 commits into from
May 20, 2020

Conversation

BogdanIrimie
Copy link
Contributor

The error message was nor printed when using puppet facts because only debug messages were using the puppet logger.
The new implementation uses the puppet logger if it is provided in facter for all log levels.

@BogdanIrimie BogdanIrimie added the bugfix Something isn't working label May 18, 2020
@BogdanIrimie BogdanIrimie force-pushed the FACT-2617 branch 2 times, most recently from 98df882 to cffadd1 Compare May 18, 2020 14:15
@BogdanIrimie BogdanIrimie marked this pull request as ready for review May 19, 2020 06:19
@BogdanIrimie BogdanIrimie requested review from a team May 19, 2020 06:19
@oanatmaria oanatmaria merged commit 2f066bd into master May 20, 2020
@oanatmaria oanatmaria deleted the FACT-2617 branch May 20, 2020 07:50
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