Skip to content

Commit 2f5addf

Browse files
author
Eric Cook
committed
explictly use the change action for udevadm trigger
centos 6 defaults to --action=add https://lists.ubuntu.com/archives/ubuntu-devel/2009-January/027260.html https://bugzilla.redhat.com/show_bug.cgi?id=570355
1 parent 4abfdeb commit 2f5addf

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

manifests/params.pp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
'debian': {
1010
$udev_package = 'udev'
1111
$udevlogpriority = 'udevadm control --log-priority'
12-
$udevtrigger = 'udevadm trigger'
12+
$udevtrigger = 'udevadm trigger --action=change'
1313
}
1414
'redhat': {
1515
if $::operatingsystem == 'Fedora' {
@@ -30,12 +30,12 @@
3030
}
3131
'6': {
3232
$udev_package = 'udev'
33-
$udevtrigger = 'udevadm trigger'
33+
$udevtrigger = 'udevadm trigger --action=change'
3434
$udevlogpriority = 'udevadm control --log-priority'
3535
}
3636
'7': {
3737
$udev_package = 'systemd'
38-
$udevtrigger = 'udevadm trigger'
38+
$udevtrigger = 'udevadm trigger --action=change'
3939
$udevlogpriority = 'udevadm control --log-priority'
4040
}
4141
default: {

spec/classes/udevadm/trigger_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111

1212
it { should contain_class('udev::udevadm::trigger') }
1313
it do
14-
should contain_exec('udevadm trigger').with({
14+
should contain_exec('udevadm trigger --action=change').with({
1515
:refreshonly => true,
1616
:path => ['/sbin'],
1717
})

0 commit comments

Comments
 (0)