Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

최소사항 클러스터 지원을 위한 여정 #174

Merged
merged 3 commits into from
Sep 12, 2023

Conversation

intelliguy
Copy link
Contributor

@intelliguy intelliguy commented Aug 29, 2023

@intelliguy intelliguy force-pushed the minimal_deploy_support branch from 2ca2e9f to 67afc19 Compare August 29, 2023 08:41
@ktkfree
Copy link
Contributor

ktkfree commented Aug 29, 2023

각각의 타입에 따라 인프라 구성이 다르고, 좀 복잡한데요..
아래의 내용으로 팀내 공유 한번하고, tks-api db scheme 작업및 로직 작업 진행하겠습니다.

정리하면,

  • control plane (
    • aws : cpReplicas
    • eks : N/A
    • replicas 로 개수를 조정하며, eks 는 control plane 을 dont care 하므로 N/A
  • worker ( tks )
    • aws, eks ( machine pool (managed:eks) 사용 )
      • 사용자는 tks-console 에서 "인프라 노드(1 - 3)" 의 값을 입력. 이 값이 곧 tksInfraNode 이며, tksInfraNodeMax 의 default
      • minSize : 0
      • maxSize : tksInfraNodeMax
      • replicas : tksInfraNode
      • 이 스펙에서는 replicas 로 개수를 조정함.
  • worker ( user )
    • aws - machine deployment 사용
      • 사용자는 tks-console 에서 "워커 노드(1 - 100)" 의 값을 입력. 이 값이 곧 tksInfraNodeMax
      • numOfAz : ? ( 아마도 1 고정일듯 )
      • minSizePerAZ : 0
      • maxSizePerAZ : tksUserNodeMax
      • 이 스펙에는 az 설정을 고정하고, replicas 가 아닌 min. max size 로 개수를 조정함.
    • eks - **machine pool (managed) ** 사용
      • 사용자는 tks-console 에서 "워커 노드(1 - 100)" 의 값을 입력. 이 값이 곧 tksUserNode 이며, tksUserNodeMax 의 default
      • minSize : 0
      • maxSize : tksUserNodeMax
      • replicas : tksUserNode
      • 이 스펙에서는 replicas 로 개수를 조정함.

@intelliguy
Copy link
Contributor Author

intelliguy commented Aug 29, 2023

numOfAz는 리전마다 다르긴한데 우리가 쓰는곳은 3일꺼에요...

저도 변경된 이름을 기준으로 workflow 수정 갑니다.

@ktkfree
Copy link
Contributor

ktkfree commented Aug 29, 2023

numOfAz는 리전마다 다르긴한데 우리가 쓰는곳은 3일꺼에요...

저도 변경된 이름을 기준으로 workflow 수정 갑니다.

그렇다면, 현재 기준으로 machineDeployment 는 구성하기 어렵습니다. numOfAz 가 3 인 상황에 사용자가 7을 입력했다면 어떻게 spec 을 구성하면 될까요?
또한, multiAZ 지원으로 lma가 포함된 user cluster 를 생성하게 된다면 ebs region 이슈가 존재하기도 하고요.
이 부분 역시 팀 공유하며 의견을 들어보면 좋겠습니다.

@intelliguy
Copy link
Contributor Author

MachineDeploy와 MachinePool은 aws 내부적으로 동작하는 방식도 다르고 효과도 다르므로 고려는 해야겠지만...
누구나 쉽게 쓸수 있는 서비스를 구성한 다는 관점에서는 이런 제약이 있고 설명이 필요한 내역은 과감히 삭제하는 것이 좋겠다는 의견입니다.
(MchineDeploy 쓰지말자..)

Copy link
Contributor

@bluejayA bluejayA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WIP 동안 merge 방지

@ktkfree ktkfree changed the title [WIP] 최소사항 클러스터 지원을 위한 여정 최소사항 클러스터 지원을 위한 여정 Sep 11, 2023
@intelliguy intelliguy force-pushed the minimal_deploy_support branch from 1b0f917 to 766850f Compare September 11, 2023 04:23
@intelliguy intelliguy force-pushed the minimal_deploy_support branch 2 times, most recently from 68e815c to f9ea661 Compare September 11, 2023 04:54
Copy link
Member

@Siyeop Siyeop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@intelliguy intelliguy merged commit 6dc71d6 into develop Sep 12, 2023
@intelliguy intelliguy deleted the minimal_deploy_support branch September 12, 2023 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants