Skip to content

Commit fdc34ed

Browse files
weizhouapacheweizhouapache
committed
utils: add UuidUtils.nameUUIDFromBytes
1 parent be22bfe commit fdc34ed

File tree

37 files changed

+100
-70
lines changed

37 files changed

+100
-70
lines changed

‎agent/src/main/java/com/cloud/agent/resource/DummyResource.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
import java.util.HashMap;
2121
import java.util.List;
2222
import java.util.Map;
23-
import java.util.UUID;
2423

2524
import com.cloud.agent.IAgentControl;
2625
import com.cloud.agent.api.Answer;
@@ -40,6 +39,7 @@
4039
import com.cloud.storage.Storage;
4140
import com.cloud.storage.Storage.StoragePoolType;
4241
import com.cloud.utils.StringUtils;
42+
import com.cloud.utils.UuidUtils;
4343

4444
public class DummyResource implements ServerResource {
4545
String _name;
@@ -133,7 +133,7 @@ protected StoragePoolInfo initializeLocalStorage() {
133133
String hostIp = getConfiguredProperty("private.ip.address", "127.0.0.1");
134134
String localStoragePath = getConfiguredProperty("local.storage.path", "/mnt");
135135
String lh = hostIp + localStoragePath;
136-
String uuid = UUID.nameUUIDFromBytes(lh.getBytes(StringUtils.getPreferredCharset())).toString();
136+
String uuid = UuidUtils.nameUUIDFromBytes(lh.getBytes(StringUtils.getPreferredCharset())).toString();
137137

138138
String capacity = getConfiguredProperty("local.storage.capacity", "1000000000");
139139
String available = getConfiguredProperty("local.storage.avail", "10000000");

‎core/src/main/java/com/cloud/agent/api/DeleteStoragePoolCommand.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,10 @@
2121

2222
import java.io.File;
2323
import java.util.Map;
24-
import java.util.UUID;
2524

2625
import com.cloud.agent.api.to.StorageFilerTO;
2726
import com.cloud.storage.StoragePool;
27+
import com.cloud.utils.UuidUtils;
2828

2929
public class DeleteStoragePoolCommand extends Command {
3030
public static final String DATASTORE_NAME = "datastoreName";
@@ -49,7 +49,7 @@ public DeleteStoragePoolCommand(StoragePool pool, String localPath) {
4949
}
5050

5151
public DeleteStoragePoolCommand(StoragePool pool) {
52-
this(pool, LOCAL_PATH_PREFIX + File.separator + UUID.nameUUIDFromBytes((pool.getHostAddress() + pool.getPath()).getBytes()));
52+
this(pool, LOCAL_PATH_PREFIX + File.separator + UuidUtils.nameUUIDFromBytes((pool.getHostAddress() + pool.getPath()).getBytes()));
5353
}
5454

5555
public void setPool(StoragePool pool) {

‎core/src/main/java/com/cloud/agent/api/ModifyStoragePoolCommand.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,10 @@
2121

2222
import java.io.File;
2323
import java.util.Map;
24-
import java.util.UUID;
2524

2625
import com.cloud.agent.api.to.StorageFilerTO;
2726
import com.cloud.storage.StoragePool;
27+
import com.cloud.utils.UuidUtils;
2828

2929
public class ModifyStoragePoolCommand extends Command {
3030
public static final String LOCAL_PATH_PREFIX = "/mnt/";
@@ -47,11 +47,11 @@ public ModifyStoragePoolCommand(boolean add, StoragePool pool, String localPath,
4747
}
4848

4949
public ModifyStoragePoolCommand(boolean add, StoragePool pool, Map<String, String> details) {
50-
this(add, pool, LOCAL_PATH_PREFIX + File.separator + UUID.nameUUIDFromBytes((pool.getHostAddress() + pool.getPath()).getBytes()), details);
50+
this(add, pool, LOCAL_PATH_PREFIX + File.separator + UuidUtils.nameUUIDFromBytes((pool.getHostAddress() + pool.getPath()).getBytes()), details);
5151
}
5252

5353
public ModifyStoragePoolCommand(boolean add, StoragePool pool) {
54-
this(add, pool, LOCAL_PATH_PREFIX + File.separator + UUID.nameUUIDFromBytes((pool.getHostAddress() + pool.getPath()).getBytes()));
54+
this(add, pool, LOCAL_PATH_PREFIX + File.separator + UuidUtils.nameUUIDFromBytes((pool.getHostAddress() + pool.getPath()).getBytes()));
5555
}
5656

5757
public boolean getAdd() {

‎engine/schema/src/main/java/com/cloud/storage/VMTemplateVO.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434

3535
import com.cloud.cpu.CPU;
3636
import com.cloud.user.UserData;
37+
import com.cloud.utils.UuidUtils;
3738
import org.apache.cloudstack.util.CPUArchConverter;
3839
import org.apache.cloudstack.util.HypervisorTypeConverter;
3940
import org.apache.cloudstack.utils.reflectiontostringbuilderutils.ReflectionToStringBuilderUtils;
@@ -340,7 +341,7 @@ private static String generateUniqueName(long id, long userId, String displayNam
340341
name.append("-");
341342
name.append(userId);
342343
name.append("-");
343-
name.append(UUID.nameUUIDFromBytes((displayName + System.currentTimeMillis()).getBytes()).toString());
344+
name.append(UuidUtils.nameUUIDFromBytes((displayName + System.currentTimeMillis()).getBytes()).toString());
344345
return name.toString();
345346
}
346347

‎engine/storage/integration-test/src/test/java/org/apache/cloudstack/storage/test/SnapshotTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,6 +100,7 @@
100100
import com.cloud.storage.dao.SnapshotDao;
101101
import com.cloud.storage.dao.VMTemplateDao;
102102
import com.cloud.storage.dao.VolumeDao;
103+
import com.cloud.utils.UuidUtils;
103104
import com.cloud.utils.component.ComponentContext;
104105

105106
import junit.framework.Assert;
@@ -287,7 +288,7 @@ protected void injectMockito() {
287288

288289
public DataStore createPrimaryDataStore() {
289290
try {
290-
String uuid = UUID.nameUUIDFromBytes(this.getPrimaryStorageUrl().getBytes()).toString();
291+
String uuid = UuidUtils.nameUUIDFromBytes(this.getPrimaryStorageUrl().getBytes()).toString();
291292
List<StoragePoolVO> pools = primaryDataStoreDao.findPoolByName(this.primaryName);
292293
if (pools.size() > 0) {
293294
return this.dataStoreMgr.getPrimaryDataStore(pools.get(0).getId());

‎engine/storage/integration-test/src/test/java/org/apache/cloudstack/storage/test/VolumeServiceTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,7 @@
8282
import com.cloud.storage.VolumeVO;
8383
import com.cloud.storage.dao.VMTemplateDao;
8484
import com.cloud.storage.dao.VolumeDao;
85+
import com.cloud.utils.UuidUtils;
8586
import com.cloud.utils.component.ComponentContext;
8687

8788
@ContextConfiguration(locations = {"classpath:/storageContext.xml"})
@@ -281,7 +282,7 @@ public void testCreatePrimaryStorage() {
281282
params.put("name", this.primaryName);
282283
params.put("port", "1");
283284
params.put("roles", DataStoreRole.Primary.toString());
284-
params.put("uuid", UUID.nameUUIDFromBytes(this.getPrimaryStorageUrl().getBytes()).toString());
285+
params.put("uuid", UuidUtils.nameUUIDFromBytes(this.getPrimaryStorageUrl().getBytes()).toString());
285286
params.put("providerName", String.valueOf(provider.getName()));
286287

287288
DataStoreLifeCycle lifeCycle = provider.getDataStoreLifeCycle();
@@ -324,7 +325,7 @@ public DataStore createPrimaryDataStore() {
324325
params.put("name", this.primaryName);
325326
params.put("port", "1");
326327
params.put("roles", DataStoreRole.Primary.toString());
327-
params.put("uuid", UUID.nameUUIDFromBytes(this.getPrimaryStorageUrl().getBytes()).toString());
328+
params.put("uuid", UuidUtils.nameUUIDFromBytes(this.getPrimaryStorageUrl().getBytes()).toString());
328329
params.put("providerName", String.valueOf(provider.getName()));
329330

330331
DataStoreLifeCycle lifeCycle = provider.getDataStoreLifeCycle();

‎engine/storage/integration-test/src/test/java/org/apache/cloudstack/storage/test/VolumeTestVmware.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,7 @@
8686
import com.cloud.storage.VolumeVO;
8787
import com.cloud.storage.dao.VMTemplateDao;
8888
import com.cloud.storage.dao.VolumeDao;
89+
import com.cloud.utils.UuidUtils;
8990
import com.cloud.utils.component.ComponentContext;
9091

9192
@ContextConfiguration(locations = {"classpath:/storageContext.xml"})
@@ -264,7 +265,7 @@ protected void injectMockito() {
264265

265266
public DataStore createPrimaryDataStore() {
266267
try {
267-
String uuid = UUID.nameUUIDFromBytes(this.getPrimaryStorageUrl().getBytes()).toString();
268+
String uuid = UuidUtils.nameUUIDFromBytes(this.getPrimaryStorageUrl().getBytes()).toString();
268269
List<StoragePoolVO> pools = primaryDataStoreDao.findPoolByName(this.primaryName);
269270
if (pools.size() > 0) {
270271
return this.dataStoreMgr.getPrimaryDataStore(pools.get(0).getId());

‎plugins/event-bus/rabbitmq/src/main/java/org/apache/cloudstack/mom/rabbitmq/RabbitMQEventBus.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232

3333
import javax.naming.ConfigurationException;
3434

35+
import com.cloud.utils.UuidUtils;
3536
import com.cloud.utils.exception.CloudRuntimeException;
3637
import com.rabbitmq.client.BlockedListener;
3738

@@ -254,7 +255,7 @@ public void unsubscribe(UUID subscriberId, EventSubscriber subscriber) throws Ev
254255
logger.debug("unsubscribing '{}'", subscriberId);
255256
try {
256257
String classname = subscriber.getClass().getName();
257-
String queueName = UUID.nameUUIDFromBytes(classname.getBytes()).toString();
258+
String queueName = UuidUtils.nameUUIDFromBytes(classname.getBytes()).toString();
258259
Ternary<String, Channel, EventSubscriber> queueDetails = s_subscribers.get(queueName);
259260
Channel channel = queueDetails.second();
260261
channel.basicCancel(queueName);

‎plugins/hypervisors/baremetal/src/main/java/com/cloud/baremetal/manager/BareMetalDiscoverer.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@
5151
import com.cloud.resource.ResourceStateAdapter;
5252
import com.cloud.resource.ServerResource;
5353
import com.cloud.resource.UnableDeleteHostException;
54+
import com.cloud.utils.UuidUtils;
5455
import com.cloud.utils.exception.CloudRuntimeException;
5556
import com.cloud.utils.script.Script;
5657
import com.cloud.utils.script.Script2;
@@ -121,7 +122,7 @@ public Map<? extends ServerResource, Map<String, String>> find(long dcId, Long p
121122
String hostname = url.getHost();
122123
InetAddress ia = InetAddress.getByName(hostname);
123124
String ipmiIp = ia.getHostAddress();
124-
String guid = UUID.nameUUIDFromBytes(ipmiIp.getBytes()).toString();
125+
String guid = UuidUtils.nameUUIDFromBytes(ipmiIp.getBytes()).toString();
125126

126127
String injectScript = "scripts/util/ipmi.py";
127128
String scriptPath = Script.findScript("", injectScript);

‎plugins/hypervisors/hyperv/src/main/java/com/cloud/hypervisor/hyperv/discoverer/HypervServerDiscoverer.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
import java.util.List;
2525
import java.util.Map;
2626
import java.util.Random;
27-
import java.util.UUID;
2827

2928
import javax.inject.Inject;
3029
import javax.naming.ConfigurationException;
@@ -64,6 +63,7 @@
6463
import com.cloud.resource.ServerResource;
6564
import com.cloud.resource.UnableDeleteHostException;
6665
import com.cloud.storage.StorageLayer;
66+
import com.cloud.utils.UuidUtils;
6767

6868
/**
6969
* Methods to discover and managem a Hyper-V agent. Prepares a
@@ -264,7 +264,7 @@ public final Map<? extends ServerResource, Map<String, String>> find(final long
264264
// pool in the database
265265
// This GUID may change.
266266
if (cluster.getGuid() == null) {
267-
cluster.setGuid(UUID.nameUUIDFromBytes(String.valueOf(clusterId).getBytes(Charset.forName("UTF-8"))).toString());
267+
cluster.setGuid(UuidUtils.nameUUIDFromBytes(String.valueOf(clusterId).getBytes(Charset.forName("UTF-8"))).toString());
268268
_clusterDao.update(clusterId, cluster);
269269
}
270270

@@ -322,15 +322,15 @@ public final Map<? extends ServerResource, Map<String, String>> find(final long
322322
/**
323323
* Encapsulate GUID calculation in public method to allow access to test
324324
* programs. Works by converting a string to a GUID using
325-
* UUID.nameUUIDFromBytes
325+
* UuidUtils.nameUUIDFromBytes
326326
*
327327
* @param uuidSeed
328328
* string to use to generate GUID
329329
*
330330
* @return GUID in form of a string.
331331
*/
332332
public static String calcServerResourceGuid(final String uuidSeed) {
333-
String guid = UUID.nameUUIDFromBytes(uuidSeed.getBytes(Charset.forName("UTF-8"))).toString();
333+
String guid = UuidUtils.nameUUIDFromBytes(uuidSeed.getBytes(Charset.forName("UTF-8"))).toString();
334334
return guid;
335335
}
336336

0 commit comments

Comments
 (0)