Skip to content

Commit c98ef69

Browse files
committed
fix persiting ilbvm
Signed-off-by: Abhishek Kumar <[email protected]>
1 parent 3069929 commit c98ef69

File tree

1 file changed

+3
-11
lines changed

1 file changed

+3
-11
lines changed

plugins/network-elements/internal-loadbalancer/src/main/java/org/apache/cloudstack/network/lb/InternalLoadBalancerVMManagerImpl.java

+3-11
Original file line numberDiff line numberDiff line change
@@ -751,9 +751,7 @@ protected String getRouterTemplateForHypervisor(HypervisorType hypervisorType, l
751751

752752
protected DomainRouterVO createOrUpdateInternalLb(DomainRouterVO internalLbVm, final long id,
753753
final long internalLbProviderId, final Account owner, final long userId, final Long vpcId,
754-
final ServiceOffering routerOffering,
755-
final LinkedHashMap<Network, List<? extends NicProfile>> networks,
756-
final VMTemplateVO template) {
754+
final ServiceOffering routerOffering, final VMTemplateVO template) {
757755
if (internalLbVm == null) {
758756
internalLbVm = new DomainRouterVO(id, routerOffering.getId(), internalLbProviderId,
759757
VirtualMachineName.getSystemVmName(id, _instance, InternalLbVmNamePrefix),
@@ -780,14 +778,8 @@ protected DomainRouterVO deployInternalLbVmWithTemplates(DomainRouterVO internal
780778
for (final Iterator<VMTemplateVO> templatesIterator = templates.iterator(); templatesIterator.hasNext();) {
781779
final VMTemplateVO template = templatesIterator.next();
782780
try {
783-
internalLbVm = new DomainRouterVO(id, routerOffering.getId(), internalLbProviderId,
784-
VirtualMachineName.getSystemVmName(id, _instance, InternalLbVmNamePrefix),
785-
template.getId(), template.getHypervisorType(), template.getGuestOSId(),
786-
owner.getDomainId(), owner.getId(), userId, false,
787-
RedundantState.UNKNOWN, false, false,
788-
VirtualMachine.Type.InternalLoadBalancerVm, vpcId);
789-
internalLbVm.setRole(Role.INTERNAL_LB_VM);
790-
internalLbVm = _internalLbVmDao.persist(internalLbVm);
781+
internalLbVm = createOrUpdateInternalLb(internalLbVm, id, internalLbProviderId, owner, userId, vpcId,
782+
routerOffering, template);
791783
_itMgr.allocate(internalLbVm.getInstanceName(), template, routerOffering, networks, plan, null);
792784
internalLbVm = _internalLbVmDao.findById(internalLbVm.getId());
793785
if (templatesIterator.hasNext()) {

0 commit comments

Comments
 (0)