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

manifests 리뷰용 PR 생성 / 클러스터 import 후 자동 manifests 생성 #127

Merged
merged 2 commits into from
Dec 9, 2022

Conversation

zugwan
Copy link
Contributor

@zugwan zugwan commented Dec 8, 2022

  • render-manifests: manifests 생성 후 main 브랜치에 바로 반영하는 대신 PR 생성하는 기능을 추가하였습니다: Git 저장소가 gitea 경우에만 한정되어 동작합니다 (if 'manifest_repo_branch' is empty and 'git_repo_type' is gitea)
  • import-cluster: 갱신 된 render-manifests를 호출하여 manifests 생성 및 저장소 반영이 자동으로 이루어지도록 변경하였습니다.

if 'manifest_repo_branch' is empty and 'git_repo_type' is gitea,
a pull request for generated manifests is created instead of pushing
them into main branch.
@zugwan zugwan force-pushed the generate_manifests branch from 56a6fd7 to 9c0cc99 Compare December 8, 2022 11:13
@ktkfree
Copy link
Contributor

ktkfree commented Dec 9, 2022

이 PR 과 연관이 있을 것 같아서 문의 드립니다.
import 시 아래 addons 설치가 필요할까요?
필요하다면 추가 PR 올리겠습니다.

        templateRef:
          name: create-application
          template: installApps
        arguments:
          parameters:
          - name: list
            value: |
              [
                { 
                  "app_group": "tks-cluster",
                  "path": "kubernetes-addons",
                  "namespace": "taco-system",
                  "target_cluster": ""
                },
                {
                  "app_group": "tks-cluster",
                  "path": "ingress-nginx",
                  "namespace": "taco-system",
                  "target_cluster": ""
                },
                {
                  "app_group": "tks-cluster",
                  "path": "metrics-server",
                  "namespace": "kube-system",
                  "target_cluster": ""
                },
                {
                  "app_group": "tks-cluster",
                  "path": "cluster-autoscaler",
                  "namespace": "kube-system",
                  "target_cluster": ""
                },
                {
                  "app_group": "tks-cluster",
                  "path": "kubed",
                  "namespace": "taco-system",
                  "target_cluster": ""
                }
              ]

@seungkyua seungkyua merged commit 5f8ea3f into main Dec 9, 2022
@seungkyua seungkyua deleted the generate_manifests branch December 9, 2022 02:55
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.

3 participants