Skip to content

Commit

Permalink
fix spelling in warn message
Browse files Browse the repository at this point in the history
previously this was:

```
Warning: Facter: Container runtime, 'docker', is unsupported, setting to, 'container_other'
```

But I think that is wrong? And it should be:

```
Warning: Facter: Container runtime, 'docker', is unsupported, setting to 'container_other'
```
  • Loading branch information
bastelfreak committed Aug 13, 2024
1 parent e776b42 commit c0fd359
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion lib/facter/resolvers/containers.rb
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ def read_environ(fact_name)
info = { 'id' => Facter::Util::FileHelper.safe_read('/etc/machine-id', nil).strip }
else
vm = 'container_other'
log.warn("Container runtime, '#{container}', is unsupported, setting to, '#{vm}'")
log.warn("Container runtime, '#{container}', is unsupported, setting to '#{vm}'")
end
@fact_list[:vm] = vm
@fact_list[:hypervisor] = { vm.to_sym => info } if vm
Expand Down
2 changes: 1 addition & 1 deletion spec/facter/resolvers/containers_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@
let(:logger) { Facter::Log.class_variable_get(:@@logger) }

it 'return container_other for vm' do
expect(logger).to receive(:warn).with(/Container runtime, 'UNKNOWN', is unsupported, setting to, 'container_other'/)
expect(logger).to receive(:warn).with(/Container runtime, 'UNKNOWN', is unsupported, setting to 'container_other'/)
expect(containers_resolver.resolve(:vm)).to eq('container_other')
end
end
Expand Down

0 comments on commit c0fd359

Please sign in to comment.