* Correct puppet-lint warnings.

This commit is contained in:
gardouille 2015-06-01 11:56:08 +02:00
parent 2aedaf1ea9
commit 697a4bfcd0
4 changed files with 6 additions and 2 deletions

View File

@ -10,6 +10,7 @@
* Some installation's instructions only run if running a PVE kernel (test the new is_pve_kernel fact).
* Use the stdlib function 'ensure_packages' to install recommended packages instead of 'if ! defined'.
* Upgrade pve-kernel-3.10.0-9-pve and pve-kernel-2.6.32-39 to the last version.
* Correct puppet-lint warnings.
##### Bugfixes
* #14 The module no longer automatically reboot nodes after kernel upgrade, show a warning message instead.

View File

@ -16,6 +16,7 @@ class proxmox::hypervisor::cluster
logoutput => 'on_failure',
}
## Quoted boolean value because can't return "true" boolean with personal fact
if $::is_proxmox == 'true' and $proxmox::hypervisor::cluster_master_ip != undef and $proxmox::hypervisor::cluster_name != undef {
# Ensure the root user got an ssh-key
exec { 'create ssh-key for root':

View File

@ -16,6 +16,7 @@ class proxmox::hypervisor::config {
}
## Quoted boolean value because can't return "true" boolean with personal fact
if $::is_proxmox == 'true' {
# Pveproxy access control list

View File

@ -10,6 +10,7 @@ class proxmox::hypervisor::install {
}
# If the system already run a PVE kernel
## Quoted boolean value because can't return "true" boolean with personal fact
if $::is_pve_kernel == 'true' {
# Installation of Virtual Environnment
@ -35,7 +36,7 @@ class proxmox::hypervisor::install {
# Installation of the PVE Kernel
if $proxmox::hypervisor::kvm_only == true {
notify { "Please REBOOT":
notify { 'Please REBOOT':
message => "Need to REBOOT the system on the new PVE kernel (${proxmox::hypervisor::kernel_kvm_pkg_name}) ...",
loglevel => warning,
}
@ -46,7 +47,7 @@ class proxmox::hypervisor::install {
}
}
else {
notify { "Please REBOOT":
notify { 'Please REBOOT':
message => "Need to REBOOT the system on the new PVE kernel (${proxmox::hypervisor::kernel_pkg_name}) ...",
loglevel => warning,
}