From 975131e9135224ee1c7caef3e0ed4ad995b31bad Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E7=BD=97=E9=93=AD=E6=B6=9B?= <45874392+981433814@users.noreply.github.com> Date: Thu, 11 Nov 2021 19:59:10 +0800 Subject: [PATCH] [Improvement#6655] [UI] remove registry monitor (#6789) * [DS-6655][fix] remove zkmonitor - delete zkmonitor ui * [DS-6655][fix] remove zkmonitor - delete zkmonitor ui - Modify the actions.js comma * [DS-6655][fix] remove zkmonitor - remove all backend API Co-authored-by: Kirs --- .../api/controller/MonitorController.java | 17 ----- .../dolphinscheduler/api/enums/Status.java | 1 - .../api/service/MonitorService.java | 8 --- .../api/service/impl/MonitorServiceImpl.java | 17 ----- .../api/controller/MonitorControllerTest.java | 15 ----- .../api/service/MonitorServiceTest.java | 8 --- .../pages/monitor/pages/servers/zookeeper.vue | 67 ------------------- .../src/js/conf/home/router/index.js | 9 --- .../src/js/conf/home/store/monitor/actions.js | 9 --- .../components/secondaryMenu/_source/menu.js | 6 -- 10 files changed, 157 deletions(-) delete mode 100644 dolphinscheduler-ui/src/js/conf/home/pages/monitor/pages/servers/zookeeper.vue diff --git a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/controller/MonitorController.java b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/controller/MonitorController.java index 92a9ffa5ad..9240ac45d8 100644 --- a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/controller/MonitorController.java +++ b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/controller/MonitorController.java @@ -20,7 +20,6 @@ package org.apache.dolphinscheduler.api.controller; import static org.apache.dolphinscheduler.api.enums.Status.LIST_MASTERS_ERROR; import static org.apache.dolphinscheduler.api.enums.Status.LIST_WORKERS_ERROR; import static org.apache.dolphinscheduler.api.enums.Status.QUERY_DATABASE_STATE_ERROR; -import static org.apache.dolphinscheduler.api.enums.Status.QUERY_ZOOKEEPER_STATE_ERROR; import org.apache.dolphinscheduler.api.aspect.AccessLogAnnotation; import org.apache.dolphinscheduler.api.exceptions.ApiException; @@ -102,20 +101,4 @@ public class MonitorController extends BaseController { return returnDataList(result); } - /** - * query zookeeper state - * - * @param loginUser login user - * @return zookeeper information list - */ - @ApiOperation(value = "queryZookeeperState", notes = "QUERY_ZOOKEEPER_STATE_NOTES") - @GetMapping(value = "/zookeepers") - @ResponseStatus(HttpStatus.OK) - @ApiException(QUERY_ZOOKEEPER_STATE_ERROR) - @AccessLogAnnotation(ignoreRequestArgs = "loginUser") - public Result queryZookeeperState(@ApiIgnore @RequestAttribute(value = Constants.SESSION_USER) User loginUser) { - Map result = monitorService.queryZookeeperState(loginUser); - return returnDataList(result); - } - } diff --git a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/enums/Status.java b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/enums/Status.java index 6f8b2d6f70..69514f7bb6 100644 --- a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/enums/Status.java +++ b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/enums/Status.java @@ -286,7 +286,6 @@ public enum Status { * for monitor */ QUERY_DATABASE_STATE_ERROR(70001, "query database state error", "查询数据库状态错误"), - QUERY_ZOOKEEPER_STATE_ERROR(70002, "query zookeeper state error", "查询zookeeper状态错误"), CREATE_ACCESS_TOKEN_ERROR(70010, "create access token error", "创建访问token错误"), GENERATE_TOKEN_ERROR(70011, "generate token error", "生成token错误"), diff --git a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/MonitorService.java b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/MonitorService.java index 0dbdc8045a..d301957955 100644 --- a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/MonitorService.java +++ b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/MonitorService.java @@ -44,14 +44,6 @@ public interface MonitorService { */ Map queryMaster(User loginUser); - /** - * query zookeeper state - * - * @param loginUser login user - * @return zookeeper information list - */ - Map queryZookeeperState(User loginUser); - /** * query worker list * diff --git a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/MonitorServiceImpl.java b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/MonitorServiceImpl.java index ccec49b39a..934aaf4464 100644 --- a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/MonitorServiceImpl.java +++ b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/MonitorServiceImpl.java @@ -88,23 +88,6 @@ public class MonitorServiceImpl extends BaseServiceImpl implements MonitorServic return result; } - /** - * query zookeeper state - * - * @param loginUser login user - * @return zookeeper information list - */ - @Override - public Map queryZookeeperState(User loginUser) { - Map result = new HashMap<>(); - - result.put(Constants.DATA_LIST, null); - putMsg(result, Status.SUCCESS); - - return result; - - } - /** * query worker list * diff --git a/dolphinscheduler-api/src/test/java/org/apache/dolphinscheduler/api/controller/MonitorControllerTest.java b/dolphinscheduler-api/src/test/java/org/apache/dolphinscheduler/api/controller/MonitorControllerTest.java index d55c293e02..02f46d9cec 100644 --- a/dolphinscheduler-api/src/test/java/org/apache/dolphinscheduler/api/controller/MonitorControllerTest.java +++ b/dolphinscheduler-api/src/test/java/org/apache/dolphinscheduler/api/controller/MonitorControllerTest.java @@ -89,19 +89,4 @@ public class MonitorControllerTest extends AbstractControllerTest { logger.info(mvcResult.getResponse().getContentAsString()); } - @Test - public void testQueryZookeeperState() throws Exception { - MvcResult mvcResult = mockMvc.perform(get("/monitor/zookeepers") - .header(SESSION_ID, sessionId) - /* .param("type", ResourceType.FILE.name())*/) - .andExpect(status().isOk()) - .andExpect(content().contentType(MediaType.APPLICATION_JSON_UTF8)) - .andReturn(); - - Result result = JSONUtils.parseObject(mvcResult.getResponse().getContentAsString(), Result.class); - result.getCode().equals(Status.SUCCESS.getCode()); - - Assert.assertEquals(Status.SUCCESS.getCode(),result.getCode().intValue()); - logger.info(mvcResult.getResponse().getContentAsString()); - } } diff --git a/dolphinscheduler-api/src/test/java/org/apache/dolphinscheduler/api/service/MonitorServiceTest.java b/dolphinscheduler-api/src/test/java/org/apache/dolphinscheduler/api/service/MonitorServiceTest.java index 9e742851ea..61d2a9b078 100644 --- a/dolphinscheduler-api/src/test/java/org/apache/dolphinscheduler/api/service/MonitorServiceTest.java +++ b/dolphinscheduler-api/src/test/java/org/apache/dolphinscheduler/api/service/MonitorServiceTest.java @@ -74,14 +74,6 @@ public class MonitorServiceTest { /*Assert.assertEquals(Status.SUCCESS,result.get(Constants.STATUS));*/ } - @Test - public void testQueryZookeeperState() { - //TODO need zk - /*Map result = monitorService.queryZookeeperState(null);*/ - /*logger.info(result.toString());*/ - /*Assert.assertEquals(Status.SUCCESS,result.get(Constants.STATUS));*/ - } - @Test public void testGetServerListFromZK() { //TODO need zk diff --git a/dolphinscheduler-ui/src/js/conf/home/pages/monitor/pages/servers/zookeeper.vue b/dolphinscheduler-ui/src/js/conf/home/pages/monitor/pages/servers/zookeeper.vue deleted file mode 100644 index 215b9051f2..0000000000 --- a/dolphinscheduler-ui/src/js/conf/home/pages/monitor/pages/servers/zookeeper.vue +++ /dev/null @@ -1,67 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed with - * this work for additional information regarding copyright ownership. - * The ASF licenses this file to You under the Apache License, Version 2.0 - * (the "License"); you may not use this file except in compliance with - * the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ - - - diff --git a/dolphinscheduler-ui/src/js/conf/home/router/index.js b/dolphinscheduler-ui/src/js/conf/home/router/index.js index 9c93560743..2c82a83cc3 100644 --- a/dolphinscheduler-ui/src/js/conf/home/router/index.js +++ b/dolphinscheduler-ui/src/js/conf/home/router/index.js @@ -547,15 +547,6 @@ const router = new Router({ refreshInSwitchedTab: config.refreshInSwitchedTab } }, - { - path: '/monitor/servers/zookeeper', - name: 'servers-zookeeper', - component: resolve => require(['../pages/monitor/pages/servers/zookeeper'], resolve), - meta: { - title: 'Zookeeper', - refreshInSwitchedTab: config.refreshInSwitchedTab - } - }, { path: '/monitor/servers/apiserver', name: 'servers-apiserver', diff --git a/dolphinscheduler-ui/src/js/conf/home/store/monitor/actions.js b/dolphinscheduler-ui/src/js/conf/home/store/monitor/actions.js index c4014ec59e..163549bcd0 100644 --- a/dolphinscheduler-ui/src/js/conf/home/store/monitor/actions.js +++ b/dolphinscheduler-ui/src/js/conf/home/store/monitor/actions.js @@ -44,14 +44,5 @@ export default { reject(e) }) }) - }, - getZookeeperData ({ state }, payload) { - return new Promise((resolve, reject) => { - io.get('monitor/zookeepers', payload, res => { - resolve(res.data) - }).catch(e => { - reject(e) - }) - }) } } diff --git a/dolphinscheduler-ui/src/js/module/components/secondaryMenu/_source/menu.js b/dolphinscheduler-ui/src/js/module/components/secondaryMenu/_source/menu.js index 371326c507..ca58dd08af 100644 --- a/dolphinscheduler-ui/src/js/module/components/secondaryMenu/_source/menu.js +++ b/dolphinscheduler-ui/src/js/module/components/secondaryMenu/_source/menu.js @@ -238,12 +238,6 @@ const menu = { id: 1, enabled: true }, - { - name: 'Zookeeper', - path: 'servers-zookeeper', - id: 4, - enabled: true - }, { name: 'DB', path: 'servers-db',