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

add WF to install tks-admin-tools #254

Merged
merged 10 commits into from
Oct 17, 2023
Merged

add WF to install tks-admin-tools #254

merged 10 commits into from
Oct 17, 2023

Conversation

robertchoi80
Copy link
Contributor

@robertchoi80 robertchoi80 commented Sep 14, 2023

https://github.com/openinfradev/tks-issues/issues/861

여기서 사용하는 차트 정의는 다음 PR에 있습니다.
openinfradev/decapod-base-yaml#264

@cho4036 님~ 대략 이런 식으로 workflow 틀 잡았으니 config-keycloak stage 내용 구현해주시면 됩니다. 이 feature 브랜치에서 같이 작업하시면 될것 같고, 혹시 논의 필요한 사항은 말씀해주세요.

본 PR이 머지되면 아래 PR도 리뷰 부탁드립니다.
openinfradev/tks-mgmt-cluster-deploy#32

@cho4036
Copy link
Contributor

cho4036 commented Sep 20, 2023

현재 server_url 변수에 suffix로 '/'가 포함되어 있지 않으면 오류가 나는 현상 확인했습니다.
수정 후 추가 Commit 예정입니다.

@robertchoi80 robertchoi80 requested a review from zugwan October 12, 2023 02:36
@robertchoi80 robertchoi80 changed the title [WIP] add WF to install tks-admin-tools add WF to install tks-admin-tools Oct 12, 2023
@cho4036
Copy link
Contributor

cho4036 commented Oct 12, 2023

말씀주신 ssl verification 에러 핸들링에 대한 커밋 하였습니다.

@zugwan zugwan merged commit 90dc392 into develop Oct 17, 2023
3 checks passed
@zugwan zugwan deleted the tks-issues-861 branch October 17, 2023 10:18
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.

4 participants