Skip to content

Commit a1a23d2

Browse files
Sateesh ChodapuneediDaanHoogland
authored andcommitted
CLOUDSTACK-6996 Adding cluster to legacy zone failed
Lookup zone_id field in legacy_zones table to search the table for legacy zone. Signed-off-by: Sateesh Chodapuneedi <sateesh@apache.org> (cherry picked from commit 75aa243)
1 parent 8f9458e commit a1a23d2

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/dao/LegacyZoneDao.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ public interface LegacyZoneDao extends GenericDao<LegacyZoneVO, Long> {
2727
* @param zoneId - id of zone
2828
* @return Object of legacy zone (LegacyZoneVO) identified by zoneId
2929
*/
30-
LegacyZoneVO findByZoneId(String zoneId);
30+
LegacyZoneVO findByZoneId(Long zoneId);
3131

3232
/**
3333
* Lists all legacy CloudStack zones

plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/dao/LegacyZoneDaoImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public LegacyZoneDaoImpl() {
5252
}
5353

5454
@Override
55-
public LegacyZoneVO findByZoneId(String zoneId) {
55+
public LegacyZoneVO findByZoneId(Long zoneId) {
5656
SearchCriteria<LegacyZoneVO> sc = zoneSearch.create();
5757
sc.setParameters("zoneId", zoneId);
5858
return findOneBy(sc);

plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1186,7 +1186,7 @@ private void validateZoneWithResources(Long zoneId, String errStr) throws Resour
11861186
@Override
11871187
public boolean isLegacyZone(long dcId) {
11881188
boolean isLegacyZone = false;
1189-
LegacyZoneVO legacyZoneVo = _legacyZoneDao.findById(dcId);
1189+
LegacyZoneVO legacyZoneVo = _legacyZoneDao.findByZoneId(dcId);
11901190
if (legacyZoneVo != null) {
11911191
isLegacyZone = true;
11921192
}

0 commit comments

Comments
 (0)