Skip to content

Commit 2ff0d59

Browse files
committed
CLOUDSTACK-8816: fixed missing resource uuid in delete network cmd
*events before* | management-server.AsyncJobEvent.submit.None.* | cloudstack-events | 7 | {"cmdInfo":"{\"id\":\"edf0a16b-54cd-442e-b644-4af933f34229\",\"response\":\"json\",\"ctxDetails\":\"{\\\"interface com.cloud.network.Network\\\":\\\"edf0a16b-54cd-442e-b644-4af933f34229\\\"}\",\"cmdEventType\":\"NETWORK.DELETE\",\"ctxUserId\":\"2\",\"httpmethod\":\"GET\",\"_\":\"1444805881664\",\"uuid\":\"edf0a16b-54cd-442e-b644-4af933f34229\",\"ctxAccountId\":\"2\",\"ctxStartEventId\":\"1378\"}","instanceType":"None","jobId":"f7cbf481-49d0-423b-8661-5d3d678f4b96","status":"IN_PROGRESS","processStatus":"0","commandEventType":"NETWORK.DELETE","resultCode":"0","command":"org.apache.cloudstack.api.command.user.network.DeleteNetworkCmd","account":"bd73dc2e-35c0-11e5-b094-d4ae52cb9af0","user":"bd7ea748-35c0-11e5-b094-d4ae52cb9af0"} | 736 | string | True | | management-server.AsyncJobEvent.complete.None.* | cloudstack-events | 6 | {"cmdInfo":"{\"id\":\"edf0a16b-54cd-442e-b644-4af933f34229\",\"response\":\"json\",\"ctxDetails\":\"{\\\"interface com.cloud.network.Network\\\":\\\"edf0a16b-54cd-442e-b644-4af933f34229\\\"}\",\"cmdEventType\":\"NETWORK.DELETE\",\"ctxUserId\":\"2\",\"httpmethod\":\"GET\",\"_\":\"1444805881664\",\"uuid\":\"edf0a16b-54cd-442e-b644-4af933f34229\",\"ctxAccountId\":\"2\",\"ctxStartEventId\":\"1378\"}","instanceType":"None","jobId":"f7cbf481-49d0-423b-8661-5d3d678f4b96","status":"FAILED","processStatus":"0","commandEventType":"NETWORK.DELETE","resultCode":"530","command":"org.apache.cloudstack.api.command.user.network.DeleteNetworkCmd","jobResult":"org.apache.cloudstack.api.response.ExceptionResponse/null/{\"uuidList\":[],\"errorcode\":530,\"errortext\":\"Failed to delete network\"}","account":"bd73dc2e-35c0-11e5-b094-d4ae52cb9af0","user":"bd7ea748-35c0-11e5-b094-d4ae52cb9af0"} | 884 | string | True | *events after* | management-server.AsyncJobEvent.submit.Network.5eccaece-a789-4b93-99c2-8b731ab6e328 | cloudstack-events | 1 | {"cmdInfo":"{\"id\":\"5eccaece-a789-4b93-99c2-8b731ab6e328\",\"response\":\"json\",\"ctxDetails\":\"{\\\"interface com.cloud.network.Network\\\":\\\"5eccaece-a789-4b93-99c2-8b731ab6e328\\\"}\",\"cmdEventType\":\"NETWORK.DELETE\",\"ctxUserId\":\"2\",\"httpmethod\":\"GET\",\"_\":\"1444814151636\",\"uuid\":\"5eccaece-a789-4b93-99c2-8b731ab6e328\",\"ctxAccountId\":\"2\",\"ctxStartEventId\":\"1424\"}","instanceType":"Network","instanceUuid":"5eccaece-a789-4b93-99c2-8b731ab6e328","jobId":"d2cd4b27-acbd-4e56-867f-fe67ebde8261","status":"IN_PROGRESS","processStatus":"0","commandEventType":"NETWORK.DELETE","resultCode":"0","command":"org.apache.cloudstack.api.command.user.network.DeleteNetworkCmd","account":"bd73dc2e-35c0-11e5-b094-d4ae52cb9af0","user":"bd7ea748-35c0-11e5-b094-d4ae52cb9af0"} | 793 | string | False | | management-server.AsyncJobEvent.complete.Network.5eccaece-a789-4b93-99c2-8b731ab6e328 | cloudstack-events | 0 | {"cmdInfo":"{\"id\":\"5eccaece-a789-4b93-99c2-8b731ab6e328\",\"response\":\"json\",\"ctxDetails\":\"{\\\"interface com.cloud.network.Network\\\":\\\"5eccaece-a789-4b93-99c2-8b731ab6e328\\\"}\",\"cmdEventType\":\"NETWORK.DELETE\",\"ctxUserId\":\"2\",\"httpmethod\":\"GET\",\"_\":\"1444814151636\",\"uuid\":\"5eccaece-a789-4b93-99c2-8b731ab6e328\",\"ctxAccountId\":\"2\",\"ctxStartEventId\":\"1424\"}","instanceType":"Network","instanceUuid":"5eccaece-a789-4b93-99c2-8b731ab6e328","jobId":"d2cd4b27-acbd-4e56-867f-fe67ebde8261","status":"SUCCEEDED","processStatus":"0","commandEventType":"NETWORK.DELETE","resultCode":"0","command":"org.apache.cloudstack.api.command.user.network.DeleteNetworkCmd","jobResult":"org.apache.cloudstack.api.response.SuccessResponse/null/{\"success\":true}","account":"bd73dc2e-35c0-11e5-b094-d4ae52cb9af0","user":"bd7ea748-35c0-11e5-b094-d4ae52cb9af0"} | 880 | string | False |
1 parent e2b02d9 commit 2ff0d59

File tree

3 files changed

+18
-2
lines changed

3 files changed

+18
-2
lines changed

api/src/org/apache/cloudstack/api/ApiCommandJobType.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,5 +53,6 @@ public enum ApiCommandJobType {
5353
IAMPolicy,
5454
IAMGroup,
5555
GuestOs,
56-
GuestOsMapping
56+
GuestOsMapping,
57+
Network
5758
}

api/src/org/apache/cloudstack/api/command/user/network/DeleteNetworkCmd.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
// under the License.
1717
package org.apache.cloudstack.api.command.user.network;
1818

19+
import org.apache.cloudstack.api.ApiCommandJobType;
1920
import org.apache.log4j.Logger;
2021

2122
import org.apache.cloudstack.acl.SecurityChecker.AccessType;
@@ -105,6 +106,15 @@ public String getEventDescription() {
105106
return "Deleting network: " + id;
106107
}
107108

109+
@Override
110+
public Long getInstanceId() {
111+
return getId();
112+
}
113+
114+
@Override
115+
public ApiCommandJobType getInstanceType() {
116+
return ApiCommandJobType.Network;
117+
}
108118
@Override
109119
public long getEntityOwnerId() {
110120
Network network = _networkService.getNetwork(id);

server/src/com/cloud/api/ApiDBUtils.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1217,7 +1217,7 @@ public static PhysicalNetworkTrafficTypeVO findPhysicalNetworkTrafficTypeById(lo
12171217
}
12181218

12191219
public static NetworkVO findNetworkById(long id) {
1220-
return s_networkDao.findById(id);
1220+
return s_networkDao.findByIdIncludingRemoved(id);
12211221
}
12221222

12231223
public static Map<Service, Map<Capability, String>> getNetworkCapabilities(long networkId, long zoneId) {
@@ -1599,6 +1599,11 @@ public static String findJobInstanceUuid(AsyncJob job) {
15991599
if (group != null) {
16001600
jobInstanceId = group.getUuid();
16011601
}
1602+
} else if (jobInstanceType == ApiCommandJobType.Network) {
1603+
NetworkVO networkVO = ApiDBUtils.findNetworkById(job.getInstanceId());
1604+
if(networkVO != null) {
1605+
jobInstanceId = networkVO.getUuid();
1606+
}
16021607
} else if (jobInstanceType != ApiCommandJobType.None) {
16031608
// TODO : when we hit here, we need to add instanceType -> UUID
16041609
// entity table mapping

0 commit comments

Comments
 (0)