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 feature to create MCIS dynamically #875

Merged
merged 6 commits into from
Oct 20, 2021

Conversation

seokho-son
Copy link
Member

  • common 리소스로 등록된 spec 및 image 명만 입력하면, 기본 자원들로 MCIS를 생성해주는 MCIS 동적 생성 기능
    • connectionConfig 를 따로 지정하지 않는 것이 핵심.
    • spec이 지정되면 역으로 connectionConfig를 조회하고, 해당 connectionConfig에 관련된 vNet 등의 자원들로 MCIS를 생성
    • 현 PR에서는 생성까지 수행하지는 않고, MCIS 구성 형태만 출력으로 보여줌.
    • 해당 피쳐 확인을 위한 여러가지 수정들이 PR에 커밋으로 포함됨.

/ns/{nsId}/mcisDynamic

입력 예시

{
  "name": "mcis01",
  "vm": [
    {
      "commonImage": "ubuntu18-04",
      "commonSpec": "ali-ap-se-2-ecs-s6-c1m1-small",
      "description": "string",
      "label": "none",
      "name": "vm",
      "vmGroupSize": "3"
    }
  ]
}

@seokho-son
Copy link
Member Author

향후 필요한 작업

  • 실제 MCIS 생성으로 연결
  • default 자원이 미리 생성되지 않은 경우, 동적으로 생성하도록 기능 추가

@seokho-son
Copy link
Member Author

ref #859

Copy link
Member

@jihoon-seo jihoon-seo left a comment

Choose a reason for hiding this comment

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

@seokho-son 고생 많으셨습니다~~
실제 실행했을 때 동작과 결과도 궁금해요
나중에 보여 주세요... 😊

src/api/rest/server/mcir/common.go Outdated Show resolved Hide resolved
src/core/mcir/common.go Show resolved Hide resolved
src/core/mcir/common.go Outdated Show resolved Hide resolved
@seokho-son
Copy link
Member Author

@jihoon-seo 리뷰 감사합니다! 모두 반영하였습니다.

@jihoon-seo jihoon-seo merged commit 1dc41c0 into cloud-barista:main Oct 20, 2021
@jihoon-seo jihoon-seo added the hacktoberfest-accepted hacktoberfest-accepted label Oct 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted hacktoberfest-accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants