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

Enhance registerExistingVNet feature #1018

Conversation

jihoon-seo
Copy link
Member

#1016 (comment) 댓글의 내용대로,

현재 option=register는

CB-TB에 없고 CB-SP에 등록되어 있는 정보를 다시 조회하여, CB-TB 오브젝트로 만드는 기능으로 사용되고 있는데,

TB, SP에는 없고 CSP에만 있는 리소스를 등록하는 경우에도
해당 옵션을 함께 사용하도록 하는 것이 좋을 것 같네요. (API를 가급적 간결하게 하기 위해)

아마도, 사용자가 option=register로 요청할 때, CSP 자원 ID를 추가로 입력할 수 있도록 하고,
CSP 자원 ID가 추가로 입력된 경우에는
CSP 자원을 직접 등록하는 것으로 이해하고 처리하도록 하면 되지 않을까 싶습니다. (CB-SP에도 등록하고, CB-TB에도 등록하고)

사용자가 registerExistingVNet API 호출 시

  1. Request body 에 cspVNetId 필드를 빈칸으로 두면
    => 사용자가 Request body 에 명시한 connectionNamename 으로 CB-Spider에 조회하여
    Get 성공하면 TB object로 등록
  2. Request body 에 cspVNetId 필드를 명시하면
    => CB-Spider의 regvpc API를 호출하여
    CSP에만 있는 자원을 SP에도 등록하고 TB에도 등록하도록 하였습니다.

image

❯ ./test-register-csp-vNet.sh -n jhseo -c aws -r 1
####################################################################
## 3. vNet: Register
####################################################################


- Register vNet in aws-ap-southeast-1
{CONN_CONFIG[4,1]}: aws-ap-southeast-1
{
  "id": "aws-ap-southeast-1-jhseo",
  "name": "aws-ap-southeast-1-jhseo",
  "connectionName": "aws-ap-southeast-1",
  "cidrBlock": "10.0.0.0/24",
  "subnetInfoList": null,
  "description": "",
  "cspVNetId": "vpc-018e42dc39078bb5d",
  "cspVNetName": "ns01-aws-ap-southeast-1-jhseo",
  "status": "",
  "keyValueList": null,
  "associatedObjectList": [],
  "isAutoGenerated": false,
  "systemLabel": "Registered from CSP resource"
}

@jihoon-seo jihoon-seo requested a review from seokho-son January 21, 2022 06:31
@seokho-son
Copy link
Member

LGTM!
감사합니다! ;0

@seokho-son
Copy link
Member

/approve

@seokho-son seokho-son merged commit 6c872c6 into cloud-barista:main Jan 21, 2022
@jihoon-seo jihoon-seo deleted the 220121_Enhance_registerExistingVNet_feature branch January 21, 2022 09:10
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