Create 'lma' namespace before deploy 'lma' apps. #53
Conversation
Jaesang
requested review from
zugwan,
ktkfree,
intelliguy,
robertchoi80,
seungkyua and
kyuho24
|
@intelliguy @Jaesang "tks-lma-federation-wftpl.yaml" 와 "lma-uniformed-wftpl.yaml"의 차이를 여쭤보기도 했었는데, 한 곳에 수정이 들어가면 다른 곳에도 유사하게 반영될 필요가 있는지 항상 봐야 하는건가요? 이 PR 내용도 tks-lma-federation-wftpl.yaml 반영여부를 봐야 하나해서요. |
|
노드에 taco-lma=enabled 레이블 붙이는 건 어디서 하기로 했었죠? |
성일님이 만드는 유저클러스터 워크플로우에서 작업하는 것으로 알고있습니당 |
tks-lma-federation-wftpl.yaml는 어디에 있는 것인가요? decapod-flow에서는 lma-uniformed-wftpl.yaml만 관리되고 있습니다. |
https://github.com/openinfradev/decapod-flow/blob/main/templates/decapod-apps/tks-lma-federation-wftpl.yaml >> 여기에 있습니다. deprecated 된건 아닌걸로 보여서 |
어제 논의한 내용과 성일님이 말씀하신 내용으로 보면 tks용 lma 워크플로우는 sktelecom/tks-flow에 저장하고 내부적으로 lma-uniformed 호출하는 것으로 정리하실 거 같습니다. @intelliguy 확인 부탁 드려요. |
저도 주관님이 말씀하신 것 처럼 생각하고 있으나, 그럼 현재 workflow template을 다시 만들어야 해서, 이번 범위에 들어갈지는 @intelliguy님 의견 들어봐야 할 듯 |
|
This PR is stale because it has been open 3 days with no activity. Remove stale label or comment or this will be closed in 3 days. |
|
This PR was closed because it has been stalled for 10 days with no activity. |
|
This PR was closed because it has been stalled for 10 days with no activity. |
|
This PR was closed because it has been stalled for 10 days with no activity. |
|
This PR was closed because it has been stalled for 10 days with no activity. |
because 'lma' namespace must exist before deploy 'lma' apps.
Jaesang
force-pushed
the
add-namespace
branch
from
b191185 to
5671c9e
Compare
because 'lma' namespace must exist before deploy 'lma' apps.