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

servicemesh refacotring #88

Merged
merged 1 commit into from
Jul 6, 2021

Conversation

seungkyua
Copy link
Contributor

@seungkyua seungkyua commented Jun 30, 2021

helm-chart 가 먼저 수정되어야 함.
관련 PR : [openinfradev/helm-charts/pull/48]

Istio operator chart version: 1.10.2
Kiali operator chart version: 1.34.0
Jaeger operator chart version: 2.20.0

decapod-site 와 servicemesh workflow 도 같이 수정되어야 함

@Jaesang
Copy link
Contributor

Jaesang commented Jul 1, 2021

decapod v1 기준으로 릴리즈 딸때 다 pod running까지만 확인했지, 실제 서비스 메시 구성은 보지 않았는데요
v1에서 설치되는 service-mesh도 문제가 있다면 해당 release-v21.06에도 패치가 되야할거 같습니다.

@estherk0
Copy link
Contributor

estherk0 commented Jul 1, 2021

@seungkyua conflict resolving부탁드립니다~

@seungkyua
Copy link
Contributor Author

@Jaesang release-1.0과 release-v21.06 의 차이는 뭘까요?

@Jaesang
Copy link
Contributor

Jaesang commented Jul 2, 2021

@seungkyua release-v21.06의 source 브랜치가 release-1.0 입니다. (tacoplay부터 decapod까지 같은 이름의 릴리즈 브랜치가 release-v21.06) 릴리즈 위키에 repo별 source 브랜치를 적어놨습니다.(https://tde.sktelecom.com/wiki/display/TACOVER/2021.06+release)

@github-actions
Copy link

github-actions bot commented Jul 5, 2021

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 label Jul 5, 2021
@seungkyua seungkyua removed the Stale label Jul 5, 2021
@estherk0 estherk0 merged commit c40ec82 into openinfradev:main Jul 6, 2021
@seungkyua seungkyua deleted the servicemesh_refactoring branch October 1, 2021 02:50
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.

None yet

3 participants