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 NCP-VPC & NHN Cloud regions & zones #1180

Conversation

jihoon-seo
Copy link
Member

@jihoon-seo jihoon-seo commented Sep 26, 2022

Comment on lines +2778 to +2792
# # region02
# IY=$NHNcloudKrPangyo2
# # Location: South Korea (Pangyo)
# RegionLocation[$IX,$IY]="South Korea (Pangyo)"
# RegionName[$IX,$IY]=nhncloud-kr-pangyo2
# RegionKey01[$IX,$IY]=Region
# RegionVal01[$IX,$IY]=KR1
# RegionKey02[$IX,$IY]=Zone
# RegionVal02[$IX,$IY]=kr-pub-b
# CONN_CONFIG[$IX,$IY]=nhncloud-kr-pangyo2
# IMAGE_NAME[$IX,$IY]=5396655e-166a-4875-80d2-ed8613aa054f
# IMAGE_TYPE[$IX,$IY]=${CommonImageType[$IX]}
# SPEC_NAME[$IX,$IY]=m2.c4m8
# DISK_TYPE[$IX,$IY]=${CommonDiskType[$IX]}
# DISK_SIZE[$IX,$IY]=${CommonDiskSize[$IX]}
Copy link
Member

Choose a reason for hiding this comment

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

@jihoon-seo
향후에도 활용이 필요한 코드인가요?

Copy link
Member Author

Choose a reason for hiding this comment

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

conf.env 에 나열되어 있는 다른 CSP, 예를 들어 AWS를 보면,
각 엔티티는 각 AWS 리전 (예: ap-southeast-1) 을 나타내며,
각 엔티티 (리전) 에 대해 존까지 지정하고 있습니다. (예: ap-southeast-1a)

한편, conf.env 에서 NHN Cloud, NCP-VPC 의 경우

  • NHNcloudKrPangyo1: KR1 리전의 kr-pub-a
  • NHNcloudKrPangyo2: KR1 리전의 kr-pub-b
  • ...

에 해당됩니다.

다른 CSP에 대해서도 각 리전별로 존을 지정하여 1 리전 (+ 1 존) - 1 엔트리 형식으로 입력되어 있는 현황을 고려하여
NHN Cloud, NCP-VPC 에 대해서도 1 리전 (+ 1 존) - 1 엔트리 형식을 가져가기 위해
각 리전별 1존에 대해서만 enable 하였으며
나머지 존은 주석 처리 해 두었습니다.

향후 유사시 주석을 해제하여 활용 가능할 것으로 보입니다. 😊

@seokho-son
Copy link
Member

/approve

감사합니다!

@seokho-son seokho-son merged commit c3e597f into cloud-barista:main Sep 26, 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.

2 participants