Skip to content

Commit 7e3ed12

Browse files
authored
Merge pull request #413 from jschoiRR/mold-diplo
[Mold API] 보안그룹용 오퍼링 기본값 생성시 세팅 값 수정(빌드시 오류는 없으나 jetty 구동 시 시작불가 상태)
2 parents 8a0e1cb + 0cf33e0 commit 7e3ed12

3 files changed

Lines changed: 7 additions & 9 deletions

File tree

engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -466,6 +466,7 @@ public boolean configure(final String name, final Map<String, Object> params) th
466466
// populate providers
467467
final Map<Network.Service, Set<Network.Provider>> defaultSharedNetworkOfferingProviders = new HashMap<Network.Service, Set<Network.Provider>>();
468468
final Set<Network.Provider> defaultProviders = new HashSet<Network.Provider>();
469+
final Set<Network.Provider> defaultSGProviders = new HashSet<Network.Provider>();
469470
final Set<Network.Provider> tungstenProvider = new HashSet<>();
470471

471472
defaultProviders.add(Network.Provider.VirtualRouter);
@@ -485,10 +486,10 @@ public boolean configure(final String name, final Map<String, Object> params) th
485486
defaultIsolatedNetworkOfferingProviders.put(Service.Vpn, defaultProviders);
486487

487488
final Map<Network.Service, Set<Network.Provider>> defaultSharedSGEnabledNetworkOfferingProviders = new HashMap<Network.Service, Set<Network.Provider>>();
488-
defaultSharedSGEnabledNetworkOfferingProviders.put(Service.Dhcp, defaultProviders);
489-
defaultSharedSGEnabledNetworkOfferingProviders.put(Service.Dns, defaultProviders);
490-
defaultSharedSGEnabledNetworkOfferingProviders.put(Service.UserData, defaultProviders);
491-
final Set<Provider> sgProviders = new HashSet<Provider>();
489+
defaultSharedSGEnabledNetworkOfferingProviders.put(Service.Dhcp, defaultSGProviders);
490+
defaultSharedSGEnabledNetworkOfferingProviders.put(Service.Dns, defaultSGProviders);
491+
defaultSharedSGEnabledNetworkOfferingProviders.put(Service.UserData, defaultSGProviders);
492+
final Set<Network.Provider> sgProviders = new HashSet<>();
492493
sgProviders.add(Provider.SecurityGroupProvider);
493494
defaultSharedSGEnabledNetworkOfferingProviders.put(Service.SecurityGroup, sgProviders);
494495

@@ -541,7 +542,7 @@ public void doInTransactionWithoutResult(final TransactionStatus status) {
541542

542543
//#2 - SG enabled network offering
543544
if (_networkOfferingDao.findByUniqueName(NetworkOffering.DefaultSharedNetworkOfferingWithSGService) == null && _networkOfferingDao.findByUniqueName("기본 공유 보안그룹 네트워크오퍼링") == null) {
544-
offering = _configMgr.createNetworkOffering(NetworkOffering.DefaultSharedNetworkOfferingWithSGService, "기본 공유 보안그룹 네트워크오퍼링",
545+
offering = _configMgr.createNetworkOffering("기본 공유 보안그룹 네트워크오퍼링", "기본 공유 보안그룹 네트워크오퍼링",
545546
TrafficType.Guest, null, true, Availability.Optional, null, defaultSharedSGEnabledNetworkOfferingProviders, true, Network.GuestType.Shared, false, null, true,
546547
null, true, false, null, false, null, true, false, false, null, null, true, null);
547548
}

server/src/main/java/com/cloud/server/ConfigurationServerImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1030,7 +1030,7 @@ protected void createDefaultNetworkOfferings() {
10301030
public void doInTransactionWithoutResult(TransactionStatus status) {
10311031
// Offering #1
10321032
NetworkOfferingVO defaultSharedSGNetworkOffering =
1033-
new NetworkOfferingVO(NetworkOffering.DefaultSharedNetworkOfferingWithSGService, "기본 공유 보안그룹 네트워크오퍼링",
1033+
new NetworkOfferingVO("기본 공유 보안그룹 네트워크오퍼링", "기본 공유 보안그룹 네트워크오퍼링",
10341034
TrafficType.Guest, false, true, null, null, true, Availability.Optional, null, Network.GuestType.Shared, true, true, false, false, false, false);
10351035

10361036
defaultSharedSGNetworkOffering.setState(NetworkOffering.State.Enabled);

server/src/main/java/com/cloud/vm/UserVmManagerImpl.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4044,9 +4044,6 @@ private UserVm createVirtualMachine(DataCenter zone, ServiceOffering serviceOffe
40444044
volumesSize += verifyAndGetDiskSize(diskOffering, diskSize);
40454045
}
40464046
UserVm vm = getCheckedUserVmResource(zone, hostName, displayName, owner, diskOfferingId, diskSize, networkList, securityGroupIdList, group, httpmethod, userData, userDataId, userDataDetails, sshKeyPairs, caller, requestedIps, defaultIps, isDisplayVm, keyboard, affinityGroupIdList, customParameters, customId, dhcpOptionMap, datadiskTemplateToDiskOfferringMap, userVmOVFPropertiesMap, dynamicScalingEnabled, vmType, template, hypervisorType, accountId, offering, isIso, rootDiskOfferingId, volumesSize);
4047-
if (! VirtualMachineManager.ResourceCountRunningVMsonly.value()) {
4048-
resourceLimitCheck(owner, isDisplayVm, offering.getCpu().longValue(), offering.getRamSize().longValue());
4049-
}
40504047

40514048
_securityGroupMgr.addInstanceToGroups(vm.getId(), securityGroupIdList);
40524049

0 commit comments

Comments
 (0)