[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 5/5] Add Kubernetes runner configuration
|
From: |
Daniel P . Berrangé |
|
Subject: |
Re: [PATCH v2 5/5] Add Kubernetes runner configuration |
|
Date: |
Tue, 23 May 2023 09:37:45 +0100 |
|
User-agent: |
Mutt/2.2.9 (2022-11-12) |
On Mon, May 22, 2023 at 06:41:54PM +0100, Camilla Conte wrote:
> Custom values for the gitlab-runner Helm chart.
> See https://wiki.qemu.org/Testing/CI/KubernetesRunners.
>
> Signed-off-by: Camilla Conte <cconte@redhat.com>
> ---
> .../ci/gitlab-kubernetes-runners/values.yaml | 30 +++++++++++++++++++
> 1 file changed, 30 insertions(+)
> create mode 100644 scripts/ci/gitlab-kubernetes-runners/values.yaml
Can't say I'm massively familiar with this, but there's nothing
obviously terrible, so on the basis that you've tested it works
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
>
> diff --git a/scripts/ci/gitlab-kubernetes-runners/values.yaml
> b/scripts/ci/gitlab-kubernetes-runners/values.yaml
> new file mode 100644
> index 0000000000..204a96a842
> --- /dev/null
> +++ b/scripts/ci/gitlab-kubernetes-runners/values.yaml
> @@ -0,0 +1,30 @@
> +gitlabUrl: "https://gitlab.com/"
> +runnerRegistrationToken: ""
> +rbac:
> + create: true
> +concurrent: 200
> +runners:
> + privileged: true
> + config: |
> + [[runners]]
> + limit = 100
> + environment = [
> + "DOCKER_HOST=tcp://docker:2376",
> + "DOCKER_TLS_CERTDIR=/certs",
> + "DOCKER_TLS_VERIFY=1",
> + "DOCKER_CERT_PATH=/certs/client"
> + ]
> + [runners.kubernetes]
> + poll_timeout = 1200
> + image = "ubuntu:20.04"
Is it viable to update to 22.04 whic is the current newest LTS release
stream ?
> + cpu_request = "0.5"
> + service_cpu_request = "0.5"
> + helper_cpu_request = "0.25"
> + cpu_request_overwrite_max_allowed = "7"
> + memory_request_overwrite_max_allowed = "30Gi"
> + [[runners.kubernetes.volumes.empty_dir]]
> + name = "docker-certs"
> + mount_path = "/certs/client"
> + medium = "Memory"
> + [runners.kubernetes.node_selector]
> + agentpool = "jobs"
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [PATCH v2 1/5] Remove redundant CI variables, (continued)
- [PATCH v2 2/5] Use docker "stable" tag, Camilla Conte, 2023/05/22
- [PATCH v2 3/5] Add loop over docker info, Camilla Conte, 2023/05/22
- [PATCH v2 4/5] Add CI variable RUNNER_TAG, Camilla Conte, 2023/05/22
- [PATCH v2 5/5] Add Kubernetes runner configuration, Camilla Conte, 2023/05/22
- Re: [PATCH v2 5/5] Add Kubernetes runner configuration,
Daniel P . Berrangé <=
- Re: Add CI configuration for Kubernetes, Richard Henderson, 2023/05/22
- [PATCH v3 5/5] Add Kubernetes runner configuration, Camilla Conte, 2023/05/23
- Re: Add CI configuration for Kubernetes, Richard Henderson, 2023/05/24