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

Istio 구성 #6

Merged
merged 6 commits into from
Jul 20, 2022
Merged

Istio 구성 #6

merged 6 commits into from
Jul 20, 2022

Conversation

DPS0340
Copy link
Member

@DPS0340 DPS0340 commented Jul 20, 2022

Istio 구성

Tasks

  • Istio 관련 차트 pull후 untar 작업 후 chart 내에 반영

Discussion

  • Argo CD 기반으로 빠르게 배포에 대한 피드백이 필요할 것 같습니다!

Jira

  • SO1S-172

Copy link
Member

@NEM-NE NEM-NE left a comment

Choose a reason for hiding this comment

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

역시 빠르게 구성하셨네요 👍

파일 구조 관해서 건의드리고 싶은 것은 ./charts/istio라는 폴더 안에 지금 올린 파일들을 넣어주는 건 어떨까요? 아니면 지금 monitoring 처럼 하위 chart로 관리해도 좋을 거 같습니다!

추가적으로 tolerations 설정 반영해서 커스텀 values.yaml 파일도 ./app/app-istio/dev-values.yaml로 추가해주시면 좋을 거 같네요!

@@ -0,0 +1,148 @@
# Istio Gateway Helm Chart
Copy link
Member

Choose a reason for hiding this comment

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

README는 삭제하면 좋을 거 같네요

Copy link
Member Author

Choose a reason for hiding this comment

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

넵!

Comment on lines 179 to 181
"tolerations": {
"type": "array"
},
Copy link
Member

Choose a reason for hiding this comment

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

나중에 커스텀 values.yaml 부분 작성하실 때 api노드로만 스케줄링 되도록 설정해주세요!

Copy link
Member Author

Choose a reason for hiding this comment

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

넵 수정하겠습니다!

@@ -0,0 +1,21 @@
# Istio base Helm Chart
Copy link
Member

Choose a reason for hiding this comment

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

여기 README도 삭제 부탁드립니다~

Copy link
Member Author

Choose a reason for hiding this comment

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

네넵!

Comment on lines 59 to 65
resources:
requests:
cpu: 100m
memory: 128Mi
limits:
cpu: 2000m
memory: 1024Mi
Copy link
Member

Choose a reason for hiding this comment

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

istio도 전체적으로 리소스를 얼마나 잡아먹을까요? 과부하가 발생할 일이 생길까요?
왜 불안하게 게이트웨이 limit을 저렇게 높게 잡을까요 🧐

Copy link
Member Author

Choose a reason for hiding this comment

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

api gateway가 모든 외부 리퀘스트가 한번씩 거쳐가기 때문에 어느정도 limit가 높을 필요가 있을 것 같아요!

@@ -0,0 +1,59 @@
# Istiod Helm Chart
Copy link
Member

Choose a reason for hiding this comment

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

.

Comment on lines 68 to 70
{{- with .Values.pilot.tolerations }}
tolerations:
{{- toYaml . | nindent 8 }}
Copy link
Member

Choose a reason for hiding this comment

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

여기도 tolerations 설정 해줘야겠네요

Copy link
Member Author

Choose a reason for hiding this comment

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

넵!! 기존 차트 레퍼런스로 반영하겠습니다!

@shinilseop
Copy link
Contributor

고생하셨습니다!!

@NEM-NE NEM-NE added Feat 기능이 추가됐습니다. Structure 프로젝트 구조가 변경됐습니다. (삭제 및 생성) labels Jul 20, 2022
@DPS0340
Copy link
Member Author

DPS0340 commented Jul 20, 2022

@NEM-NE 수정사항 반영하였습니다! 확인해 주세요~~

@NEM-NE NEM-NE merged commit c1608e4 into main Jul 20, 2022
@DPS0340 DPS0340 deleted the SO1S-172-istio-구성 branch December 6, 2022 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 기능이 추가됐습니다. Structure 프로젝트 구조가 변경됐습니다. (삭제 및 생성)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants