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 refactoring #44

Merged
merged 1 commit into from
Jul 9, 2021

Conversation

seungkyua
Copy link
Contributor

@seungkyua seungkyua commented Jul 1, 2021

  • helm-chart 가 먼저 적용되어야 함.
    관련 PR: [servicemesh chart refactoring helm-charts#48]

  • decapod-base-yaml 이 먼저 적용되어야 함
    관련 PR: [servicemesh refacotring decapod-base-yaml#88]

  • decapod-site 가 먼저 적용되어야 함
    관련 PR: [servicemesh refactoring decapod-site#39]

  • istio-system ns 는 servicemesh-controlplane 차트에서는 생성하지 않고 workflow 에서 생성함 (argocd app 으로 삭제하면 istio-system ns 가 같이 삭제되기 때문)

  • 설치 마지막에 argocd app sync -l app=service-mesh 로 모든 앱에 sync 를 호출함

    • Missing 이나 OutOfSync 로 변하는 app 이 존재하여 sync 를 호출하면 해결됨
    • create-application workflowtemplate 에서 app 생성시에 app=service-mesh 라벨을 추가함
  • 아래 label 을 노드에 추가해야 함 (추후 tacoplay 에 추가 예정)

  1. servicemesh control node
    servicemesh=enabled

  2. servicemesh ingress node
    taco-ingress-gateway=enabled

  3. servicemesh egress node
    taco-egress-gateway=enabled

  • Ingress 와 resources 의 값 cpu: 1 이 cpu:'1' 로 설정되어 outofsync 나는 것을 해결하기 위해 추가해야 함 (추후 tacoplay 에 추가 예정)
apiVersion: v1
data:
  accounts.admin: apiKey, login
  application.instanceLabelKey: argocd.argoproj.io/instance
  resource.customizations: |
    jaegertracing.io/Jaeger:
      ignoreDifferences: |
        jsonPointers:
        - /spec/collector/resources
      health.lua: |
        health_status = {}
        health_status.status = "Healthy"
        health_status.message = "Jaeger Controlplane is Running"
        return health_status
    networking.k8s.io/Ingress:
      health.lua: |
        health_status = {}
        health_status.status = "Healthy"
        health_status.message = "Jaeger Ingress is Running"
        return health_status
  url: https://argocd.example.com
kind: ConfigMap
...

@github-actions
Copy link

github-actions bot commented Jul 4, 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 There has been no activity on this label Jul 4, 2021
@seungkyua seungkyua removed the Stale There has been no activity on this label Jul 4, 2021
@github-actions
Copy link

github-actions bot commented Jul 8, 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 There has been no activity on this label Jul 8, 2021
@estherk0
Copy link
Contributor

estherk0 commented Jul 9, 2021

@seungkyua servicemesh를 설치하기 위해 ingress controller가 필수로 필요한데 ingress controller 설치는 tacoplay쪽에 들어가게 되는걸까요?
아니면 이것도 decapod으로 설치하게 되는걸까요?

@seungkyua
Copy link
Contributor Author

@seungkyua servicemesh를 설치하기 위해 ingress controller가 필수로 필요한데 ingress controller 설치는 tacoplay쪽에 들어가게 되는걸까요?
아니면 이것도 decapod으로 설치하게 되는걸까요?

네. kubespray 에 있기 때문에 tacoplay 에서 설치하면 될 것 같네요.

@estherk0 estherk0 merged commit 15505a7 into openinfradev:main Jul 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale There has been no activity on this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants