From bc44dd447f7add89c1a17a0cd48277b1aa3ae5c9 Mon Sep 17 00:00:00 2001 From: Patrik Holmqvist Date: Fri, 17 Jan 2025 14:33:15 +0100 Subject: [PATCH] Fix syntax --- .../etc/puppet/modules/net/manifests/baas2_restoretest.pp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/global/overlay/etc/puppet/modules/net/manifests/baas2_restoretest.pp b/global/overlay/etc/puppet/modules/net/manifests/baas2_restoretest.pp index cdda451..031fab6 100644 --- a/global/overlay/etc/puppet/modules/net/manifests/baas2_restoretest.pp +++ b/global/overlay/etc/puppet/modules/net/manifests/baas2_restoretest.pp @@ -1,6 +1,9 @@ # This puppet manifest is used to test backup and restore in an automated fashion # This tests the backup and restores when manged by sunet::baas2 from puppet-sunet # Jira-ref: SUNETOPS-1997 + +# @param node_type The type of node, backupnode does the backups, restorenode does the restores +# @param backup_nodename The name of the node that does the backups (only used by the restore node) class net::baas2_restoretest( Enum['backupnode', 'restorenode'] $node_type, Optional[String] $backup_nodename = undef, @@ -15,9 +18,11 @@ class net::baas2_restoretest( if $node_type == 'restorenode' { if $backup_nodename == undef { - fail{'Variable backup_nodename must be set for a restorenode':} + fail('Variable "backup_nodename" must be set for a restorenode') } + package { 'expect': ensure => installed } + file { '/opt/restore-target': ensure => directory, mode => '0755', @@ -90,5 +95,4 @@ class net::baas2_restoretest( ok_criteria => ['exit_status=0', 'max_age=48h'], } } - }