-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add WF to remove-user-cluster #43
Conversation
@Jaesang workflow 엔 이상이 없는 거 같은데 계속 lint 에러가 나네요. 혹시 원인이 뭔지 아실까요?? |
in remove-usercluster WF
7c5705b
to
0f6c0af
Compare
among csi-driver and k8s-addons
dab69f8
to
8242910
Compare
create-usercluster-wftpl.yaml 파일 lint 오류는 아래처럼 수정하면 나타나지 않습니다.
|
05847aa
to
70bbc9b
Compare
@zugwan 말씀하신 내용들 수정했습니다. 근데 지금은 https://github.com/openinfradev/tks-flow/runs/5441868927?check_suite_focus=true 에러가 발생하는데, decapod-flow의 workflow 등 의존성 있는 workflow를 불러오지 못해 에러가 발생하는 걸로 보입니다. fake kubeconfig를 사용해서 생기는 문제인듯 한데, 어떻게든 실제 cluster로부터 workflow들을 읽어올 수 있도록 조치가 필요할 듯 합니다. 싱크때 논의하시죠 |
919e740
to
8bef9d6
Compare
delete cluster 전반에 대한 질문입니다. 현재 lma 생성시에 지금 작성하신대로라면, cluster 생성시에 생성한 argocd app 3개는 삭제가 되지만, lma 생성시에 생성한 상기 app 들은 '명시적'으로 삭제가되지 않을 것 같은데요. |
LMA 등 서비스 삭제는 user-cluster 삭제 전에 수행된다고 생각했습니다. 클러스터 생성 -> 서비스 생성 -> 서비스 삭제 -> 클러스터 삭제 의 순서로요 제가 질문을 잘 이해한 게 맞나요? '명시적'이라는게 어떤 의미로 말씀하신 건지요? |
=> 구현 완료 |
94e5e07
to
ac3db31
Compare
8ad48b4
to
3e2fee0
Compare
since it's unnecessary
- name: app_prefix | ||
value: "{{=sprig.substr(0, 8, workflow.parameters.cluster_id)}}" | ||
- name: filter | ||
value: "app={{=sprig.substr(0, 8, workflow.parameters.cluster_id)}}-{{workflow.parameters.app_group}}" |
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.
이미 substr 들어가있는데요?
관련 이슈: openinfradev/decapod-issues#7
workflow 초안 작성 후 admin cluster 에서 테스트 중입니다. -> 테스트 완료!
openinfradev/decapod-flow#77 도 같이 머지되어야 합니다.