Skip to content

Commit f6ec12f

Browse files
Srivastava, PiyushSrivastava, Piyush
authored andcommitted
Delete export policy NFS review comments
1 parent 2a27851 commit f6ec12f

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/service/UnifiedNASStrategy.java

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -170,16 +170,14 @@ public void deleteAccessGroup(AccessGroup accessGroup) {
170170

171171
try {
172172
nasFeignClient.deleteExportPolicyById(authHeader,exportPolicyId);
173-
s_logger.info("Export policy deleted successfully: {}", exportPolicyName);
174-
175173
s_logger.info("deleteAccessGroup: Successfully deleted export policy '{}'", exportPolicyName);
176174
} catch (Exception e) {
177-
s_logger.error("deleteAccessGroup: Failed to delete iGroup. Exception: {}", e.getMessage(), e);
178-
throw new CloudRuntimeException("Failed to delete iGroup: " + e.getMessage(), e);
175+
s_logger.error("deleteAccessGroup: Failed to delete export policy. Exception: {}", e.getMessage(), e);
176+
throw new CloudRuntimeException("Failed to delete export policy: " + e.getMessage(), e);
179177
}
180178
} catch (Exception e) {
181-
s_logger.error("deleteAccessGroup: Failed to delete iGroup. Exception: {}", e.getMessage(), e);
182-
throw new CloudRuntimeException("Failed to delete iGroup: " + e.getMessage(), e);
179+
s_logger.error("deleteAccessGroup: Failed to delete export policy. Exception: {}", e.getMessage(), e);
180+
throw new CloudRuntimeException("Failed to delete export policy: " + e.getMessage(), e);
183181
}
184182
}
185183

0 commit comments

Comments
 (0)