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 testscripts #1767

Conversation

sykim-etri
Copy link
Member

k8scluster용 테스트 스크립트를 업데이트합니다.

@seokho-son
Copy link
Member

감사합니다! 한시적으로 test 지원을 위한 개선이라고 생각하겠습니다.
가급적 test 스크립트 쪽 코드의 양을 늘리지 않으시길 ^^ (향후 모두 삭제 가능성도 있습니다....)

@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 23, 2024
@cb-github-robot cb-github-robot merged commit b153867 into cloud-barista:main Aug 23, 2024
2 checks passed
@sykim-etri
Copy link
Member Author

감사합니다! 한시적으로 test 지원을 위한 개선이라고 생각하겠습니다. 가급적 test 스크립트 쪽 코드의 양을 늘리지 않으시길 ^^ (향후 모두 삭제 가능성도 있습니다....)

여러 리전을 대상으로 생성, 삭제, 변경, 배포 등의 다양한 테스트하는 경우 스크립트를 활용하는 방식이 유용한데, 삭제할 필요가 있을까요?

@seokho-son
Copy link
Member

@sykim-etri 기본적으로는 코드가 많아지고 메인테이닝이 어려워지는 상황(코드, API 등이 변하면, 세부 test scripts 코드들도 수정해야 함)이라서 삭제를 고려하고 있는 부분입니다. 기여자 및 메인테이닝 인력은 늘~ 부족하므로 sustainable 한 메인테이닝 방법을 꼭 찾아야 합니다.

따라서, 메인테이닝이 더 쉬운 형태의 cli 형태로 변경해나갈 생각을 가지고 있습니다. 자동으로 생성되는 cli 코드를 활용하는 형태나, 여러 csp 리전 테스트도 진행할 수 있는 방안을 제시할 수 있는 경우,
점차 삭제해나갈 생각을 가지고 있습니다. 더 좋은 제안을 해주셔도 좋습니다. :)

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