arif/xcat-core
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

fixing bug 3815: check options again in process_request to make sure makedhcp options can be processes on service node.

Browse Source
This commit is contained in:
wanghuaz 2013-11-06 17:31:36 +08:00
parent 957403a1d2
commit 4624c6b9fa
1 changed files with 16 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
xCAT-server/lib/xcat/plugins/dhcp.pm
View File

@ -849,7 +849,8 @@ sub check_options
Getopt::Long::Configure("no_pass_through");
# Exit if the packet has been preprocessed
if ($req->{_xcatpreprocessed}->[0] == 1) { return [$req]; }
# Comment this line to make sure check_options can be processed on service node.
#if ($req->{_xcatpreprocessed}->[0] == 1) { return [$req]; }
# Save the arguements in ARGV for GetOptions
if ($req && $req->{arg}) { @ARGV = @{$req->{arg}}; }
@ -882,7 +883,7 @@ sub check_options
{
my $rsp = {};
$rsp->{data}->[0] = $usage;
xCAT::MsgUtils->message("I", $rsp, $callback, 1);
xCAT::MsgUtils->message("I", $rsp, $callback, 0);
return 0;
}
@ -910,6 +911,8 @@ sub check_options
xCAT::MsgUtils->message("I", $rsp, $callback, 1);
return;
}
return 0;
}
############################################################
@ -922,7 +925,10 @@ sub preprocess_request
my $rc = 0;
# check the syntax
check_options($req,$callback);
$rc = check_options($req,$callback);
if ( $rc ) {
return [];
}
my $snonly=0;
my @entries = xCAT::TableUtils->get_site_attribute("disjointdhcps");
@ -1095,7 +1101,13 @@ sub process_request
#print Dumper($req);
# Check options again in case we are called from plugin and options have not been processed
check_options($req,$callback);
my $rc = 0;
$rc = check_options($req,$callback);
if ( $rc ) {
return [];
}
# if option is query then call listnode for each node and return
if ($::opt_q)