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 validation for numOfAz #36

Merged
merged 1 commit into from
Mar 23, 2022
Merged

add validation for numOfAz #36

merged 1 commit into from
Mar 23, 2022

Conversation

robertchoi80
Copy link
Contributor

CAPA에서 허용하는 num-of-az 최대값인 3을 넘을 경우 에러 발생하도록 함

@Jaesang
Copy link
Contributor

Jaesang commented Mar 22, 2022

그러면 region별 az개수와 무관하게 3개가 고정된다고 봐야하나요?

@zugwan
Copy link
Contributor

zugwan commented Mar 22, 2022

그러면 region별 az개수와 무관하게 3개가 고정된다고 봐야하나요?

사용자에 따라서 컨트롤 플레인 HA 구성을 하지 않을 수 있으니 1개, 3개 둘 중 하나라고 봐야 할 듯 합니다.

@robertchoi80
Copy link
Contributor Author

그러면 region별 az개수와 무관하게 3개가 고정된다고 봐야하나요?

고정은 아니고요, 3개와 'region별 AZ 갯수'를 넘지 않는 범위에서 사용자가 지정할 수 있는 거죠. 실제로는 거의 2 아니면 3으로 사용하겠네요

@Jaesang
Copy link
Contributor

Jaesang commented Mar 22, 2022

etcd때문에 2는 안되는거죠? 1 아니면 3으로 고정해야하나;;

@zugwan
Copy link
Contributor

zugwan commented Mar 22, 2022

etcd때문에 2는 안되는거죠? 1 아니면 3으로 고정해야하나;;

네, 1 과 3 둘 중 하나만 선택해야 하고 그렇다면 컨트롤 플레인 개수 설정하는 것과 동일해야 한다는 결론이 나는데, 지금은 우리가 지원하지 않는 이미 준비된 AWS 자원 (VPC, Subnet 등등등) 을 사용하는 경우는 또 앞 내용이 유효하지는 않을 수 있습니다.

@Jaesang
Copy link
Contributor

Jaesang commented Mar 22, 2022

아 machineDeployment 갯수랑 etcd 갯수랑도 관계가 있는 건가요??

numofAz가 controlplane하고도 관계있던거 같아서요

@zugwan
Copy link
Contributor

zugwan commented Mar 22, 2022

아 machineDeployment 갯수랑 etcd 갯수랑도 관계가 있는 건가요??

워커 노드 생성을 위한 MachinePool이나 MachineDeployment 자원 모두 Subnet 정의가 필요한데 현재 우리는 cluster-api에서 자동으로 생성해주는 자원을 사용하고 있습니다. cluster-api는 컨트롤 플레인 갯수 설정 (= etcd 갯수) 에 맞추어서 생성을 하는 것이고 이러한 시나리오에서는 컨트롤 플레인 갯수 = numOfAz 여야 하는 상황입니다.

@robertchoi80
Copy link
Contributor Author

아 machineDeployment 갯수랑 etcd 갯수랑도 관계가 있는 건가요??

numofAz가 controlplane하고도 관계있던거 같아서요

그럼 일단은 1 또는 3 만 유효한 값이네요. 위에 주관님 말씀하신 예외 케이스는 차후 상황을 봐야 할듯 하고..

@ktkfree ktkfree merged commit e752470 into openinfradev:main Mar 23, 2022
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