diff --git a/openGauss-datakit/visualtool-api/src/main/java/org/opengauss/admin/web/controller/ops/HostController.java b/openGauss-datakit/visualtool-api/src/main/java/org/opengauss/admin/web/controller/ops/HostController.java index d7a907a415b1eb1ed4e3e64b10b1baeec06d61dd..1ccba7cdce848e54163aed7d6d36f0e1193e4713 100644 --- a/openGauss-datakit/visualtool-api/src/main/java/org/opengauss/admin/web/controller/ops/HostController.java +++ b/openGauss-datakit/visualtool-api/src/main/java/org/opengauss/admin/web/controller/ops/HostController.java @@ -175,8 +175,8 @@ public class HostController extends BaseController { @GetMapping("/ping/{hostId}") public AjaxResult ping(@PathVariable String hostId, @RequestParam(value = "rootPassword", required = false) String rootPassword) { - boolean ping = hostService.ping(hostId, rootPassword); - return ping ? AjaxResult.success() : AjaxResult.error("Connectivity test failed"); + hostService.ping(hostId, rootPassword); + return AjaxResult.success(); } @DeleteMapping("/{hostId}") diff --git a/openGauss-datakit/visualtool-service/src/main/java/org/opengauss/admin/system/service/ops/impl/HostServiceImpl.java b/openGauss-datakit/visualtool-service/src/main/java/org/opengauss/admin/system/service/ops/impl/HostServiceImpl.java index d065b57dea804dbfd7a93e30b6c70ec8dd2f790d..dff272802c32dc0e893c54f05f75fe6d03e83319 100644 --- a/openGauss-datakit/visualtool-service/src/main/java/org/opengauss/admin/system/service/ops/impl/HostServiceImpl.java +++ b/openGauss-datakit/visualtool-service/src/main/java/org/opengauss/admin/system/service/ops/impl/HostServiceImpl.java @@ -438,6 +438,9 @@ public class HostServiceImpl extends ServiceImpl i OpsHostUserEntity userEntity = opsHostUserEntities.stream().findAny().get(); SshLogin pingLogin = new SshLogin(hostEntity.getPublicIp(), hostEntity.getPort(), userEntity.getUsername(), encryptionUtils.decrypt(userEntity.getPassword())); + if (!jschExecutorService.checkOsUserExist(pingLogin)) { + throw new OpsException("check os user:" + userEntity.getUsername() + " is not exist."); + } return jschExecutorService.checkOsUserExist(pingLogin); }