Skip to content

Commit 91bc8be

Browse files
authored
Merge pull request #196 from baby-gnu/fix/tofs-pillars
fix(pillar): `tofs` must not be under `mine_functions`
2 parents 9576b72 + c0d5052 commit 91bc8be

File tree

16 files changed

+160
-20
lines changed

16 files changed

+160
-20
lines changed

pillar.example

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -361,16 +361,6 @@ openssh:
361361
# salt://files/ssh/moduli.hash
362362
# These will be automatically referenced to by the ssh_moduli state.
363363

364-
# Required for openssh.known_hosts
365-
mine_functions:
366-
public_ssh_host_keys:
367-
mine_function: cmd.run
368-
cmd: cat /etc/ssh/ssh_host_*_key.pub
369-
python_shell: true
370-
public_ssh_hostname:
371-
mine_function: grains.get
372-
key: id
373-
374364
tofs:
375365
# The files_switch key serves as a selector for alternative
376366
# directories under the formula files directory. See TOFS pattern
@@ -399,3 +389,13 @@ mine_functions:
399389
- alt_ssh_config
400390
sshd_banner:
401391
- fire_banner
392+
393+
# Required for openssh.known_hosts
394+
mine_functions:
395+
public_ssh_host_keys:
396+
mine_function: cmd.run
397+
cmd: cat /etc/ssh/ssh_host_*_key.pub
398+
python_shell: true
399+
public_ssh_hostname:
400+
mine_function: grains.get
401+
key: id

test/integration/default/files/_mapdata/amazonlinux-1.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,16 @@ openssh:
151151
sshd_config_src: sshd_config
152152
sshd_config_user: root
153153
sshd_enable: true
154+
tofs:
155+
source_files:
156+
manage ssh_known_hosts file:
157+
- alt_ssh_known_hosts
158+
ssh_config:
159+
- alt_ssh_config
160+
sshd_banner:
161+
- fire_banner
162+
sshd_config:
163+
- alt_sshd_config
154164
ssh_config:
155165
Hosts:
156166
'*':

test/integration/default/files/_mapdata/amazonlinux-2.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,16 @@ openssh:
151151
sshd_config_src: sshd_config
152152
sshd_config_user: root
153153
sshd_enable: true
154+
tofs:
155+
source_files:
156+
manage ssh_known_hosts file:
157+
- alt_ssh_known_hosts
158+
ssh_config:
159+
- alt_ssh_config
160+
sshd_banner:
161+
- fire_banner
162+
sshd_config:
163+
- alt_sshd_config
154164
ssh_config:
155165
Hosts:
156166
'*':

test/integration/default/files/_mapdata/arch-base-latest.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,16 @@ openssh:
151151
sshd_config_src: sshd_config
152152
sshd_config_user: root
153153
sshd_enable: true
154+
tofs:
155+
source_files:
156+
manage ssh_known_hosts file:
157+
- alt_ssh_known_hosts
158+
ssh_config:
159+
- alt_ssh_config
160+
sshd_banner:
161+
- fire_banner
162+
sshd_config:
163+
- alt_sshd_config
154164
ssh_config:
155165
Hosts:
156166
'*':

test/integration/default/files/_mapdata/centos-6.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,16 @@ openssh:
151151
sshd_config_src: sshd_config
152152
sshd_config_user: root
153153
sshd_enable: true
154+
tofs:
155+
source_files:
156+
manage ssh_known_hosts file:
157+
- alt_ssh_known_hosts
158+
ssh_config:
159+
- alt_ssh_config
160+
sshd_banner:
161+
- fire_banner
162+
sshd_config:
163+
- alt_sshd_config
154164
ssh_config:
155165
Hosts:
156166
'*':

test/integration/default/files/_mapdata/centos-7.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,16 @@ openssh:
151151
sshd_config_src: sshd_config
152152
sshd_config_user: root
153153
sshd_enable: true
154+
tofs:
155+
source_files:
156+
manage ssh_known_hosts file:
157+
- alt_ssh_known_hosts
158+
ssh_config:
159+
- alt_ssh_config
160+
sshd_banner:
161+
- fire_banner
162+
sshd_config:
163+
- alt_sshd_config
154164
ssh_config:
155165
Hosts:
156166
'*':

test/integration/default/files/_mapdata/centos-8.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,16 @@ openssh:
151151
sshd_config_src: sshd_config
152152
sshd_config_user: root
153153
sshd_enable: true
154+
tofs:
155+
source_files:
156+
manage ssh_known_hosts file:
157+
- alt_ssh_known_hosts
158+
ssh_config:
159+
- alt_ssh_config
160+
sshd_banner:
161+
- fire_banner
162+
sshd_config:
163+
- alt_sshd_config
154164
ssh_config:
155165
Hosts:
156166
'*':

test/integration/default/files/_mapdata/debian-10.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,16 @@ openssh:
151151
sshd_config_src: sshd_config
152152
sshd_config_user: root
153153
sshd_enable: true
154+
tofs:
155+
source_files:
156+
manage ssh_known_hosts file:
157+
- alt_ssh_known_hosts
158+
ssh_config:
159+
- alt_ssh_config
160+
sshd_banner:
161+
- fire_banner
162+
sshd_config:
163+
- alt_sshd_config
154164
ssh_config:
155165
Hosts:
156166
'*':

test/integration/default/files/_mapdata/debian-9.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,16 @@ openssh:
151151
sshd_config_src: sshd_config
152152
sshd_config_user: root
153153
sshd_enable: true
154+
tofs:
155+
source_files:
156+
manage ssh_known_hosts file:
157+
- alt_ssh_known_hosts
158+
ssh_config:
159+
- alt_ssh_config
160+
sshd_banner:
161+
- fire_banner
162+
sshd_config:
163+
- alt_sshd_config
154164
ssh_config:
155165
Hosts:
156166
'*':

test/integration/default/files/_mapdata/fedora-31.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,16 @@ openssh:
151151
sshd_config_src: sshd_config
152152
sshd_config_user: root
153153
sshd_enable: true
154+
tofs:
155+
source_files:
156+
manage ssh_known_hosts file:
157+
- alt_ssh_known_hosts
158+
ssh_config:
159+
- alt_ssh_config
160+
sshd_banner:
161+
- fire_banner
162+
sshd_config:
163+
- alt_sshd_config
154164
ssh_config:
155165
Hosts:
156166
'*':

0 commit comments

Comments
 (0)