Skip to content

Commit 16a8e11

Browse files
committed
fix(chart): fix broken helm chart tests
1 parent bab675d commit 16a8e11

File tree

6 files changed

+12
-10
lines changed

6 files changed

+12
-10
lines changed

.github/workflows/operator-ci.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ on:
3030
- containers/operator.Dockerfile
3131
- .github/workflows/operator-ci.yaml
3232
- k8s-tests/**
33+
- chart/**
3334
push:
3435
branches:
3536
- main
@@ -40,6 +41,7 @@ on:
4041
- containers/operator.Dockerfile
4142
- .github/workflows/operator-ci.yaml
4243
- k8s-tests/**
44+
- chart/**
4345

4446
# NOTE: we may want to switch to matrix build for multi-platform support if this is taking too long
4547
# https://docs.docker.com/build/ci/github-actions/multi-platform/#distribute-build-across-multiple-runners

.gitlab-ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -279,8 +279,8 @@ publish-chart:
279279
needs: [bootstrap]
280280
variables:
281281
ENV: prod
282-
REGISTRY: staging
283-
URL: helm.ngc.nvidia.com/staging
282+
REGISTRY: skyhook
283+
URL: helm.ngc.nvidia.com/nvstaging/skyhook
284284
variables:
285285
USERNAME: "$$oauthtoken"
286286
PASSWORD: ${NVCR_REGISTRY_PASSWORD}

chart/templates/deployment.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,8 @@ spec:
105105
cpu: {{ maxf 1000 (mulf 1.6 .Values.estimatedNodeCount (maxf 1 (mulf 0.4 .Values.estimatedPackageCount))) | int }}m
106106
memory: {{ maxf 512 (mulf 0.8 .Values.estimatedNodeCount (maxf 1 (mulf 0.4 .Values.estimatedPackageCount))) | int }}Mi
107107
requests:
108-
cpu: 500m
109-
memory: {{ maxf 256 (mulf 0.45 .Values.estimatedNodeCount) | int }}Mi
108+
cpu: {{ maxf 500 (mulf 0.5 (mulf 1.6 .Values.estimatedNodeCount (maxf 1 (mulf 0.4 .Values.estimatedPackageCount)))) | int }}m
109+
memory: {{ maxf 256 (mulf 0.4 .Values.estimatedNodeCount) | int }}Mi
110110
{{- end }}
111111
securityContext: {{- toYaml .Values.controllerManager.manager.containerSecurityContext
112112
| nindent 10 }}

k8s-tests/chainsaw/helm/helm-chart-test/assert-no-schedule.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,8 +84,8 @@ spec:
8484
(cpu == '1000m' || cpu == '1' || cpu == `1`): true
8585
memory: 512Mi
8686
requests:
87-
cpu: 200m
88-
memory: 64Mi
87+
cpu: 500m
88+
memory: 256Mi
8989
securityContext:
9090
allowPrivilegeEscalation: false
9191
capabilities:

k8s-tests/chainsaw/helm/helm-chart-test/assert-scheduled.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,8 @@ spec:
8383
(cpu == '1000m' || cpu == '1' || cpu == `1`): true
8484
memory: 512Mi
8585
requests:
86-
cpu: 200m
87-
memory: 64Mi
86+
cpu: 500m
87+
memory: 256Mi
8888
securityContext:
8989
allowPrivilegeEscalation: false
9090
capabilities:

k8s-tests/chainsaw/helm/helm-scale-test/assert-scaled-resources.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@ spec:
3838
cpu: 1280m
3939
memory: 640Mi
4040
requests:
41-
cpu: 200m
42-
memory: 180Mi
41+
cpu: 640m
42+
memory: 256Mi
4343
- name: kube-rbac-proxy
4444
resources:
4545
limits:

0 commit comments

Comments
 (0)