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

Disable resource requests in Helm CI runs #2735

Merged
merged 1 commit into from
Aug 17, 2022

Conversation

dimitarvdimitrov
Copy link
Contributor

@dimitarvdimitrov dimitarvdimitrov commented Aug 16, 2022

Signed-off-by: Dimitar Dimitrov dimitar.dimitrov@grafana.com

What this PR does

During the development of #2087 helm CI runs (call-lint-test / lint-test) started failing because pods could not be scheduled (example). The info at the end of the run which describes all pods showed the following for two pods:

Events:
  Type     Reason            Age                 From               Message
  ----     ------            ----                ----               -------
  Warning  FailedScheduling  11s (x13 over 10m)  default-scheduler  0/1 nodes are available: 1 Insufficient cpu.

GitHub hosted runners only have 2 CPUs and 7GB of memory (docs).

This PR removes resource limits and requirements for some pods in the CI values files for Helm. This way Kubernetes won't have to calculate resource quotas. The caveat is that now CI runs may become less stable because of resource contention.

The changes here were cherry-picked onto the work in #2087 to demonstrate that they unblock the CI runs - example run

An alternative approach is to install self-hosted runners with more available CPU and memory. A second alternative is to migrate the CI runs to drone, for which we have bigger agents.

Signed-off-by: Dimitar Dimitrov <dimitar.dimitrov@grafana.com>
@krajorama krajorama merged commit 6531704 into main Aug 17, 2022
@krajorama krajorama deleted the dimitar/disable-resource-requests-in-helm-ci branch August 17, 2022 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants