From dba7c5d40dcff14c8b93ff16da8350de0675dc28 Mon Sep 17 00:00:00 2001 From: lissav Date: Sun, 26 Aug 2012 11:22:55 +0000 Subject: [PATCH] fix defect 3555748 git-svn-id: https://svn.code.sf.net/p/xcat/code/xcat-core/trunk@13613 8638fb3e-16cb-4fca-ae20-7b5d299a9bcd --- xCAT-server/lib/xcat/plugins/updatenode.pm | 26 +++++++++++++--------- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/xCAT-server/lib/xcat/plugins/updatenode.pm b/xCAT-server/lib/xcat/plugins/updatenode.pm index 34ed1f615..d428efd63 100644 --- a/xCAT-server/lib/xcat/plugins/updatenode.pm +++ b/xCAT-server/lib/xcat/plugins/updatenode.pm @@ -826,10 +826,10 @@ sub updatenode push @{$syncfile_node{$synclist}}, $node; } } - # Check the existence of the synclist file - foreach my $synclist (keys %syncfile_node) - { + if (%syncfile_node) { # there are files to sync defined + foreach my $synclist (keys %syncfile_node) + { if (!(-r $synclist)) { my $rsp = {}; @@ -838,11 +838,11 @@ sub updatenode xCAT::MsgUtils->message("E", $rsp, $callback); return 1; } - } + } - # Sync files to the target nodes - foreach my $synclist (keys %syncfile_node) - { + # Sync files to the target nodes + foreach my $synclist (keys %syncfile_node) + { if ($::VERBOSE) { my $rsp = {}; @@ -873,10 +873,16 @@ sub updatenode }, $callback ); + } + my $rsp = {}; + $rsp->{data}->[0] = "File synchronization has completed."; + $callback->($rsp); + } else { # no syncfiles defined + my $rsp = {}; + $rsp->{data}->[0] = "There were no syncfiles defined to process. File synchronization has completed."; + $callback->($rsp); + } - my $rsp = {}; - $rsp->{data}->[0] = "File synchronization has completed."; - $callback->($rsp); } if (scalar(@$AIXnodes))