Added new app group tks-cluster #89
Conversation
estherk0
commented
Jul 5, 2021
estherk0
commented
- tks-cluster app group 추가
- cluster-api-aws, cluster-api-openstack 등의 kubernetes cluster provisioning을 위한 구성 정보를 담는다.
- 현재는 cluster-api-aws만 추가
tks-cluster/aws/site-values.yaml
Outdated
| name: CLUSTER_NAME | ||
| region: ap-northeast-2 | ||
| networkSpec: | ||
| vpc: vpc-00000000000 |
There was a problem hiding this comment.
테스트할때 바로 사용할 수 있는 site-values.yaml인지 궁금합니다.
There was a problem hiding this comment.
유효한 값은 아닙니다. 테스트할 때 사용가능한 vpc를 각자 만들어야 되지 않을까요?
마찬가지로 이 밑쪽에 subnets 값도 유효하지 않은 값입니다.
There was a problem hiding this comment.
실제 사용가능한 값이 아닌경우에는 반드시 환경에 맞춰 변경해줘야할 필요가 있어 주석이 있었으면 좋겠습니다.
There was a problem hiding this comment.
그러면 저희 aws 환경에 테스트 가능한 vpc 환경을 만들어놓고 그 값으로 세팅해놓을까요? 따로 주석도 추가하고..
대신 Elastic IP를 항상 할당해놔야해서 비용은 들 것 같습니다.
There was a problem hiding this comment.
같은 VPC에 여러 클러스터를 만들어도 상관없나요? 상관없다고 해도 비용적인 측면을 들여가면서 실제 값을 넣을 필요는 없지 않을까 합니다. 오히려 실제로 돌아가지 않는 현재 값에다가 주석을 통해 인지시켜주는게 더 좋을것 같아요
There was a problem hiding this comment.
네 그럼 주석만 처리하도록 하겠습니다. TO_BE_FIXED로 하지 않은 이유는 실제 값과 유사한 값을 예시로 보여주기 위함인데 TO_BE_FIXED로 하는게 나을까요?
There was a problem hiding this comment.
TO_BE_FIXED 로 하고 옆에 주석으로 예시를 넣으면 어떨까요?
There was a problem hiding this comment.
각 필드에 대한 설명 주석을 추가하고 값에 대한 예시도 주석으로 남겨뒀습니다.
