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 README #43

Merged
merged 3 commits into from
Apr 15, 2022
Merged

update README #43

merged 3 commits into from
Apr 15, 2022

Conversation

robertchoi80
Copy link
Contributor

@robertchoi80 robertchoi80 commented Apr 12, 2022

README 파일 업데이트합니다.
( 재상님 알려주신 렌더링 사이트: https://stackedit.io/app# )

@robertchoi80 robertchoi80 changed the title [WIP] update README update README Apr 12, 2022
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@robertchoi80 robertchoi80 requested a review from ktkfree April 14, 2022 01:01
@ktkfree
Copy link
Contributor

ktkfree commented Apr 14, 2022

생각해보니...
cmd/server/main.go 는 빼서 새로운 PR로 만드는게 좋을 것 같습니다.

helm-charts 와 동시에 수정되어야 하는데, 이 부분이 지금 merge 되면 helm-repo 와 불일치로 문제가 생길 것 같네요.

참고로 helm-repo 는 v2.0.0 tag 기준으로 packaging 되어 있는 상태입니다.
. openinfradev/helm-charts#121

@robertchoi80
Copy link
Contributor Author

생각해보니... cmd/server/main.go 는 빼서 새로운 PR로 만드는게 좋을 것 같습니다.

helm-charts 와 동시에 수정되어야 하는데, 이 부분이 지금 merge 되면 helm-repo 와 불일치로 문제가 생길 것 같네요.

참고로 helm-repo 는 v2.0.0 tag 기준으로 packaging 되어 있는 상태입니다. . openinfradev/helm-charts#121

아 helm-chart 에 token 입력 부분이 있는 건가요?

@robertchoi80
Copy link
Contributor Author

@ktkfree helm-chart에서 token을 환경변수로 mount하는 부분 때문에 그러시죠? 근데 차트에서는 그대로 mount해도, lcm 코드 내에서 사용을 안하는 것 뿐이니 동작에는 지장이 없을것 같습니다.

@ktkfree
Copy link
Contributor

ktkfree commented Apr 14, 2022

@ktkfree helm-chart에서 token을 환경변수로 mount하는 부분 때문에 그러시죠? 근데 차트에서는 그대로 mount해도, lcm 코드 내에서 사용을 안하는 것 뿐이니 동작에는 지장이 없을것 같습니다.

아 파라미터가 아닌 환경 변수이므로 지장 없겠네요. 파라미터로 착각했습니다~

@ktkfree ktkfree merged commit 94b6906 into release-v2 Apr 15, 2022
@ktkfree ktkfree deleted the update_readme branch April 15, 2022 01:36
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.

2 participants