Skip to content

Commit 7c7041b

Browse files
committed
revert NsxService inject
Will re-implement later either with ComponentContext.getDelegateComponentOfType. Need to check Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
1 parent f917a54 commit 7c7041b

3 files changed

Lines changed: 5 additions & 10 deletions

File tree

api/src/main/java/com/cloud/network/nsx/NsxService.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,5 +33,4 @@ public interface NsxService {
3333

3434
boolean createVpcNetwork(Long zoneId, long accountId, long domainId, Long vpcId, String vpcName, boolean sourceNatEnabled);
3535
boolean updateVpcSourceNatIp(Vpc vpc, IpAddress address);
36-
String getSegmentId(long domainId, long accountId, long zoneId, Long vpcId, long networkId);
3736
}

plugins/network-elements/nsx/src/main/java/org/apache/cloudstack/service/NsxServiceImpl.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -200,9 +200,4 @@ public ConfigKey<?>[] getConfigKeys() {
200200
NSX_API_FAILURE_RETRIES, NSX_API_FAILURE_INTERVAL
201201
};
202202
}
203-
204-
@Override
205-
public String getSegmentId(long domainId, long accountId, long zoneId, Long vpcId, long networkId) {
206-
return NsxControllerUtils.getNsxSegmentId(domainId, accountId, zoneId, vpcId, networkId);
207-
}
208203
}

server/src/main/java/com/cloud/network/NetworkServiceImpl.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,6 @@
178178
import com.cloud.network.element.VpcVirtualRouterElement;
179179
import com.cloud.network.guru.GuestNetworkGuru;
180180
import com.cloud.network.guru.NetworkGuru;
181-
import com.cloud.network.nsx.NsxService;
182181
import com.cloud.network.router.CommandSetupHelper;
183182
import com.cloud.network.router.NetworkHelper;
184183
import com.cloud.network.router.VirtualRouter;
@@ -426,8 +425,6 @@ public class NetworkServiceImpl extends ManagerBase implements NetworkService, C
426425
private BGPService bgpService;
427426
@Inject
428427
private ASNumberDao asNumberDao;
429-
@Inject
430-
NsxService nsxService;
431428

432429
List<InternalLoadBalancerElementService> internalLoadBalancerElementServices = new ArrayList<>();
433430
Map<String, InternalLoadBalancerElementService> internalLoadBalancerElementServiceMap = new HashMap<>();
@@ -6347,7 +6344,11 @@ public String getNicVlanValueForExternalVm(NicTO nic) {
63476344
Networks.BroadcastDomainType broadcastDomainType = Networks.BroadcastDomainType.getSchemeValue(nic.getBroadcastUri());
63486345
if (Networks.BroadcastDomainType.NSX.equals(broadcastDomainType)) {
63496346
NetworkVO networkVO = _networksDao.findById(nic.getNetworkId());
6350-
return nsxService.getSegmentId(networkVO.getDomainId(), networkVO.getDataCenterId(), networkVO.getAccountId(), networkVO.getVpcId(), networkVO.getId());
6347+
String segmentName = String.format("D%s-A%s-Z%s", networkVO.getDomainId(), networkVO.getAccountId(), networkVO.getDataCenterId());
6348+
if (Objects.isNull(networkVO.getVpcId())) {
6349+
return String.format("%s-S%s", segmentName, networkVO.getId());
6350+
}
6351+
return String.format("%s-V%s-S%s",segmentName, networkVO.getVpcId(), networkVO.getId());
63516352
}
63526353
return Networks.BroadcastDomainType.getValue(nic.getBroadcastUri());
63536354
}

0 commit comments

Comments
 (0)