harness / gitness

Gitness is an Open Source developer platform with Source Control management, Continuous Integration and Continuous Delivery.
https://gitness.com
Apache License 2.0
32.09k stars 2.8k forks source link

Feature: Helm kubernetes Deployment #3417

Closed alexohneander closed 10 months ago

alexohneander commented 10 months ago

Added helm deployment so you can quickly deploy it to your Kubernetes cluster.

The deployment consists of:

CLAassistant commented 10 months ago

CLA assistant check
All committers have signed the CLA.

alexohneander commented 10 months ago

@willzhang @arya-harness @abhinav-harness Accidentally closed the pull request during the last commit and could not reopen it. It is the same request as #3397 with the additional env vars.

willzhang commented 10 months ago

I found that the Docker Dind section has been deleted. Is this not needed, or is there any other solution?

docker:
  enabled: true
  image: docker:24.0.3-dind
  resources: {}
    # limits:
    #   cpu: 100m
    #   memory: 128Mi
    # requests:
    #   cpu: 100m
    #   memory: 128Mi

or can we mount host docker.sock or containerd.sock ? Uncertain whether gitness supports containerd

host:
  docker:
    sock: /var/run/docker.sock
  containerd:
    sock: /var/run/containerd/containerd.sock
  crio:
    sock: var/run/crio/crio.sock
volumes:

Perhaps in the k8s environment, gigness directly communicates with the k8s API and runs jobs by creating multiple worke pods?

or use drone-runner https://github.com/drone-runners/drone-runner-kube

bradrydzewski commented 10 months ago

thanks @alexohneander for the contribution!

@willzhang good point, we would want a docker:dind container added for pipeline execution, similar to what we have in the official Drone help chart. See https://github.com/drone/charts/blob/master/charts/drone-runner-docker/templates/deployment.yaml#L40

adding @jimsheldon as a reviewer (he is the creator of the Drone chart)

alexohneander commented 10 months ago

@willzhang @bradrydzewski I made a mistake yesterday. We still need the dind and that definitely has to go back in. I'll adjust it.

alexohneander commented 10 months ago

Is there anything missing or can I tweak, I would love to do that. 😃

Lite5h4dow commented 3 months ago

there is no index.yaml file for using these templates as a helm chart. opened issue: https://github.com/harness/gitness/issues/3512#issue-2290930581