Skip to content

Fix MCIS label input bug - #712

Merged
jihoon-seo merged 3 commits into
cloud-barista:mainfrom
seokho-son:main
Sep 7, 2021
Merged

jihoon-seo merged 3 commits into
cloud-barista:mainfrom
seokho-son:main

Conversation

@seokho-son

Copy link
Copy Markdown
Member

This PR fixes MCIS label bug.

Bug reported by @sunmi from Slack (https://cloud-barista.slack.com/archives/CUUR82Q4D/p1630909405004300?thread_ts=1630627713.002100&cid=CUUR82Q4D)

sunmi
@seokho Son (seokho-son) 안녕하세요. 문의 드립니다. mcis와 mcis.vm에 label을 보내보았는데, 결과값에 나타나지 않습니다.
테스트 환경은 다음과 같습니다.

  • cb-spider : docker 기반 실행 v0.4.9
  • cb-tumblebug : docker 기반 실행 v0.4.4
    위의 버전에서는 적용되지 않는 것인지, 아니면 보낸 값에 문제가 있는 것인지 확인 한번 부탁드립니다.

테스트

curl -sX POST http://localhost:1323/tumblebug/ns/mcks-ns/mcis -H 'Content-Type: application/json' -H "Authorization: Basic $(echo -n default:default | base64)" -d
'{
"name": "cluster-01",
"description": "label test",
"label": "mcks",
"vm": [ {
"name": "cluster-01-01",
"imageId": "aws-ap-northeast-1-ubuntu1804",
"vmUserAccount": "cb-user",
"connectionName": "config-aws-ap-northeast-1",
"sshKeyId": "aws-ap-northeast-1-sshkey",
"specId": "aws-ap-northeast-1-t2-medium-spec",
"securityGroupIds": [
"aws-ap-northeast-1-sg"
],
"vNetId": "aws-ap-northeast-1-vpc",
"subnetId": "aws-ap-northeast-1-vpc-subnet",
"description": "description",
"vmUserPassword": "",
"label": "mcks-vm"
}
]
}' | jq ''

결과

{
"id": "cluster-01",
"name": "cluster-01",
"status": "Running-1(1/1)",
"targetStatus": "Running",
"targetAction": "Create",
"installMonAgent": "",
"label": "",
"placementAlgo": "",
"description": "label test",
"vm": [
{
"id": "cluster-01-01",
"name": "cluster-01-01",
"vmGroupId": "",
"location": {...
},
"status": "Running",
"targetStatus": "None",
"targetAction": "None",
"monAgentStatus": "notInstalled",
"systemMessage": "",
"createdTime": "2021-09-06 06:15:52",
"label": "",
"description": "description",
"region": {
"Region": "ap-northeast-1",
"Zone": "ap-northeast-1a"
},
...
}
]
}

@jihoon-seo
jihoon-seo merged commit e6572b6 into cloud-barista:main Sep 7, 2021
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