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

LMA workflow 수정 #67

Merged
merged 2 commits into from
Jan 25, 2022
Merged

LMA workflow 수정 #67

merged 2 commits into from
Jan 25, 2022

Conversation

robertchoi80
Copy link
Contributor

@robertchoi80 robertchoi80 commented Jan 24, 2022

다음 두가지 내용 퀵하게 fix해서 올립니다

  • Loki 설치 시 eventrouter 함께 배포
  • LMA 설치 시 target_cluster parameter 지정 (bug fix)

@intelliguy
Copy link
Contributor

intelliguy commented Jan 24, 2022

target_cluster 변수를 넣어줘야 한다면 다른 앱(service-mesh, openstack..)의 값들도 바꿔줘야 하나요?

{ "path": "istio-operator", "namespace": "istio-operator" }

로직 상으로는 없으면 workflow에 지정한것 가져오는 것 같은데, 여기서 bugfix로 개별 입력하신 이유도 궁금합니다.

TARGET_CLUSTER="{{workflow.parameters.site_name}}"

@robertchoi80
Copy link
Contributor Author

{ "path": "istio-operator", "namespace": "istio-operator" }

네, 맞습니다. 결국 service-mesh 등 다른 앱 쪽에도 적용되어야 하는데, 그쪽은 decapod - tks 간의 정리가 끝나면 적용하려고 생각하고 있어요.

말씀하신 대로 로직 상에는 param을 받아서 처리하는 쪽에는 null 값에 대한 고려가 되어 있는데, 호출하는 입장에서 target_cluster를 지정하지 않으면 https://github.com/openinfradev/decapod-flow/blob/main/templates/argo-cd/createapp-wftpl.yaml#L90 부분에서 item.target_cluster 를 참조하지 못하여 에러가 발생해버리네요. workflow에서 다양한 조건문 사용이 가능하다면 "if item has target_cluster" 처럼 분기 처리할 수 있을 텐데 현재 그런 기능은 찾지 못했습니다.

@zugwan
Copy link
Contributor

zugwan commented Jan 24, 2022

prepare-etcd-secret 의 경우 계속 에러가 발생하고 있어 아래 issue 해결 후 다시 추가해야할 것 같습니다. openinfradev/tks-issues#39

이 부분 때문에 prepare-etcd-secret 호출하는 부분이 제거되었는데 TKS (cluster-api) 환경에서는 정상 동작하니 그대로 두는게 좋을 듯 합니다.

아래는 하나카드 환경 준비하느라 구성한 site 파일인데 client_certfile, client_keyfile 파일 이름이 kubespray에서는 실제 노드 이름으로 설정되기 때문에 환경에 따라 달라질 수 밖에 없어 정적으로 site 파일을 만들어도 추가로 다시 수정이 필요하게 됩니다.
https://github.com/tks-management/hanacard-mydata/blob/28ab388c9ada8213ea4842497449f3664f8f77df/hanacard-mydata-reference/lma/site-values.yaml#L227

@robertchoi80
Copy link
Contributor Author

prepare-etcd-secret 의 경우 계속 에러가 발생하고 있어 아래 issue 해결 후 다시 추가해야할 것 같습니다. openinfradev/tks-issues#39

이 부분 때문에 prepare-etcd-secret 호출하는 부분이 제거되었는데 TKS (cluster-api) 환경에서는 정상 동작하니 그대로 두는게 좋을 듯 합니다.

아래는 하나카드 환경 준비하느라 구성한 site 파일인데 client_certfile, client_keyfile 파일 이름이 kubespray에서는 실제 노드 이름으로 설정되기 때문에 환경에 따라 달라질 수 밖에 없어 정적으로 site 파일을 만들어도 추가로 다시 수정이 필요하게 됩니다. https://github.com/tks-management/hanacard-mydata/blob/28ab388c9ada8213ea4842497449f3664f8f77df/hanacard-mydata-reference/lma/site-values.yaml#L227

prepare-etcd-secret 다시 복원했습니다. site-file 구성 부분은 주신 의견 참고하여 좋은 방법이 있을지 고민해보겠습니다.

@robertchoi80
Copy link
Contributor Author

prepare-etcd-secret 의 경우 계속 에러가 발생하고 있어 아래 issue 해결 후 다시 추가해야할 것 같습니다. openinfradev/tks-issues#39

이 부분 때문에 prepare-etcd-secret 호출하는 부분이 제거되었는데 TKS (cluster-api) 환경에서는 정상 동작하니 그대로 두는게 좋을 듯 합니다.
아래는 하나카드 환경 준비하느라 구성한 site 파일인데 client_certfile, client_keyfile 파일 이름이 kubespray에서는 실제 노드 이름으로 설정되기 때문에 환경에 따라 달라질 수 밖에 없어 정적으로 site 파일을 만들어도 추가로 다시 수정이 필요하게 됩니다. https://github.com/tks-management/hanacard-mydata/blob/28ab388c9ada8213ea4842497449f3664f8f77df/hanacard-mydata-reference/lma/site-values.yaml#L227

prepare-etcd-secret 다시 복원했습니다. site-file 구성 부분은 주신 의견 참고하여 좋은 방법이 있을지 고민해보겠습니다.

@zugwan 혹시 TKS 로 설치할 때는 cert 및 key 파일명이 어떻게 정해지나요??

@zugwan
Copy link
Contributor

zugwan commented Jan 25, 2022

@zugwan 혹시 TKS 로 설치할 때는 cert 및 key 파일명이 어떻게 정해지나요??

차트 기본 값 내용대로 구성됩니다.
https://github.com/openinfradev/helm-charts/blob/ebd8d2bf5518b6fe17ae22920d276889c3cb142f/prepare-etcd-secret/values.yaml#L4

@zugwan zugwan merged commit f0923a7 into openinfradev:main Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants