데스크톱 클러스터 IP 관리 기능 개발#62
Merged
dhslove merged 21 commits intoablecloud-team:ablestack-brontofrom Jul 30, 2021
Merged
Conversation
master merge
Ablestack allo
Desktop 플러그인 프로토타입 샘플 수정
master merge
Desktop 플러그인 샘플 수정
master merge로 인한 경로 변경
Master Template 프로토타입 샘플 추가
로그 삭제, MasterVersion Response 추가
Desktop 플러그인 i18n merge, Desktop Cluster IP 범위 조회 샘플 API 추가
IP 범위 조회 API 개발 완료
데스크톱 IP 범위 추가 개발
데스크톱 클러스터 IP 범위 삭제 개발 완료
데스크톱 클러스터 IP 범위 추가시 gateway, netmask, ip 벨리데이션 체크
데스크톱 클러스터 IP 조회 시 로직 변경 (전체검색, 파라미터검색)
데스크톱 클러스터 IP 범위 추가시 전체 클러스터에서 사용중인 IP범위 조회하여 밸리데이션 체크
Desktop 플러그인용 ApiConstants 수정
EventType 수정
This was
linked to
issues
Jul 30, 2021
dhslove
approved these changes
Jul 30, 2021
| public static final String TEMPLATE_OS_TYPE = "templateostype"; | ||
| public static final String DESKTOP_CLUSTER_ID = "desktopclusterid"; | ||
| public static final String DESKTOP_CLUSTER_NAME = "desktopclustername"; | ||
| public static final String IP_RANGE = "iprange"; |
There was a problem hiding this comment.
Works를 위해 만들어진 상수인데, 일반 상수인 것 처럼 보여서 명확한 구분이 필요해 보입니다.
상수에 접두어 "DESKTOP_"를 붙여서 명명하는 것은 어떤지요?
dhslove
approved these changes
Jul 30, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR 설명
Mold 의 신규 기능인 데스크톱 서비스의 샘플 API 및 UI 가 포함되었으며,
클러스터 메뉴 > 상세화면 IP 관리 탭 을 개발하기 위한 PR입니다.
관련 이슈
변경 구분
기능/개선 규모 또는 버그 심각도
기능/개선 규모
스크린샷
테스트 방법 및 결과
위 PR 적용된 테스트 환경 172.16.0.91:5050