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

updating decapod deploying part of document #4

Merged
merged 3 commits into from
Aug 18, 2021

Conversation

ktkfree
Copy link
Contributor

@ktkfree ktkfree commented Aug 10, 2021

일단, decapod deploying 에 대한 부분을 현행화 하였습니다.

검토 부탁 드립니다.

docs/getting-started/deploying.md Outdated Show resolved Hide resolved
docs/getting-started/deploying.md Outdated Show resolved Hide resolved
docs/getting-started/deploying.md Show resolved Hide resolved
@ktkfree
Copy link
Contributor Author

ktkfree commented Aug 10, 2021

LMA 설치를 위해서는 메뉴얼의 내용뿐만아니라 ( 맞는 step 인지는 모르겠어서 추가하지 않았습니다. ) 아래의 내용도 필요한것 같아요.
아래 스텝을 개발 중으로 알고 있는데, 아래 내용 중 메뉴얼화 해야 할 만한 내용이 있다면 코멘트 바랍니다.

taco-storage 에러
. pv와 storageClass 를 taco-storage로 수동 추가

cluster name
. argo-cd cluster 이름을 사이트명과 동일하게 UI에서 변경 ( 예. hanu-reference로 )

namespace 생성
. kubectl create ns lma

노드 label
. kubectl label nodes kind-control-plane taco-lma=enabled

@Jaesang
Copy link

Jaesang commented Aug 10, 2021

LMA 설치를 위해서는 메뉴얼의 내용뿐만아니라 ( 맞는 step 인지는 모르겠어서 추가하지 않았습니다. ) 아래의 내용도 필요한것 같아요.
아래 스텝을 개발 중으로 알고 있는데, 아래 내용 중 메뉴얼화 해야 할 만한 내용이 있다면 코멘트 바랍니다.

taco-storage 에러
. pv와 storageClass 를 taco-storage로 수동 추가

cluster name
. argo-cd cluster 이름을 사이트명과 동일하게 UI에서 변경 ( 예. hanu-reference로 )

namespace 생성
. kubectl create ns lma

노드 label
. kubectl label nodes kind-control-plane taco-lma=enabled

이 부분은 Decapod보다는 LMA 특화된 부분이라 이 문서에서 다룰 필요는 없지 않을까합니다

@Jaesang Jaesang self-requested a review August 10, 2021 13:50
```

2. lma 배포
```sh
argo submit -n argo --from wftmpl/lma-federation \
-p site_name="사이트명" # decapod-manifests의 사이트 디렉토리명과 일치해야한다. \
-p site_name="싸이트명" # decapod-manifests의 사이트 디렉토리명과 반드시 일치해야한다. \
-p app_name="lma" \
-p repository_url="https://github.com/openinfradev/decapod-manifests" # decapod-manifests repository 주소
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

openinfradev가 아닌 "YOUR_REPOSITORY_NAME" 으로 바뀌어야할 것 같습니다.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ktkfree 업데이트해주세요~

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@robertchoi80 @Jaesang 업데이트는 했는데, github 인증방식이 8/13에 변경되면서 push 를 못하고 있었어요. 이부분 확인후 push 하도록 하겠습니다. 감사합니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

변경했습니다~

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

앗! 이거 엉뚱한 부분이 수정되었는데요? site_name 이 아니라 repository_url 의 'openinfradev' 부분을 말씀드린 것입니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

앗 수정부분이 저걸로 떠서 무의식적으로 site name을 수정해버렸네요.
수정 pr 올릴게요. 감사합니다

@bluejayA bluejayA changed the title updating document updating decapod deploying part of document Aug 18, 2021
@bluejayA bluejayA merged commit 99739f1 into openinfradev:main Aug 18, 2021
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