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

update service-mesh site-values #69

Merged
merged 2 commits into from
Jan 13, 2022
Merged

update service-mesh site-values #69

merged 2 commits into from
Jan 13, 2022

Conversation

robertchoi80
Copy link
Contributor

@robertchoi80 robertchoi80 commented Jan 4, 2022

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

hanu-reference template을 tks 쪽에서 함께 사용하기로 했으므로, service-mesh ingress 생성 부분을 false로 변경합니다.

@github-actions
Copy link

github-actions bot commented Jan 7, 2022

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.

@github-actions github-actions bot added the Stale There has been no activity on this label Jan 7, 2022
@github-actions
Copy link

This PR was closed because it has been stalled for 10 days with no activity.

@github-actions github-actions bot closed this Jan 10, 2022
@bluejayA bluejayA reopened this Jan 10, 2022
to sync with tks-custom-base
@ktkfree
Copy link
Contributor

ktkfree commented Jan 13, 2022

tks-cluster -> tks-cluster-aws 로 폴더명이 변경되었는데, 의도하신 변경인지요

@robertchoi80
Copy link
Contributor Author

robertchoi80 commented Jan 13, 2022

tks-cluster -> tks-cluster-aws 로 폴더명이 변경되었는데, 의도하신 변경인지요

네, 의도한 변경입니다. decapod-base (또는 tks-custom-base) 와 site 를 머지하는 과정에서 모든 app 간에 디렉토리 구조와 depth가 통일되어야겠더라구요

@robertchoi80 robertchoi80 merged commit ed12ea2 into main Jan 13, 2022
@robertchoi80 robertchoi80 deleted the update-service-mesh branch January 13, 2022 04:59
@seungkyua seungkyua removed the Stale There has been no activity on this label Jan 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants