Commit f1ab375d authored by Leigh Stoller's avatar Leigh Stoller

Bug Fix: We are not setting the startstatus in the nodes table on

the Geni path. Typically this is done in libvtop after node allocation,
but we call the mapper from getticket with alloconly set, and we end
up skipping that. I have a note in libvtop to look at this, but I want
to get a fix in before the push to stable.
parent fad67e7f
......@@ -1001,11 +1001,17 @@ sub Provision($;$)
# Mark
$node->ModifyReservation({"genisliver_idx" => $self->idx()});
# This would typically happen in libvtop, but we call the mapper
# in a way that causes it to not happen.
$node->ResetStartStatus();
my $subtype = GeniXML::GetVirtualizationSubtype($self->rspec());
if (defined($subtype)) {
if (!$pnode->sharing_mode()) {
$pnode->ModifyReservation({"genisliver_idx" => $self->idx()});
# This would typically happen in libvtop, but we call the mapper
# in a way that causes it to not happen.
$node->ResetStartStatus();
}
}
}
......@@ -1017,6 +1023,9 @@ sub Provision($;$)
if ($node->ModifyReservation({"genisliver_idx" => $self->idx()})) {
return -1;
}
# This would typically happen in libvtop, but we call the mapper
# in a way that causes it to not happen.
$node->ResetStartStatus();
}
return 0;
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment