-
Notifications
You must be signed in to change notification settings - Fork 494
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add processors.extensions fact #2664
Conversation
Can one of the admins verify this patch? |
b0a0915
to
5977abb
Compare
456fe92
to
951ab1b
Compare
90d1586
to
a09d504
Compare
f25a5c1
to
cd8cc11
Compare
8e6f350
to
40a198a
Compare
40a198a
to
8de941e
Compare
Adding notes from some debugging about the order-dependent test failure, which is showing up on Windows. This passes:
But this doesn't:
It seems there's a logger instance spy leaking from one test to another. First, we're not consistent with how loggers are initialized, for example, this is evaluated as soon as the resolver is required:
But this is done lazily: facter/lib/facter/resolvers/base_resolver.rb Lines 6 to 8 in dbfb450
Also the resolver tests aren't consistent with using instance_spy's, for example, here we use the real
Here we use an
And here we don't specify a logger at all: facter/spec/facter/resolvers/processors_spec.rb Lines 3 to 6 in 5194a37
|
8de941e
to
d4ceb12
Compare
Rebased off HEAD |
d4ceb12
to
71cc964
Compare
Signed-off-by: Pat Riehecky <[email protected]>
71cc964
to
d6a0020
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$ bundle exec facter processors.extensions
[
"x86_64",
"x86_64-v1",
"x86_64-v2",
"x86_64-v3",
"x86_64-v4"
]
Thanks @jcpunk ! |
This should add in an extra set of attributes for finding your microarch or CPU extensions.
Addresses #2663
I'll probably need help with the tests...