Cb-tumblebug: Change MCIS provisioning meta data for not assigned IP address

Created on 30 Sep 2021  ยท  3Comments  ยท  Source: cloud-barista/cb-tumblebug

What would you like to be enhanced

  • https://github.com/cloud-barista/cb-tumblebug/blob/main/src/core/mcis/provisioning.go ์—๋Š” MCIS ์ƒ์„ฑ๊ด€๋ จ ์†Œ์Šค๊ฐ€ ํฌํ•จ๋˜์–ด ์žˆ์Šต๋‹ˆ๋‹ค.
  • https://github.com/cloud-barista/cb-tumblebug/blob/11af4b1c221dd0f3942cbcdc9b0f1a1e7ad9fcac/src/core/mcis/provisioning.go#L706 func CreateMcis ๋กœ MCIS๋ฅผ ์ƒ์„ฑํ•˜๋ฉด, ์š”์ฒญ๋œ VM๋“ค์ด CSP์— ์ƒ์„ฑ์ด ๋ฉ๋‹ˆ๋‹ค.
  • VM ์ƒ์„ฑ์ด ์™„๋ฃŒ๋˜๋ฉด vmInfoData.PublicIP ์™€ vmInfoData.PublicDNS ์— ์‹ค์ œ IP, DomainName์ •๋ณด๊ฐ€ ์ €์žฅ๋ฉ๋‹ˆ๋‹ค.
  • VM ์ƒ์„ฑ์ด ์ง„ํ–‰ ์ค‘์ด๊ฑฐ๋‚˜, ๋ฌธ์ œ๊ฐ€ ๋ฐœ์ƒํ•œ ๊ฒฝ์šฐ์—๋Š” "Not assigned yet" ์ด๋ผ๋Š” ๋ฉ”์‹œ์ง€๋ฅผ ์ถœ๋ ฅํ•˜๋„๋ก ๊ฐ’์ด ์ง€์ •๋˜์–ด ์žˆ์Šต๋‹ˆ๋‹ค.

    • "Not assigned yet" ๋ฉ”์‹œ์ง€๊ฐ€ ์˜๋ฏธ ์—†์ด ๊ธธ๊ณ , "yet" ์ด๋ผ๋Š” ์‹œ์ ์ด ๋ถˆ๋ช…ํ™•ํ•˜๋ฏ€๋กœ, ์ด๋ฅผ "empty"๋กœ ๋ณ€๊ฒฝํ•˜๋ฉด ์ข‹๊ฒ ์Šต๋‹ˆ๋‹ค.

Why is this needed

  • MCIS ํ”„๋กœ๋น„์ €๋‹ ์ถœ๋ ฅ ๋ฉ”์‹œ์ง€ ๊ฐœ์„ 

Proposed solution

https://github.com/cloud-barista/cb-tumblebug/blob/main/src/core/mcis/provisioning.go ์—์„œ

"Not assigned yet" ๋ฅผ ๊ฒ€์ƒ‰ํ•˜์—ฌ "empty" ๋กœ ๋ณ€๊ฒฝ

CB-TB ๊ธฐ์—ฌ์ž์˜ ํ…Œ์ŠคํŒ… ์ค€์ˆ˜ ์‚ฌํ•ญ ํ™•์ธ ํ•„์š”ํ•ฉ๋‹ˆ๋‹ค..^^

ํ•ด๋‹น ์ด์Šˆ๊ฐ€ ์ •๋ฆฌ๋˜๋ฉด
$ ./create-all.sh ์˜ ๊ฒฐ๊ณผ์—์„œ VM ์ •๋ณด ํ•ญ๋ชฉ์ด ๋ฐ€๋ ค๋‚˜๋Š” ๋ฌธ์ œ๋„ ํ•ด๊ฒฐ์ด ๋ฉ๋‹ˆ๋‹ค..^^ (์•„๋ž˜ ๊ทธ๋ฆผ์—์„œ Status๊ฐ€ Failed์ธ ํ•ญ๋ชฉ ์ฐธ๊ณ )

image

enhancement good first issue

All 3 comments

@leedohun ๋‹˜~
ํ•ด๋‹น ์ด์Šˆ๋Š” ์†Œ์Šค์ฝ”๋“œ๋ฅผ ์‚ด์ง ์ˆ˜์ •ํ•ด๋ณผ ์ˆ˜ ์žˆ๋Š” good first issue ์ž…๋‹ˆ๋‹ค.
ํ˜น์‹œ ๊ด€์‹ฌ ์žˆ์œผ์‹ ๊ฐ€์š”~? ^^

/assign @leedohun

์ด ์ด์Šˆ๋Š” PR #826 ์— ์˜ํ•ด ํ•ด๊ฒฐ๋˜์—ˆ์œผ๋ฏ€๋กœ close ํ•ฉ๋‹ˆ๋‹ค~~ ๐Ÿ˜Š

Was this page helpful?
0 / 5 - 0 ratings

Related issues

jihoon-seo picture jihoon-seo  ยท  6Comments

jihoon-seo picture jihoon-seo  ยท  6Comments

seokho-son picture seokho-son  ยท  3Comments

jihoon-seo picture jihoon-seo  ยท  4Comments

seokho-son picture seokho-son  ยท  5Comments