<fix>[multi]: fix review findings: dryRun completion, initializeHostAttachedPSMountPath NPE

Resolves: ZSTAC-69300

Change-Id: I1b39a9e7b76751e8a4ef4cc53e9ac2028386e334

Author: AlanJager

compute/src/main/java/org/zstack/compute/allocator/HostAllocatorChain.java

@@ -147,7 +147,11 @@ private void runFlow(AbstractHostAllocatorFlow flow) {
         } catch (Throwable t) {
             logger.warn("unhandled throwable", t);
             String errMsg = t != null ? t.toString() : "unknown error";
-            if (completion != null) {
+            if (isDryRun) {
+                if (dryRunCompletion != null) {
+                    dryRunCompletion.fail(inerr(ORG_ZSTACK_COMPUTE_ALLOCATOR_10019, errMsg));
+                }
+            } else if (completion != null) {
                 completion.fail(inerr(ORG_ZSTACK_COMPUTE_ALLOCATOR_10019, errMsg));
             }
         }

plugin/localstorage/src/main/java/org/zstack/storage/primary/local/LocalStorageKvmBackend.java

@@ -3819,7 +3819,8 @@ public void success(AgentResponse rsp) {
 
             @Override
             public void fail(ErrorCode errorCode) {
-                completion.fail(operr(ORG_ZSTACK_STORAGE_PRIMARY_LOCAL_10082, "cannot create flag file [%s] on host [%s], because: %s", makeInitializedFilePath(), hostUuid, errorCode.getCause().getDetails()));
+                String causeDetails = errorCode.getCause() != null ? errorCode.getCause().getDetails() : errorCode.getDetails();
+                completion.fail(operr(ORG_ZSTACK_STORAGE_PRIMARY_LOCAL_10082, "cannot create flag file [%s] on host [%s], because: %s", makeInitializedFilePath(), hostUuid, causeDetails));
             }
         });
     }