4524
87%
master: 87%

Ran 09 Oct 2018 01:55AM UTC

Jobs 1

Files 380

Run time 11s

Badge

Embed ▾

pending completion

Build # 4524

Build Type

Pull #1753

travis-ci

Committed by Enis Inan

Commit Message

(FACT-1873) Use builtin fact if custom facts don't resolve or have 0 weight

Previously, if any builtin facts have conflicting custom facts that all
fail to resolve, and at least one of the custom facts has a weight > 0,
Facter will not report a value for that fact. This is undesirable
behavior because we always want to fall back to the builtin value if
all custom facts fail to resolve. Note that the "weight > 0" part is
important because Facter _will_ use the builtin value if all custom
facts have 0 weight.

Furthermore, the builtin fact should have precedence over 0 weight
custom facts. The existing code enforces this condition when _all_
custom facts have 0 weight, but it does _not_ enforce the condition
when only _some_ custom facts have 0 weight. In the latter case, the 0
weight custom facts will take precedence over the builtin fact value.

This commit fixes the fact resolution logic so that it falls back to the
built-in fact if all custom facts fail to resolve, while also ensuring
that it has precedence over 0 weight custom facts.

Pull Request Pull Request #1753:

Run Details

9796 of 11229 relevant lines covered (87.24%)

83.56 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
6	4524.6 (TRAVIS_TARGET=DEBUG)	09 Oct 2018 01:55AM UTC	0	87.24	Travis Job 4524.6

Source Files on build 4524

Detailed source file information is not available for this build.

puppetlabs / facter / 4524
87%
master: 87%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 4524

puppetlabs / facter / 4524 87% master: 87%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 4524

puppetlabs / facter / 4524
87%
master: 87%

README BADGES
x