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

feature. add tks-portal #155

Merged
merged 1 commit into from
Mar 22, 2022
Merged

feature. add tks-portal #155

merged 1 commit into from
Mar 22, 2022

Conversation

ktkfree
Copy link
Contributor

@ktkfree ktkfree commented Mar 21, 2022

tks-portal 을 위한 base yaml 을 추가합니다.
테스트를 위해 테스트 정보를 사용하였습니다.

@gitguardian
Copy link

gitguardian bot commented Mar 21, 2022

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!

@ktkfree ktkfree force-pushed the add_tks_portal branch 3 times, most recently from ce225b0 to 26bf9b5 Compare March 21, 2022 08:32
@zugwan
Copy link
Contributor

zugwan commented Mar 21, 2022

테스트하신 값을 기본 값으로 사용하기 보다 URL이나 암호 등은 좀 더 일반적인 형식으로 채워서 반영하는 것이 좋을 듯 합니다.

@zugwan
Copy link
Contributor

zugwan commented Mar 21, 2022

그리고 servicemesh portal은 decapod-base가 아니라 tks-custom에 추가해야 하는 것 아닌지 문의 드립니다.

@ktkfree ktkfree changed the title feature. add tks-portal [WIP] feature. add tks-portal Mar 21, 2022
@ktkfree
Copy link
Contributor Author

ktkfree commented Mar 21, 2022

그리고 servicemesh portal은 decapod-base가 아니라 tks-custom에 추가해야 하는 것 아닌지 문의 드립니다.

[WIP] tag 로 변경하였습니다. 아직 portal 이 잘뜨지 않아서... 현재 테스트 value 기준으로 일단 커밋된 상태입니다. 값은 조정하겠습니다.

오늘 오전에 논의된 내용이었고, servicemesh-portal 도 decapod 기반으로 일단 올리자는 결론이었습니다.
tks-custom-base-yaml, tks-user-site-template 쪽에 처음 PR 올렸다가 decapod 쪽으로 변경한 내용입니다.

@zugwan
Copy link
Contributor

zugwan commented Mar 21, 2022

그리고 servicemesh portal은 decapod-base가 아니라 tks-custom에 추가해야 하는 것 아닌지 문의 드립니다.

[WIP] tag 로 변경하였습니다. 아직 portal 이 잘뜨지 않아서... 현재 테스트 value 기준으로 일단 커밋된 상태입니다. 값은 조정하겠습니다.

오늘 오전에 논의된 내용이었고, servicemesh-portal 도 decapod 기반으로 일단 올리자는 결론이었습니다. tks-custom-base-yaml, tks-user-site-template 쪽에 처음 PR 올렸다가 decapod 쪽으로 변경한 내용입니다.

네, 알겠습니다.

@ktkfree ktkfree changed the title [WIP] feature. add tks-portal feature. add tks-portal Mar 22, 2022
@seungkyua seungkyua merged commit 4249f62 into main Mar 22, 2022
@seungkyua seungkyua deleted the add_tks_portal branch March 22, 2022 05:48
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.

3 participants