-
Notifications
You must be signed in to change notification settings - Fork 9
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
Refactor createapp #63
Conversation
{ "path": "prometheus-operator", "namespace": "lma" }, | ||
{ "path": "eck-operator", "namespace": "elastic-system" }, | ||
{ "path": "fluentbit-operator", "namespace": "lma" } | ||
{ "app_group": "lma", "path": "prometheus-operator", "namespace": "lma", }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
여기 ,
는 문법에러 안나나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정했습니다. (테스트는 usercluster 생성 시나리오로 해봤었네요)
이 부분은 1번에 동의합니다.
|
https://github.com/openinfradev/tks-issues/issues/55 관련 내용을 반영한 PR 입니다.
어디서든 공통의 create-application workflow를 사용하게 하였고, 다양한 케이스에 대처하기 위해 몇가지 parameter를 추가하였습니다.
자세한 내용은 위의 이슈 내용 참고하시면 됩니다.
한가지 의견을 구하고 싶은 점은, app_prefix 인자를 추가하였는데, 둘 중 어느 쪽이 나은지 의견 부탁드립니다. (현재는 1번 안으로 되어있음)