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

[k8scluster] update CheckNodeGroupsOnK8sCreation API #1715

Conversation

sykim-etri
Copy link
Member

  • route to RestCheckNodeGroupsOnK8sCreation
  • return K8sClusterNodeGroupsOnCreation(json)
  • add testscript for CheckNodeGroupsOnK8sCreation

- route to RestCheckNodeGroupsOnK8sCreation
- return K8sClusterNodeGroupsOnCreation(json)
- add testscript for CheckNodeGroupsOnK8sCreation
@seokho-son
Copy link
Member

@sykim-etri 추가적으로, 정상 동작하지 않는 API는 설명 부분에 대략.. (WIP: This feature may not be fully functional yet) 라는 표기를 해주시면 감사하겠습니다. (swagger에 표기되도록)

@sykim-etri
Copy link
Member Author

@sykim-etri 추가적으로, 정상 동작하지 않는 API는 설명 부분에 대략.. (WIP: This feature may not be fully functional yet) 라는 표기를 해주시면 감사하겠습니다. (swagger에 표기되도록)

테스트가 완료되어서 정상 동작하는 API입니다.

@seokho-son
Copy link
Member

/approve

@github-actions github-actions bot added the approved This PR is approved and will be merged soon. label Aug 14, 2024
@cb-github-robot cb-github-robot merged commit dc47fc5 into cloud-barista:main Aug 14, 2024
4 checks passed
@seokho-son
Copy link
Member

@sykim-etri 향후에 주석 추가는 부탁드릴게요~

https://github.com/cloud-barista/cb-tumblebug/pull/1715/files#diff-cb255517d9a31aeb9089253f826a9e1d6961dc57f838500ccd2e7756e70ae7abR74

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved This PR is approved and will be merged soon.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants