This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a commit to branch nsx-vpc-routed-mode in repository https://gitbox.apache.org/repos/asf/cloudstack.git
commit 36b1af05a9015785a2485b0c2ec13bf8a35658fe Author: Pearl Dsilva <pearl1...@gmail.com> AuthorDate: Tue Jan 23 18:51:36 2024 -0500 NSX: Fix VPC routed mode --- .../apache/cloudstack/engine/orchestration/NetworkOrchestrator.java | 2 +- server/src/main/java/com/cloud/network/vpc/VpcManagerImpl.java | 6 +++--- ui/src/views/network/VpcTiersTab.vue | 6 ++++++ 3 files changed, 10 insertions(+), 4 deletions(-) diff --git a/engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java b/engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java index 9e3a6db0a20..738f6be6560 100644 --- a/engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java +++ b/engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java @@ -941,7 +941,7 @@ public class NetworkOrchestrator extends ManagerBase implements NetworkOrchestra * @return ordered list of Network and NicProfile pair * @param networks the map od networks to nic profiles list */ - private List<Pair<Network, NicProfile>> getOrderedNetworkNicProfileMapping(final LinkedHashMap<? extends Network, List<? extends NicProfile>> networks) { + private List<Pair<Network, NicProfile>> getOrderedNetworkNicProfileMapping(final LinkedHashMap<? extends Network, List<? extends NicProfile>> networks) { List<Pair<Network, NicProfile>> profilesList = new ArrayList<>(); for (final Map.Entry<? extends Network, List<? extends NicProfile>> network : networks.entrySet()) { List<? extends NicProfile> requestedProfiles = network.getValue(); diff --git a/server/src/main/java/com/cloud/network/vpc/VpcManagerImpl.java b/server/src/main/java/com/cloud/network/vpc/VpcManagerImpl.java index 17277b2f7dd..5f22e8470ef 100644 --- a/server/src/main/java/com/cloud/network/vpc/VpcManagerImpl.java +++ b/server/src/main/java/com/cloud/network/vpc/VpcManagerImpl.java @@ -403,7 +403,7 @@ public class VpcManagerImpl extends ManagerBase implements VpcManager, VpcProvis if (List.of(Service.UserData, Service.Dhcp, Service.Dns).contains(svc)) { final Set<Provider> userDataProvider = Set.of(Provider.VPCVirtualRouter); svcProviderMap.put(svc, userDataProvider); - } else { + } else if (Service.NetworkACL.equals(svc)){ svcProviderMap.put(svc, defaultProviders); } } @@ -3189,8 +3189,8 @@ public class VpcManagerImpl extends ManagerBase implements VpcManager, VpcProvis @Override public boolean isSrcNatIpRequired(long vpcOfferingId) { final Map<Network.Service, Set<Network.Provider>> vpcOffSvcProvidersMap = getVpcOffSvcProvidersMap(vpcOfferingId); - return vpcOffSvcProvidersMap.get(Network.Service.SourceNat).contains(Network.Provider.VPCVirtualRouter) || - vpcOffSvcProvidersMap.get(Service.SourceNat).contains(Provider.Nsx); + return Objects.nonNull(vpcOffSvcProvidersMap.get(Network.Service.SourceNat)) && (vpcOffSvcProvidersMap.get(Network.Service.SourceNat).contains(Network.Provider.VPCVirtualRouter) || + vpcOffSvcProvidersMap.get(Service.SourceNat).contains(Provider.Nsx)); } /** diff --git a/ui/src/views/network/VpcTiersTab.vue b/ui/src/views/network/VpcTiersTab.vue index 8bb84392bd3..5742cdf1341 100644 --- a/ui/src/views/network/VpcTiersTab.vue +++ b/ui/src/views/network/VpcTiersTab.vue @@ -461,8 +461,14 @@ export default { this.form = reactive({}) this.rules = reactive({}) }, +<<<<<<< Updated upstream showIlb (network) { return network.service.filter(s => (s.name === 'Lb') && (s.capability.filter(c => c.name === 'LbSchemes' && c.value === 'Internal').length > 0)).length > 0 || false +======= + async showIlb (network) { + const networkOffering = await this.getNetworkOffering(network.networkofferingid) + return ((networkOffering.supportsinternallb && network.service.filter(s => (s.name === 'Lb') && (s.capability.filter(c => c.name === 'LbSchemes' && c.value.split(',').includes('Internal')).length > 0)).length > 0)) +>>>>>>> Stashed changes }, updateMtu () { if (this.form.privatemtu > this.privateMtuMax) {