Revert "CLOUDSTACK-5900: Failed to delete template/ISO that failed to download."

This reverts commit 0e4b93c6c1.
This commit is contained in:
Sanjay Tripathi 2014-01-29 12:56:03 +05:30
parent 661a19bde3
commit f5ba6a34f1

View File

@ -1740,16 +1740,16 @@ public class NfsSecondaryStorageResource extends ServerResourceBase implements S
parent += File.separator;
}
String absoluteTemplatePath = parent + relativeTemplatePath;
File tmpltDir = new File(absoluteTemplatePath);
File tmpltParent = new File(absoluteTemplatePath).getParentFile();
String details = null;
if (!tmpltDir.exists()) {
details = "template parent directory " + tmpltDir.getName() + " doesn't exist";
if (!tmpltParent.exists()) {
details = "template parent directory " + tmpltParent.getName() + " doesn't exist";
s_logger.debug(details);
return new Answer(cmd, true, details);
}
File[] tmpltFiles = tmpltDir.listFiles();
File[] tmpltFiles = tmpltParent.listFiles();
if (tmpltFiles == null || tmpltFiles.length == 0) {
details = "No files under template parent directory " + tmpltDir.getName();
details = "No files under template parent directory " + tmpltParent.getName();
s_logger.debug(details);
} else {
boolean found = false;
@ -1776,12 +1776,12 @@ public class NfsSecondaryStorageResource extends ServerResourceBase implements S
}
if (!found) {
details = "Can not find template.properties under " + tmpltDir.getName();
details = "Can not find template.properties under " + tmpltParent.getName();
s_logger.debug(details);
}
}
if (!tmpltDir.delete()) {
details = "Unable to delete directory " + tmpltDir.getName() + " under Template path "
if (!tmpltParent.delete()) {
details = "Unable to delete directory " + tmpltParent.getName() + " under Template path "
+ relativeTemplatePath;
s_logger.debug(details);
return new Answer(cmd, false, details);