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

Fix. enables download tks image #87

Merged
merged 4 commits into from
Jan 7, 2022
Merged

Fix. enables download tks image #87

merged 4 commits into from
Jan 7, 2022

Conversation

ktkfree
Copy link
Contributor

@ktkfree ktkfree commented Dec 21, 2021

TKS 최신 버전의 이미지를 download 하기 위해 chart 를 수정합니다.

  • values.yaml 에서 동적인 function string value를 전달하는 방법을 모르겠기도 하고, chart 자체의 수정이라고 보고 chart 를 직접 수정하였습니다.

tks-apis 를 변경후, admin cluster 에서 helm upgrade 를 실행해야 latest 이미지가 deploy 됩니다.

admin cluster 는 아직 gitops 처리가 되어 있지 않기에, 추후 자동 deploy 기능을 개발합니다.

@github-actions
Copy link

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 Dec 24, 2021
@Jaesang
Copy link
Contributor

Jaesang commented Dec 27, 2021

Imagepullpolicy가 Always이면 늘 최신 이미지를 다운받는것 아닌가요?

@Jaesang Jaesang removed the Stale There has been no activity on this label Dec 27, 2021
@ktkfree
Copy link
Contributor Author

ktkfree commented Dec 27, 2021

Imagepullpolicy가 Always이면 늘 최신 이미지를 다운받는것 아닌가요?

Imagepullpolicy가 always라고 하더라도 머신에 이미 받아놓은 동일버전의 이미지가 있다면 새로운 이미지를 받지 않더라구요

@Jaesang
Copy link
Contributor

Jaesang commented Dec 27, 2021

Imagepullpolicy가 Always이면 늘 최신 이미지를 다운받는것 아닌가요?

Imagepullpolicy가 always라고 하더라도 머신에 이미 받아놓은 동일버전의 이미지가 있다면 새로운 이미지를 받지 않더라구요

동일한 tag라도 Image의 sha값이 다르면 받아오기때문에 원하시는 동작이 이루어집니다. deployment의 latest태그 사용해서 테스트해봤어요

@ktkfree
Copy link
Contributor Author

ktkfree commented Dec 28, 2021

Imagepullpolicy가 Always이면 늘 최신 이미지를 다운받는것 아닌가요?

Imagepullpolicy가 always라고 하더라도 머신에 이미 받아놓은 동일버전의 이미지가 있다면 새로운 이미지를 받지 않더라구요

동일한 tag라도 Image의 sha값이 다르면 받아오기때문에 원하시는 동작이 이루어집니다. deployment의 latest태그 사용해서 테스트해봤어요

혹시 어떻게 테스트하셨는지 알 수 있을까요? 아래와 같이 테스트 했을때 저는 최신 이미지로 반영되지 않는 것 같아요.
. tks-admin-v2 cluster >> cd ~/tks/tmp/helm-charts/tks-info
. deployment 에서 rollme annotation 삭제 확인
. image 를 local에서 build 후 push ( ktkfree/tks-info:latest )
. helm upgrade -n tks tks-info -f values.yaml .

@Jaesang
Copy link
Contributor

Jaesang commented Dec 28, 2021

Imagepullpolicy가 Always이면 늘 최신 이미지를 다운받는것 아닌가요?

Imagepullpolicy가 always라고 하더라도 머신에 이미 받아놓은 동일버전의 이미지가 있다면 새로운 이미지를 받지 않더라구요

동일한 tag라도 Image의 sha값이 다르면 받아오기때문에 원하시는 동작이 이루어집니다. deployment의 latest태그 사용해서 테스트해봤어요

혹시 어떻게 테스트하셨는지 알 수 있을까요? 아래와 같이 테스트 했을때 저는 최신 이미지로 반영되지 않는 것 같아요. . tks-admin-v2 cluster >> cd ~/tks/tmp/helm-charts/tks-info . deployment 에서 rollme annotation 삭제 확인 . image 를 local에서 build 후 push ( ktkfree/tks-info:latest ) . helm upgrade -n tks tks-info -f values.yaml .

helm upgrade에서는 values값의 변화가 없기 때문에 helm에서 아무일도 하지 않을것이고,
저는 그냥 pod삭제했습니다. 그럼 다시 만들어지면서 이미지가 갱신되죠.

@ktkfree
Copy link
Contributor Author

ktkfree commented Dec 30, 2021

Imagepullpolicy가 Always이면 늘 최신 이미지를 다운받는것 아닌가요?

Imagepullpolicy가 always라고 하더라도 머신에 이미 받아놓은 동일버전의 이미지가 있다면 새로운 이미지를 받지 않더라구요

동일한 tag라도 Image의 sha값이 다르면 받아오기때문에 원하시는 동작이 이루어집니다. deployment의 latest태그 사용해서 테스트해봤어요

혹시 어떻게 테스트하셨는지 알 수 있을까요? 아래와 같이 테스트 했을때 저는 최신 이미지로 반영되지 않는 것 같아요. . tks-admin-v2 cluster >> cd ~/tks/tmp/helm-charts/tks-info . deployment 에서 rollme annotation 삭제 확인 . image 를 local에서 build 후 push ( ktkfree/tks-info:latest ) . helm upgrade -n tks tks-info -f values.yaml .

helm upgrade에서는 values값의 변화가 없기 때문에 helm에서 아무일도 하지 않을것이고, 저는 그냥 pod삭제했습니다. 그럼 다시 만들어지면서 이미지가 갱신되죠.

아, helm upgrade 시 latest 이미지 갱신을 위해 이 작업을 했었어요.
생각해보니 그냥 delete pod 을 deploy 액션이라고 간주해도 뭐... 일단은 무리 없을 것 같습니다. 이 부분은 한번 고민해보고 차주에 얘기 나눠보시죠~

@github-actions
Copy link

github-actions bot commented Jan 2, 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 2, 2022
@zugwan zugwan removed the Stale There has been no activity on this label Jan 3, 2022
@ktkfree
Copy link
Contributor Author

ktkfree commented Jan 3, 2022

deployment 의 annotation 을 롤백하였고, image 저장소만 sktcloud 로 변경하였습니다. tks-apis deployment는 추후 gitops 구축시 진행하면 좋겠네요.

@ktkfree ktkfree merged commit e7f055e into openinfradev:main Jan 7, 2022
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