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 git rev #13

Merged
merged 2 commits into from
Apr 21, 2022
Merged

Add git rev #13

merged 2 commits into from
Apr 21, 2022

Conversation

zugwan
Copy link
Contributor

@zugwan zugwan commented Apr 20, 2022

git 리비전 옵션을 추가하고 'release-v2' 브랜치 사용하도록 변경하였습니다.

@robertchoi80
Copy link
Contributor

현재 template 버전의 경우 --git-rev 에 브랜치든 태그든 자유롭게 지정할 수 있을 것으로 보이는데, non-template 파일의 경우는 release-v2 로 고정하셨는데, 이 부분 관련하여 2가지 질문 드립니다.

  1. 브랜치로 지정하는 것이 충분할지 좀 고민입니다. 혹시 rc1 -> rc2 간의 차이가 유의미해진다면 Tag로 구분해야하지 않을지..
  2. (만약 tag base로 하게 된다면) 매번 수동으로 수정하지 않고, promote action 등에서 자동으로 치환하게 만드는 건 어떻게 생각하시나요?

@zugwan
Copy link
Contributor Author

zugwan commented Apr 20, 2022

현재 template 버전의 경우 --git-rev 에 브랜치든 태그든 자유롭게 지정할 수 있을 것으로 보이는데, non-template 파일의 경우는 release-v2 로 고정하셨는데, 이 부분 관련하여 2가지 질문 드립니다.

1. 브랜치로 지정하는 것이 충분할지 좀 고민입니다.  혹시 rc1 -> rc2 간의 차이가 유의미해진다면 Tag로 구분해야하지 않을지..

2. (만약 tag base로 하게 된다면) 매번 수동으로 수정하지 않고, promote action 등에서 자동으로 치환하게 만드는 건 어떻게 생각하시나요?

말씀하신 것처럼 태그로 구분하는게 더 바람직할 수 있습니다.
https://github.com/openinfradev/tks-release/blob/573078bae3c6ece6f645d73785134246b02adf94/.github/workflows/create_tag.yml#L45 부분에 repo_name이 decapod-bootstrap인 경우 해당 태그에 맞추어서 generate_yaml.sh을 실행하도록 해서 반영하면 될 듯 합니다. (git tag 전에 이루어져야함)

@robertchoi80
Copy link
Contributor

robertchoi80 commented Apr 20, 2022

현재 template 버전의 경우 --git-rev 에 브랜치든 태그든 자유롭게 지정할 수 있을 것으로 보이는데, non-template 파일의 경우는 release-v2 로 고정하셨는데, 이 부분 관련하여 2가지 질문 드립니다.

1. 브랜치로 지정하는 것이 충분할지 좀 고민입니다.  혹시 rc1 -> rc2 간의 차이가 유의미해진다면 Tag로 구분해야하지 않을지..

2. (만약 tag base로 하게 된다면) 매번 수동으로 수정하지 않고, promote action 등에서 자동으로 치환하게 만드는 건 어떻게 생각하시나요?

말씀하신 것처럼 태그로 구분하는게 더 바람직할 수 있습니다. https://github.com/openinfradev/tks-release/blob/573078bae3c6ece6f645d73785134246b02adf94/.github/workflows/create_tag.yml#L45 부분에 repo_name이 decapod-bootstrap인 경우 해당 태그에 맞추어서 generate_yaml.sh을 실행하도록 해서 반영하면 될 듯 합니다. (git tag 전에 이루어져야함)

네, 그렇게 하면 좋을 것 같습니다! 그 부분 자동화는 해당 action 에서 수행하면 될테고, 다음번 Tag 딸 때 자동으로 해당 tag로 치환될 것이니, 지금 당장은 release-v2로 충분한 듯 합니다.

@kyuho24
Copy link

kyuho24 commented Apr 20, 2022

Test 방법
release-v2 기준 tks-mgmt-cluster-deploy의 ./01_prepare-asshet.sh 중, Decapod bootstrap reop git clon 부분 Branch 설정 변경 후 스크립트 다시 실행...본 테스트의 경우 K3S까지 깔린상태라.. 01 실행후 바로 05실행

103 git clone --quiet https://github.com/openinfradev/decapod-bootstrap $ASSETS_DIR/decapod-bootstrap -b add_git_rev

하지만 아래와 같은 에러발생.. 2번 시도에 2번 발생

error: timed out waiting for the condition on pods/argo-cd-argocd-application-controller-578f5f84f5-q447m

파드상태 이상함. 뭔가가 꼬여서 파드 안뜨는 듯 함. 짤 뜨던 argo-cd도 안됨요

$ kubectl get pod -n argo --kubeconfig kubeconfig_tks-test-cluster
NAME                                                           READY   STATUS              RESTARTS   AGE
argo-cd-argocd-application-controller-578f5f84f5-wtgvs         0/1     Running             0          4m52s
argo-cd-argocd-dex-server-5d6559bb68-pbhbq                     0/1     PodInitializing     0          4m52s
argo-cd-argocd-redis-57fff7645b-9sqd2                          1/1     Running             0          4m52s
argo-cd-argocd-repo-server-df888f964-t86hr                     0/1     PodInitializing     0          4m52s
argo-cd-argocd-server-697bf74494-tr7ll                         0/1     Running             0          4m52s
argo-workflows-operator-server-6b449485dd-h89rh                0/1     ContainerCreating   0          4m52s
argo-workflows-operator-workflow-controller-76848bdd45-68t9l   1/1     Running             0          4m52s

bootstrap server ssh

ssh -i v2-test.pem [email protected]

@ktkfree ktkfree merged commit 2b47d33 into release-v2 Apr 21, 2022
@ktkfree ktkfree deleted the add_git_rev branch April 21, 2022 01:12
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