test(inspec): no more need to mangle mapdata for hostname
We force the hostname in `kitchen` so the `_mapdata` files can be static.
This commit is contained in:
parent
208f87380c
commit
8cb31c6967
@ -1,12 +1,7 @@
|
|||||||
# frozen_string_literal: true
|
# frozen_string_literal: true
|
||||||
|
|
||||||
# Replace per minion strings
|
|
||||||
replacement = {
|
|
||||||
hostname: system.hostname
|
|
||||||
}
|
|
||||||
|
|
||||||
mapdata_file = "_mapdata/#{system.platform[:finger].split('.').first}.yaml"
|
mapdata_file = "_mapdata/#{system.platform[:finger].split('.').first}.yaml"
|
||||||
mapdata_dump = inspec.profile.file(mapdata_file) % replacement
|
mapdata_dump = inspec.profile.file(mapdata_file)
|
||||||
|
|
||||||
control '`map.jinja` YAML dump' do
|
control '`map.jinja` YAML dump' do
|
||||||
title 'should contain the lines'
|
title 'should contain the lines'
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-clients
|
client: openssh-clients
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-clients
|
client: openssh-clients
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh
|
client: openssh
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-clients
|
client: openssh-clients
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-clients
|
client: openssh-clients
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-clients
|
client: openssh-clients
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-client
|
client: openssh-client
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-client
|
client: openssh-client
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-clients
|
client: openssh-clients
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-clients
|
client: openssh-clients
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh
|
client: openssh
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-client
|
client: openssh-client
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-client
|
client: openssh-client
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -42,7 +42,7 @@ openssh:
|
|||||||
present: false
|
present: false
|
||||||
banner: /etc/ssh/banner
|
banner: /etc/ssh/banner
|
||||||
banner_src: banner
|
banner_src: banner
|
||||||
banner_string: 'Welcome to %{hostname}!
|
banner_string: 'Welcome to example.net!
|
||||||
'
|
'
|
||||||
client: openssh-client
|
client: openssh-client
|
||||||
client_version: latest
|
client_version: latest
|
||||||
|
@ -10,7 +10,6 @@ It's goal is to share the libraries between all profiles.
|
|||||||
|
|
||||||
The `system` library provides easy access to system dependents informations:
|
The `system` library provides easy access to system dependents informations:
|
||||||
|
|
||||||
- `system.hostname`: return the result of `hostname -s` or `hostnamectl --static` based on the availability of each commans
|
|
||||||
- `system.platform`: take `inspec.platform` and mangle things a bit to be useful
|
- `system.platform`: take `inspec.platform` and mangle things a bit to be useful
|
||||||
- `system.platform[:family]` provides a family name for Arch
|
- `system.platform[:family]` provides a family name for Arch
|
||||||
- `system.platform[:name]` modify `amazon` to `amazonlinux`
|
- `system.platform[:name]` modify `amazon` to `amazonlinux`
|
||||||
|
@ -4,47 +4,17 @@
|
|||||||
# Author: Daniel Dehennin <daniel.dehennin@ac-dijon.fr>
|
# Author: Daniel Dehennin <daniel.dehennin@ac-dijon.fr>
|
||||||
# Copyright (C) 2020 Daniel Dehennin <daniel.dehennin@ac-dijon.fr>
|
# Copyright (C) 2020 Daniel Dehennin <daniel.dehennin@ac-dijon.fr>
|
||||||
|
|
||||||
HOSTNAME_CMDS = %w[hostname hostnamectl].freeze
|
|
||||||
HOSTNAME_CMDS_OPT = {
|
|
||||||
'hostname' => '-s',
|
|
||||||
'hostnamectl' => '--static'
|
|
||||||
}.freeze
|
|
||||||
|
|
||||||
class SystemResource < Inspec.resource(1)
|
class SystemResource < Inspec.resource(1)
|
||||||
name 'system'
|
name 'system'
|
||||||
|
|
||||||
attr_reader :platform
|
attr_reader :platform
|
||||||
attr_reader :hostname
|
|
||||||
|
|
||||||
def initialize
|
def initialize
|
||||||
@platform = build_platform
|
@platform = build_platform
|
||||||
@hostname = found_hostname
|
|
||||||
end
|
end
|
||||||
|
|
||||||
private
|
private
|
||||||
|
|
||||||
def found_hostname
|
|
||||||
cmd = guess_hostname_cmd
|
|
||||||
|
|
||||||
unless cmd.exit_status.zero?
|
|
||||||
raise Inspec::Exceptions::ResourceSkipped,
|
|
||||||
"Error running '#{cmd}': #{cmd.stderr}"
|
|
||||||
end
|
|
||||||
|
|
||||||
cmd.stdout.chomp
|
|
||||||
end
|
|
||||||
|
|
||||||
def guess_hostname_cmd
|
|
||||||
HOSTNAME_CMDS.each do |cmd|
|
|
||||||
if inspec.command(cmd).exist?
|
|
||||||
return inspec.command("#{cmd} #{HOSTNAME_CMDS_OPT[cmd]}")
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
raise Inspec::Exceptions::ResourceSkipped,
|
|
||||||
"Error: #{@platform[:finger]}} has none of #{HOSTNAME_CMDS.join(', ')}"
|
|
||||||
end
|
|
||||||
|
|
||||||
def build_platform
|
def build_platform
|
||||||
{
|
{
|
||||||
family: build_platform_family,
|
family: build_platform_family,
|
||||||
|
Loading…
Reference in New Issue
Block a user