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

Update KTcloud metadata [skip ci] #953

Merged

Conversation

jihoon-seo
Copy link
Member

For everyone's information:
https://github.com/cloud-barista/ktcloud/blob/main/ktcloud/main/config/config.yaml

주의 : KT Cloud CSP에서는 Region 단위가 없이 Zone 단위로 구분되어 서비스되나, 편의를 위해 Region 정보를 기입했음.


  • 참고: KT Cloud는 Zone ID, Image ID, Spec ID 가 UUID 형태로 되어 있습니다.
  • 모든 존의 Image ID가 45e20478-6c24-4c32-a5e5-2ba2d464a8d1 으로 동일한데,
    • 'Seoul M2' 만 63de6d04-7f1b-4924-8e95-1acd6581ca0c 로 다릅니다.
    • Spec ID 도 마찬가지 입니다.
    • [안] 동일한 것들은 CommonImage, CommonSpec 으로 묶을 수도 있겠습니다.
  • 이 PR에서 '김해' 존을 추가했는데, KT Cloud 에서는 'KOR-HA' 라는 이름으로 부르고 있습니다.
    • 이전 PR에서 추가하지 않아 이 PR에서 새롭게 추가하는 것이기도 하고,
      이름처럼 HA용으로 사용하는 것을 가정하고 있을까 하여
      conf.env 에서 맨 마지막 순서로 넣었습니다.
    • [안] 김해도 한국이므로, 다른 한국 존 직후에 오도록 US West 위쪽으로 순서를 바꿀 수도 있겠습니다.

[KT Cloud 웹 콘솔]
image

@jihoon-seo jihoon-seo changed the title Update KTcloud metadata Update KTcloud metadata [skip ci] Nov 10, 2021
@seokho-son
Copy link
Member

@jihoon-seo 음.. 리전 등이 없어서 임의로 지정된 문자열 사용하는 것이, 향후 이슈로 될 수도 있을 것 같긴한데.. (MCIR 자동 생성 등에)
일단.. kt는 바로 사용하는 것이 아니니, 머지하고 문제가 발생하면 수정해나가도록 할까요?

@jihoon-seo
Copy link
Member Author

@seokho-son 넵 😊
현재 KT cloud 관련하여 TB 테스트 스크립트에는

RegionName Region Zone CONN_CONFIG 비고
kt-kr-central-a KOR-Cheonan eceb5d65-6571-4696-875f-5a17949f3317 kt-kr-central-a 천안
kt-kr-central-b KOR-Cheonan ... kt-kr-central-b 천안
kt-kr-seoul-m KOR-Seoul ... kt-kr-seoul-m 서울
kt-kr-seoul-m2 KOR-Seoul ... kt-kr-seoul-m2 서울
kt-us-west US ... kt-us-west LA
kt-kr-kimhae KOR-Kimhae ... kt-kr-kimhae 김해

이렇게 해 놓았으며
예를 들어 kt-kr-central-akt-kr-central-bRegion 값이 KOR-Cheonan 으로 같아서
만약 TB 스크립트에서 Region 값을 이용하는 부분이 있다면
구분이 안 되거나 중복 에러가 발생할 것입니다.

아마도 Region 값은 TB에서 임의로 적어서 Spider에 등록해도 될 것 같으며 (테스트는 필요)

KT cloud도 다른 CSP와 결을 맞추면서 (다른 CSP는 conf.env 각 엔티티가 각각 다른 Region임)
KT cloud의 각 Zone을 마치 Region처럼 여기고 다루려면

위의 Region 값을 각각 다르게 하는 것이 좋을 수도 있겠습니다. 😊

@seokho-son seokho-son merged commit afc1b55 into cloud-barista:main Nov 10, 2021
@jihoon-seo jihoon-seo deleted the 211110_Update_KTcloud_metadata branch November 15, 2021 05:24
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.

2 participants