refactor: remove use of term entry-point from extensions code base (a…

…pache#11488) Addresses apache#11483 Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
sandeeplocharla · Sep 30, 2025 · Oct 1, 2025 · Oct 2, 2025 · Oct 3, 2025 · Oct 4, 2025
commit 4a9b4a454ed5010d4cc68f017c18de161a415529
diff --git a/...ions/src/main/java/org/apache/cloudstack/framework/extensions/api/DeleteExtensionCmd.java b/...ions/src/main/java/org/apache/cloudstack/framework/extensions/api/DeleteExtensionCmd.java
@@ -56,7 +56,8 @@ public class DeleteExtensionCmd extends BaseCmd {
     private Long id;
 
     @Parameter(name = ApiConstants.CLEANUP, type = CommandType.BOOLEAN,
-            entityType = ExtensionResponse.class, description = "Whether cleanup entry-point files for the extension")
+            entityType = ExtensionResponse.class,
+            description = "Whether to cleanup files for the extension. If true, the extension files will be deleted from all the management servers.")
     private Boolean cleanup;
 
     /////////////////////////////////////////////////////

diff --git a/...c/main/java/org/apache/cloudstack/framework/extensions/manager/ExtensionsManagerImpl.java b/...c/main/java/org/apache/cloudstack/framework/extensions/manager/ExtensionsManagerImpl.java
@@ -152,7 +152,7 @@ public class ExtensionsManagerImpl extends ManagerBase implements ExtensionsMana
 
     ConfigKey<Integer> PathStateCheckInterval = new ConfigKey<>("Advanced", Integer.class,
             "extension.path.state.check.interval", "300",
-            "Interval (in seconds) for checking entry-point state of extensions",
+            "Interval (in seconds) for checking state of extensions path",
             false, ConfigKey.Scope.Global);
 
     @Inject
@@ -264,19 +264,19 @@ protected Pair<Boolean, String> getResultFromAnswersString(String answersStr, Ex
 
     protected boolean prepareExtensionPathOnMSPeer(Extension extension, ManagementServerHostVO msHost) {
         final String msPeer = Long.toString(msHost.getMsid());
-        logger.debug("Sending prepare extension entry-point for {} command to MS: {}", extension, msPeer);
+        logger.debug("Sending prepare extension path for {} command to MS: {}", extension, msPeer);
         final Command[] commands = new Command[1];
         commands[0] = new PrepareExtensionPathCommand(ManagementServerNode.getManagementServerId(), extension);
         String answersStr = clusterManager.execute(msPeer, 0L, GsonHelper.getGson().toJson(commands), true);
-        return getResultFromAnswersString(answersStr, extension, msHost, "prepare entry-point").first();
+        return getResultFromAnswersString(answersStr, extension, msHost, "prepare path").first();
     }
 
     protected Pair<Boolean, String> prepareExtensionPathOnCurrentServer(String name, boolean userDefined,
                                                                               String relativePath) {
         try {
             externalProvisioner.prepareExtensionPath(name, userDefined, relativePath);
         } catch (CloudRuntimeException e) {
-            logger.error("Failed to prepare entry-point for Extension [name: {}, userDefined: {}, relativePath: {}] on this server",
+            logger.error("Failed to prepare path for Extension [name: {}, userDefined: {}, relativePath: {}] on this server",
                     name, userDefined, relativePath, e);
             return new Pair<>(false, e.getMessage());
         }
@@ -285,38 +285,38 @@ protected Pair<Boolean, String> prepareExtensionPathOnCurrentServer(String name,
 
     protected boolean cleanupExtensionFilesOnMSPeer(Extension extension, ManagementServerHostVO msHost) {
         final String msPeer = Long.toString(msHost.getMsid());
-        logger.debug("Sending cleanup extension entry-point for {} command to MS: {}", extension, msPeer);
+        logger.debug("Sending cleanup extension files for {} command to MS: {}", extension, msPeer);
         final Command[] commands = new Command[1];
         commands[0] = new CleanupExtensionFilesCommand(ManagementServerNode.getManagementServerId(), extension);
         String answersStr = clusterManager.execute(msPeer, 0L, GsonHelper.getGson().toJson(commands), true);
-        return getResultFromAnswersString(answersStr, extension, msHost, "cleanup entry-point").first();
+        return getResultFromAnswersString(answersStr, extension, msHost, "cleanup files").first();
     }
 
     protected Pair<Boolean, String> cleanupExtensionFilesOnCurrentServer(String name, String relativePath) {
         try {
             externalProvisioner.cleanupExtensionPath(name, relativePath);
             externalProvisioner.cleanupExtensionData(name, 0, true);
         } catch (CloudRuntimeException e) {
-            logger.error("Failed to cleanup entry-point files for Extension [name: {}, relativePath: {}] on this server",
+            logger.error("Failed to cleanup files for Extension [name: {}, relativePath: {}] on this server",
                     name, relativePath, e);
             return new Pair<>(false, e.getMessage());
         }
         return new Pair<>(true, null);
     }
 
     protected void cleanupExtensionFilesAcrossServers(Extension extension) {
-        boolean cleanup = true;
+        boolean cleanedUp = true;
         List<ManagementServerHostVO> msHosts = managementServerHostDao.listBy(ManagementServerHost.State.Up);
         for (ManagementServerHostVO msHost : msHosts) {
             if (msHost.getMsid() == ManagementServerNode.getManagementServerId()) {
-                cleanup = cleanup && cleanupExtensionFilesOnCurrentServer(extension.getName(),
+                cleanedUp = cleanedUp && cleanupExtensionFilesOnCurrentServer(extension.getName(),
                         extension.getRelativePath()).first();
                 continue;
             }
-            cleanup = cleanup && cleanupExtensionFilesOnMSPeer(extension, msHost);
+            cleanedUp = cleanedUp && cleanupExtensionFilesOnMSPeer(extension, msHost);
         }
-        if (!cleanup) {
-            throw new CloudRuntimeException("Extension is deleted but its entry-point files are not cleaned up across servers");
+        if (!cleanedUp) {
+            throw new CloudRuntimeException("Extension is deleted but its files are not cleaned up across servers");
         }
     }
 
@@ -327,7 +327,7 @@ protected Pair<Boolean, String> getChecksumForExtensionPathOnMSPeer(Extension ex
         cmds[0] = new GetExtensionPathChecksumCommand(ManagementServerNode.getManagementServerId(),
                 extension);
         String answersStr = clusterManager.execute(msPeer, 0L, GsonHelper.getGson().toJson(cmds), true);
-        return getResultFromAnswersString(answersStr, extension, msHost, "prepare entry-point");
+        return getResultFromAnswersString(answersStr, extension, msHost, "get path checksum");
     }
 
     protected List<ExtensionCustomAction.Parameter> getParametersListFromMap(String actionName, Map parametersMap) {
@@ -549,7 +549,7 @@ protected void checkExtensionPathState(Extension extension, List<ManagementServe
             final Pair<Boolean, String> msPeerChecksumResult = getChecksumForExtensionPathOnMSPeer(extension,
                     msHost);
             if (!msPeerChecksumResult.first() || !checksum.equals(msPeerChecksumResult.second())) {
-                logger.error("Entry-point checksum for {} is different [msid: {}, checksum: {}] and [msid: {}, checksum: {}]",
+                logger.error("Path checksum for {} is different [msid: {}, checksum: {}] and [msid: {}, checksum: {}]",
                         extension, ManagementServerNode.getManagementServerId(), checksum, msHost.getMsid(),
                         (msPeerChecksumResult.first() ? msPeerChecksumResult.second() : "unknown"));
                 updateExtensionPathReady(extension, false);
@@ -630,7 +630,7 @@ public Extension createExtension(CreateExtensionCmd cmd) {
                 !prepareExtensionPathAcrossServers(extensionVO)) {
             disableExtension(extensionVO.getId());
             throw new CloudRuntimeException(String.format(
-                    "Failed to enable extension: %s as it entry-point is not ready",
+                    "Failed to enable extension: %s as its path is not ready",
                     extensionVO.getName()));
         }
         return extensionVO;
@@ -736,7 +736,7 @@ public Extension updateExtension(UpdateExtensionCmd cmd) {
                 !prepareExtensionPathAcrossServers(result)) {
                 disableExtension(result.getId());
                 throw new CloudRuntimeException(String.format(
-                        "Failed to enable extension: %s as it entry-point is not ready",
+                        "Failed to enable extension: %s as it path is not ready",
                         extensionVO.getName()));
             }
             updateAllExtensionHosts(extensionVO, null, false);

diff --git a/...org/apache/cloudstack/hypervisor/external/provisioner/ExternalPathPayloadProvisioner.java b/...org/apache/cloudstack/hypervisor/external/provisioner/ExternalPathPayloadProvisioner.java
@@ -631,18 +631,18 @@ public void cleanupExtensionPath(String extensionName, String extensionRelativeP
             }
             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 " +
+                        String.format("Failed to cleanup path: %s for extension: %s as it either " +
                                         "does not exist or is not a regular file/directory",
                                 extensionName, extensionRelativePath));
             }
             if (!FileUtil.deleteRecursively(filePath)) {
                 throw new CloudRuntimeException(
-                        String.format("Failed to delete extension entry-point: %s for extension: %s",
+                        String.format("Failed to delete path: %s for extension: %s",
                                 extensionName, filePath));
             }
         } catch (IOException e) {
             throw new CloudRuntimeException(
-                    String.format("Failed to cleanup extension entry-point: %s for extension: %s due to: %s",
+                    String.format("Failed to cleanup path: %s for extension: %s due to: %s",
                             extensionName, normalizedPath, e.getMessage()), e);
         }
     }

diff --git a/ui/src/views/extension/CreateExtension.vue b/ui/src/views/extension/CreateExtension.vue
@@ -46,15 +46,15 @@
         <template #label>
           <tooltip-label :title="$t('label.path')" :tooltip="apiParams.path.description"/>
         </template>
-        <div class="entry-point-input-container">
-          <span v-if="!!safeName" :title="extenstionBasePath" class="entry-point-input-base-path">
+        <div class="path-input-container">
+          <span v-if="!!safeName" :title="extenstionBasePath" class="path-input-base">
             {{ extenstionBasePath }}
           </span>
           <a-input
             v-model:value="form.path"
             :placeholder="apiParams.path.description"
             @input="markPathModified"
-            class="entry-point-input-relative-path"
+            class="path-input-relative"
           />
         </div>
       </a-form-item>
@@ -236,20 +236,20 @@ export default {
   }
 }
 
-.entry-point-input-container {
+.path-input-container {
   display: flex;
   align-items: center;
   gap: 8px;
 }
 
-.entry-point-input-base-path {
+.path-input-base {
   max-width: 70%;
   overflow: hidden;
   white-space: nowrap;
   text-overflow: ellipsis;
 }
 
-.entry-point-input-relative-path {
+.path-input-relative {
   flex: 1 1 0%;
   min-width: 0;
 }