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

add cluster-autoscaler chart for cluster-api provider #94

Merged
merged 2 commits into from
Feb 8, 2022

Conversation

zugwan
Copy link
Contributor

@zugwan zugwan commented Jan 27, 2022

Cluster-API 프로바이더만을 고려한 cluster-autoscaler 차트를 추가합니다.
설치 등에 관한 내용은 README.md 에 작성하였습니다.

@github-actions
Copy link

This PR is stale because it has been open 3 days with no activity. Remove stale label or comment or this will be closed in 3 days.

@github-actions github-actions bot added the Stale There has been no activity on this label Jan 31, 2022
@github-actions
Copy link

github-actions bot commented Feb 3, 2022

This PR was closed because it has been stalled for 10 days with no activity.

@github-actions github-actions bot closed this Feb 3, 2022
@bluejayA bluejayA removed the Stale There has been no activity on this label Feb 3, 2022
@zugwan zugwan reopened this Feb 7, 2022
@bluejayA
Copy link
Contributor

bluejayA commented Feb 7, 2022

https://github.com/openinfradev/tks-issues/issues/60 에 아래와 같이 정리되어 있는데

  1. 클러스터에 auto scaling 기능 활성화는 생성 시점에(만) 옵션으로 선택할 수 있도록 한다.
  2. auto scaling이 활성화된 클러스터의 normal worker는 machinepool 대신 machinedeployment를 사용해서 배포한다.
    2.1. autoscaler clusterapi 프로바이더에서는 machinepool을 노드 그룹으로 지원하지 않음
    2.2. 현재 machinepool에서는 replica 숫자만 설정할 수 있기 때문에 scale-down 수행 시 대상 노드 지정이 불가능함
  3. machinedeployment가 하나의 AZ/Subnet만 사용할 수 있는 문제를 보완하기 위해 각 AZ/Subnet 별로 machinedeployment를 생성하는 방식으로 진행한다: 방법은 현재 machinepool의 subnet들을 설정하는 방식과 유사하게 구현한다.

Cluster auto-scaler 제약 사항

  • 예를 들어 AZ가 3개인 경우 3개의 machinedeployment를 사용함. 단일 노드 확장 요청 때 하나의 machinedeployment만을 무작위로 선정하여 작업을 수행하기 때문에 한 번에 여러 노드를 확장하는 경우 특정 machinedeployment (특정 AZ)의 노드만 증가하는 문제가 발생할 수 있음
  • 모든 machinedeployment (AZ) 별 균등 배치는 보장하지 않음

저희가 디자인 문서를 따로 만들지 않은 만큼 (epic과 ticket에 관련 내용들과 논의과정은 적혀 있으나)기본적인 구현방식과 제약사항은 README에 "참고사항" 으로 들어가면 어떨까 생각했습니다만,

아니면,

  • 이건 별도로 tks cli까지 완료한 후에 공식 릴리즈시 tks 아키텍처 문서 (앞으로 만들 예정)과 사용자/운영자 매뉴얼에 넣는게 더 좋을까요?

의견 주세요

@seungkyua seungkyua merged commit 2311567 into main Feb 8, 2022
@seungkyua seungkyua deleted the add-cluster-autoscaler branch February 8, 2022 01:16
@zugwan
Copy link
Contributor Author

zugwan commented Feb 8, 2022

openinfradev/tks-issues#60 에 아래와 같이 정리되어 있는데

  1. 클러스터에 auto scaling 기능 활성화는 생성 시점에(만) 옵션으로 선택할 수 있도록 한다.
  2. auto scaling이 활성화된 클러스터의 normal worker는 machinepool 대신 machinedeployment를 사용해서 배포한다.
    2.1. autoscaler clusterapi 프로바이더에서는 machinepool을 노드 그룹으로 지원하지 않음
    2.2. 현재 machinepool에서는 replica 숫자만 설정할 수 있기 때문에 scale-down 수행 시 대상 노드 지정이 불가능함
  3. machinedeployment가 하나의 AZ/Subnet만 사용할 수 있는 문제를 보완하기 위해 각 AZ/Subnet 별로 machinedeployment를 생성하는 방식으로 진행한다: 방법은 현재 machinepool의 subnet들을 설정하는 방식과 유사하게 구현한다.

Cluster auto-scaler 제약 사항

  • 예를 들어 AZ가 3개인 경우 3개의 machinedeployment를 사용함. 단일 노드 확장 요청 때 하나의 machinedeployment만을 무작위로 선정하여 작업을 수행하기 때문에 한 번에 여러 노드를 확장하는 경우 특정 machinedeployment (특정 AZ)의 노드만 증가하는 문제가 발생할 수 있음
  • 모든 machinedeployment (AZ) 별 균등 배치는 보장하지 않음

저희가 디자인 문서를 따로 만들지 않은 만큼 (epic과 ticket에 관련 내용들과 논의과정은 적혀 있으나)기본적인 구현방식과 제약사항은 README에 "참고사항" 으로 들어가면 어떨까 생각했습니다만,

아니면,

* 이건 별도로 tks cli까지 완료한 후에 공식 릴리즈시 tks 아키텍처 문서 (앞으로 만들 예정)과 사용자/운영자 매뉴얼에 넣는게 더 좋을까요?

의견 주세요

README에만 넣으면 이후에 사용자에게 내용을 전달할 때 다시 자료가 필요하게 되니 사용자/운영자 메뉴얼에 넣는 것이 좋겠습니다. (혹은 둘다)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants