minor fixes

Signed-off-by: Abhishek Kumar <[email protected]>

Author: shwstppr

framework/extensions/src/main/java/org/apache/cloudstack/framework/extensions/manager/ExtensionsManagerImpl.java

@@ -556,6 +556,9 @@ public Extension createExtension(CreateExtensionCmd cmd) {
         ExtensionVO extensionVO = Transaction.execute((TransactionCallbackWithException<ExtensionVO, CloudRuntimeException>) status -> {
             ExtensionVO extension = new ExtensionVO(name, description, type,
                     entryPointFinal, stateFinal);
+            if (!Extension.State.Enabled.equals(stateFinal)) {
+                extension.setEntryPointReady(false);
+            }
             extension = extensionDao.persist(extension);
 
             Map<String, String> details = cmd.getDetails();
@@ -658,10 +661,6 @@ public Extension updateExtension(UpdateExtensionCmd cmd) {
         if (description != null && !description.equals(extensionVO.getDescription())) {
             extensionVO.setDescription(description);
             updateNeeded = true;
-            if (!extensionDao.update(id, extensionVO)) {
-                throw new CloudRuntimeException(String.format("Failed to updated the extension: %s",
-                        extensionVO.getName()));
-            }
         }
         if (orchestratorRequiresPrepareVm != null && !Extension.Type.Orchestrator.equals(extensionVO.getType())) {
             throw new InvalidParameterValueException(String.format("%s is applicable only with %s type",

plugins/hypervisors/external/src/main/java/org/apache/cloudstack/hypervisor/external/provisioner/ExternalEntryPointPayloadProvisioner.java

@@ -548,6 +548,9 @@ public void cleanupExtensionEntryPoint(String extensionName, String extensionRel
             Path filePath = rootPath
                     .resolve(normalizedPath.startsWith(extensionDirName) ? extensionDirName : normalizedPath)
                     .normalize();
+            if (!Files.exists(filePath)) {
+                return;
+            }
             if (!Files.isDirectory(filePath) && !Files.isRegularFile(filePath)) {
                 throw new CloudRuntimeException(
                         String.format("Failed to cleanup extension entry-point: %s for extension: %s as it either " +

ui/src/config/section/extension.js

@@ -44,7 +44,7 @@ export default {
       }, 'created']
     return fields
   },
-  details: ['name', 'id', 'type', 'details', 'entrypoint', 'entrypointready', 'isuserdefined', 'orchestratorrequirespreparevm', 'created'],
+  details: ['name', 'description', 'id', 'type', 'details', 'entrypoint', 'entrypointready', 'isuserdefined', 'orchestratorrequirespreparevm', 'created'],
   filters: ['orchestrator'],
   tabs: [{
     name: 'details',