From 135980d8afb84f722ea0a15b6fb6c11ea559d7bf Mon Sep 17 00:00:00 2001 From: shreddedbacon Date: Fri, 7 Oct 2022 13:46:03 +1100 Subject: [PATCH 01/11] chore: remove old kubernetes-build-deploy serviceaccount --- charts/lagoon-remote/Chart.yaml | 2 +- charts/lagoon-remote/templates/_helpers.tpl | 39 ------------------- ...netes-build-deploy.clusterrolebinding.yaml | 14 ------- ...ubernetes-build-deploy.serviceaccount.yaml | 10 ----- charts/lagoon-remote/values.yaml | 7 ---- 5 files changed, 1 insertion(+), 71 deletions(-) delete mode 100644 charts/lagoon-remote/templates/kubernetes-build-deploy.clusterrolebinding.yaml delete mode 100644 charts/lagoon-remote/templates/kubernetes-build-deploy.serviceaccount.yaml diff --git a/charts/lagoon-remote/Chart.yaml b/charts/lagoon-remote/Chart.yaml index 21a05c8a..e8aeec2f 100644 --- a/charts/lagoon-remote/Chart.yaml +++ b/charts/lagoon-remote/Chart.yaml @@ -19,7 +19,7 @@ type: application # This is the chart version. This version number should be incremented each # time you make changes to the chart and its templates, including the app # version. -version: 0.63.0 +version: 1.0.0 dependencies: - name: lagoon-build-deploy diff --git a/charts/lagoon-remote/templates/_helpers.tpl b/charts/lagoon-remote/templates/_helpers.tpl index 2597b3f1..32b7d914 100644 --- a/charts/lagoon-remote/templates/_helpers.tpl +++ b/charts/lagoon-remote/templates/_helpers.tpl @@ -83,45 +83,6 @@ app.kubernetes.io/component: {{ include "lagoon-remote.dockerHost.fullname" . }} app.kubernetes.io/instance: {{ .Release.Name }} {{- end }} - - -{{/* -Create the name of the service account to use for kubernetesBuildDeploy. -*/}} -{{- define "lagoon-remote.kubernetesBuildDeploy.serviceAccountName" -}} -{{- default (include "lagoon-remote.kubernetesBuildDeploy.fullname" .) .Values.kubernetesBuildDeploy.serviceAccount.name }} -{{- end }} - -{{/* -Create a default fully qualified app name for kubernetesBuildDeploy. -*/}} -{{- define "lagoon-remote.kubernetesBuildDeploy.fullname" -}} -{{- include "lagoon-remote.fullname" . }}-kubernetes-build-deploy -{{- end }} - -{{/* -Common labels kubernetesBuildDeploy. -*/}} -{{- define "lagoon-remote.kubernetesBuildDeploy.labels" -}} -helm.sh/chart: {{ include "lagoon-remote.chart" . }} -{{ include "lagoon-remote.kubernetesBuildDeploy.selectorLabels" . }} -{{- if .Chart.AppVersion }} -app.kubernetes.io/version: {{ .Chart.AppVersion | quote }} -{{- end }} -app.kubernetes.io/managed-by: {{ .Release.Service }} -{{- end }} - -{{/* -Selector labels kubernetesBuildDeploy. -*/}} -{{- define "lagoon-remote.kubernetesBuildDeploy.selectorLabels" -}} -app.kubernetes.io/name: {{ include "lagoon-remote.name" . }} -app.kubernetes.io/component: {{ include "lagoon-remote.kubernetesBuildDeploy.fullname" . }} -app.kubernetes.io/instance: {{ .Release.Name }} -{{- end }} - - - {{/* Create the name of the service account to use for sshPortal. */}} diff --git a/charts/lagoon-remote/templates/kubernetes-build-deploy.clusterrolebinding.yaml b/charts/lagoon-remote/templates/kubernetes-build-deploy.clusterrolebinding.yaml deleted file mode 100644 index a7e08f51..00000000 --- a/charts/lagoon-remote/templates/kubernetes-build-deploy.clusterrolebinding.yaml +++ /dev/null @@ -1,14 +0,0 @@ -apiVersion: rbac.authorization.k8s.io/v1 -kind: ClusterRoleBinding -metadata: - name: {{ include "lagoon-remote.kubernetesBuildDeploy.fullname" . }} - labels: - {{- include "lagoon-remote.kubernetesBuildDeploy.labels" . | nindent 4 }} -subjects: -- kind: ServiceAccount - name: {{ include "lagoon-remote.kubernetesBuildDeploy.serviceAccountName" . }} - namespace: {{ .Release.Namespace | quote }} -roleRef: - kind: ClusterRole - name: cluster-admin - apiGroup: rbac.authorization.k8s.io diff --git a/charts/lagoon-remote/templates/kubernetes-build-deploy.serviceaccount.yaml b/charts/lagoon-remote/templates/kubernetes-build-deploy.serviceaccount.yaml deleted file mode 100644 index 01136b72..00000000 --- a/charts/lagoon-remote/templates/kubernetes-build-deploy.serviceaccount.yaml +++ /dev/null @@ -1,10 +0,0 @@ -apiVersion: v1 -kind: ServiceAccount -metadata: - name: {{ include "lagoon-remote.kubernetesBuildDeploy.serviceAccountName" . }} - labels: - {{- include "lagoon-remote.kubernetesBuildDeploy.labels" . | nindent 4 }} - {{- with .Values.kubernetesBuildDeploy.serviceAccount.annotations }} - annotations: - {{- toYaml . | nindent 4 }} - {{- end }} diff --git a/charts/lagoon-remote/values.yaml b/charts/lagoon-remote/values.yaml index f91ee04b..2bfdaab4 100644 --- a/charts/lagoon-remote/values.yaml +++ b/charts/lagoon-remote/values.yaml @@ -103,13 +103,6 @@ dockerHost: effect: PreferNoSchedule operator: Exists -# this account is used by the legacy Lagoon kubernetes build deploy system. -kubernetesBuildDeploy: - serviceAccount: - # The name of the service account to use. - # If not set, a name is generated using the fullname template. - name: - # sshPortal is an optional service providing low-latency SSH connectivity to # Lagoon environments. sshPortal: From ac51b9a0421e9b3f343b8a5318654175dc2bf014 Mon Sep 17 00:00:00 2001 From: shreddedbacon Date: Mon, 5 Jun 2023 13:14:29 +1000 Subject: [PATCH 02/11] chore: update artifactub change --- charts/lagoon-remote/Chart.yaml | 4 ++-- charts/lagoon-remote/templates/_helpers.tpl | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/charts/lagoon-remote/Chart.yaml b/charts/lagoon-remote/Chart.yaml index caa4be1b..fde1daca 100644 --- a/charts/lagoon-remote/Chart.yaml +++ b/charts/lagoon-remote/Chart.yaml @@ -44,5 +44,5 @@ dependencies: # Valid supported kinds are added, changed, deprecated, removed, fixed and security annotations: artifacthub.io/changes: | - - kind: changed - description: update lagoon-build-deploy subchart to 0.22.0 + - kind: removed + description: removed old kubernetes build deploy references diff --git a/charts/lagoon-remote/templates/_helpers.tpl b/charts/lagoon-remote/templates/_helpers.tpl index 9da7dca5..a0fd1b6f 100644 --- a/charts/lagoon-remote/templates/_helpers.tpl +++ b/charts/lagoon-remote/templates/_helpers.tpl @@ -98,7 +98,7 @@ Create a default fully qualified app name for storageCalculator. {{- end }} {{/* -Common labels storageCalculator.` +Common labels storageCalculator. */}} {{- define "lagoon-remote.storageCalculator.labels" -}} helm.sh/chart: {{ include "lagoon-remote.chart" . }} From 5bb3d45e8d8b517b625456294280161bfc08de25 Mon Sep 17 00:00:00 2001 From: Toby Bellwood Date: Tue, 14 Nov 2023 14:30:18 +1100 Subject: [PATCH 03/11] add minio settings to api-data-watcher-pusher --- Makefile | 2 +- charts/lagoon-test/Chart.yaml | 4 ++-- .../templates/local-api-data-watcher-pusher.deployment.yaml | 6 ++++++ charts/lagoon-test/values.yaml | 3 +++ 4 files changed, 12 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index fa677c27..27e28b75 100644 --- a/Makefile +++ b/Makefile @@ -158,7 +158,7 @@ install-minio: install-ingress --wait \ --timeout $(TIMEOUT) \ --set auth.rootUser=lagoonFilesAccessKey,auth.rootPassword=lagoonFilesSecretKey \ - --set defaultBuckets=lagoon-files \ + --set defaultBuckets='lagoon-files\,restores' \ --version=12.8.7 \ minio \ bitnami/minio diff --git a/charts/lagoon-test/Chart.yaml b/charts/lagoon-test/Chart.yaml index 836924df..cc055519 100644 --- a/charts/lagoon-test/Chart.yaml +++ b/charts/lagoon-test/Chart.yaml @@ -15,7 +15,7 @@ type: application # time you make changes to the chart and its templates, including the app # version. # Versions are expected to follow Semantic Versioning (https://semver.org/) -version: 0.52.0 +version: 0.53.0 # This is the version number of the application being deployed. This version # number should be incremented each time you make changes to the application. @@ -29,4 +29,4 @@ appVersion: v2.16.0 annotations: artifacthub.io/changes: | - kind: changed - description: require minimum Kubernetes 1.23 + description: add minio settings to api-data-watcher-pusher diff --git a/charts/lagoon-test/templates/local-api-data-watcher-pusher.deployment.yaml b/charts/lagoon-test/templates/local-api-data-watcher-pusher.deployment.yaml index eadd3040..bede06d4 100644 --- a/charts/lagoon-test/templates/local-api-data-watcher-pusher.deployment.yaml +++ b/charts/lagoon-test/templates/local-api-data-watcher-pusher.deployment.yaml @@ -47,6 +47,12 @@ spec: secretKeyRef: name: {{ .Values.jwtSecretSecret | quote }} key: JWTSECRET + - name: MINIO_SERVER_URL + value: {{ .Values.minioURL | quote }} + - name: MINIO_ROOT_USER + value: {{ .Values.minioUser | quote }} + - name: MINIO_ROOT_PASSWORD + value: {{ .Values.minioPass | quote }} - name: TOKEN value: {{ required "A valid .Values.token required!" .Values.token | quote }} startupProbe: diff --git a/charts/lagoon-test/values.yaml b/charts/lagoon-test/values.yaml index 0bea382c..49108445 100644 --- a/charts/lagoon-test/values.yaml +++ b/charts/lagoon-test/values.yaml @@ -8,6 +8,9 @@ gitAuthorizedKeys: > ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAACAQDEZlms5XsiyWjmnnUyhpt93VgHypse9Bl8kNkmZJTiM3Ex/wZAfwogzqd2LrTEiIOWSH1HnQazR+Cc9oHCmMyNxRrLkS/MEl0yZ38Q+GDfn37h/llCIZNVoHlSgYkqD0MQrhfGL5AulDUKIle93dA6qdCUlnZZjDPiR0vEXR36xGuX7QYAhK30aD2SrrBruTtFGvj87IP/0OEOvUZe8dcU9G/pCoqrTzgKqJRpqs/s5xtkqLkTIyR/SzzplO21A+pCKNax6csDDq3snS8zfx6iM8MwVfh8nvBW9seax1zBvZjHAPSTsjzmZXm4z32/ujAn/RhIkZw3ZgRKrxzryttGnWJJ8OFyF31JTJgwWWuPdH53G15PC83ZbmEgSV3win51RZRVppN4uQUuaqZWG9wwk2a6P5aen1RLCSLpTkd2mAEk9PlgmJrf8vITkiU9pF9n68ENCoo556qSdxW2pxnjrzKVPSqmqO1Xg5K4LOX4/9N4n4qkLEOiqnzzJClhFif3O28RW86RPxERGdPT81UI0oDAcU5euQr8Emz+Hd+PY1115UIld3CIHib5PYL9Ee0bFUKiWpR/acSe1fHB64mCoHP7hjFepGsq7inkvg2651wUDKBshGltpNkMj6+aZedNc0/rKYyjl80nT8g8QECgOSRzpmYp0zli2HpFoLOiWw== ansible-testing jwtSecretSecret: lagoon-core-secrets keycloakURL: http://lagoon-core-keycloak:8080 +minioURL: http://minio.minio.svc:9000 +minioUser: lagoonFilesAccessKey +minioPass: lagoonFilesSecretKey routeSuffixHTTPPort: 32080 routeSuffixHTTPSPort: 32443 sshHost: lagoon-core-ssh From ab1496673e71ccd97ecf50a5d5119c5b2abc2926 Mon Sep 17 00:00:00 2001 From: Toby Bellwood Date: Tue, 14 Nov 2023 14:30:18 +1100 Subject: [PATCH 04/11] add minio settings to dwp and make into a helm test job --- charts/lagoon-test/Chart.yaml | 2 +- charts/lagoon-test/templates/_helpers.tpl | 6 ++-- ...=> local-api-data-watcher-pusher.job.yaml} | 32 ++++++------------- .../templates/tests/test-connection.yaml | 3 +- .../templates/tests/test-suite.yaml | 3 +- 5 files changed, 18 insertions(+), 28 deletions(-) rename charts/lagoon-test/templates/{local-api-data-watcher-pusher.deployment.yaml => local-api-data-watcher-pusher.job.yaml} (80%) diff --git a/charts/lagoon-test/Chart.yaml b/charts/lagoon-test/Chart.yaml index cc055519..6a1cedbc 100644 --- a/charts/lagoon-test/Chart.yaml +++ b/charts/lagoon-test/Chart.yaml @@ -29,4 +29,4 @@ appVersion: v2.16.0 annotations: artifacthub.io/changes: | - kind: changed - description: add minio settings to api-data-watcher-pusher + description: add minio settings to api-data-watcher-pusher and make into a helm test job diff --git a/charts/lagoon-test/templates/_helpers.tpl b/charts/lagoon-test/templates/_helpers.tpl index 3fe70843..e7810695 100644 --- a/charts/lagoon-test/templates/_helpers.tpl +++ b/charts/lagoon-test/templates/_helpers.tpl @@ -92,14 +92,14 @@ app.kubernetes.io/instance: {{ .Release.Name }} {{/* -Create a default fully qualified app name for local-git. +Create a default fully qualified app name for local-api-data-watcher-pusher. */}} {{- define "lagoon-test.localAPIDataWatcherPusher.fullname" -}} {{- include "lagoon-test.fullname" . }}-local-api-data-watcher-pusher {{- end }} {{/* -Common labels local-git. +Common labels local-api-data-watcher-pusher. */}} {{- define "lagoon-test.localAPIDataWatcherPusher.labels" -}} helm.sh/chart: {{ include "lagoon-test.chart" . }} @@ -111,7 +111,7 @@ app.kubernetes.io/managed-by: {{ .Release.Service }} {{- end }} {{/* -Selector labels local-git. +Selector labels local-api-data-watcher-pusher. */}} {{- define "lagoon-test.localAPIDataWatcherPusher.selectorLabels" -}} app.kubernetes.io/name: {{ include "lagoon-test.name" . }} diff --git a/charts/lagoon-test/templates/local-api-data-watcher-pusher.deployment.yaml b/charts/lagoon-test/templates/local-api-data-watcher-pusher.job.yaml similarity index 80% rename from charts/lagoon-test/templates/local-api-data-watcher-pusher.deployment.yaml rename to charts/lagoon-test/templates/local-api-data-watcher-pusher.job.yaml index bede06d4..2efae436 100644 --- a/charts/lagoon-test/templates/local-api-data-watcher-pusher.deployment.yaml +++ b/charts/lagoon-test/templates/local-api-data-watcher-pusher.job.yaml @@ -1,34 +1,29 @@ -apiVersion: apps/v1 -kind: Deployment +apiVersion: batch/v1 +kind: Job metadata: name: {{ include "lagoon-test.localAPIDataWatcherPusher.fullname" . }} labels: {{- include "lagoon-test.localAPIDataWatcherPusher.labels" . | nindent 4 }} + annotations: + "helm.sh/hook": test + "helm.sh/hook-weight": "10" spec: - selector: - matchLabels: - {{- include "lagoon-test.localAPIDataWatcherPusher.selectorLabels" . | nindent 6 }} + backoffLimit: 2 template: metadata: - {{- with .Values.localAPIDataWatcherPusher.podAnnotations }} - annotations: - {{- toYaml . | nindent 8 }} - {{- end }} labels: {{- include "lagoon-test.localAPIDataWatcherPusher.selectorLabels" . | nindent 8 }} spec: - {{- with .Values.localAPIDataWatcherPusher.imagePullSecrets }} - imagePullSecrets: - {{- toYaml . | nindent 8 }} - {{- end }} + restartPolicy: Never securityContext: {{- toYaml .Values.localAPIDataWatcherPusher.podSecurityContext | nindent 8 }} + terminationGracePeriodSeconds: 120 containers: - name: api-data-watcher-pusher - securityContext: - {{- toYaml .Values.localAPIDataWatcherPusher.securityContext | nindent 10 }} image: "{{ .Values.localAPIDataWatcherPusher.image.repository }}:{{ coalesce .Values.localAPIDataWatcherPusher.image.tag .Values.imageTag .Chart.AppVersion }}" imagePullPolicy: {{ .Values.localAPIDataWatcherPusher.image.pullPolicy }} + securityContext: + {{- toYaml .Values.localAPIDataWatcherPusher.securityContext | nindent 10 }} env: - name: API_HOST value: {{ .Values.apiHost | quote }} @@ -55,13 +50,6 @@ spec: value: {{ .Values.minioPass | quote }} - name: TOKEN value: {{ required "A valid .Values.token required!" .Values.token | quote }} - startupProbe: - exec: - command: - - test - - -f - - /tmp/api-data-pushed - failureThreshold: 90 resources: {{- toYaml .Values.localAPIDataWatcherPusher.resources | nindent 10 }} {{- with .Values.localAPIDataWatcherPusher.nodeSelector }} diff --git a/charts/lagoon-test/templates/tests/test-connection.yaml b/charts/lagoon-test/templates/tests/test-connection.yaml index cba025e5..ddaa44b1 100644 --- a/charts/lagoon-test/templates/tests/test-connection.yaml +++ b/charts/lagoon-test/templates/tests/test-connection.yaml @@ -5,7 +5,8 @@ metadata: labels: {{- include "lagoon-test.labels" . | nindent 4 }} annotations: - "helm.sh/hook": test-success + "helm.sh/hook": test + "helm.sh/hook-weight": "30" spec: containers: - name: nc diff --git a/charts/lagoon-test/templates/tests/test-suite.yaml b/charts/lagoon-test/templates/tests/test-suite.yaml index e83d858b..546051bc 100644 --- a/charts/lagoon-test/templates/tests/test-suite.yaml +++ b/charts/lagoon-test/templates/tests/test-suite.yaml @@ -6,7 +6,8 @@ metadata: labels: {{- include "lagoon-test.labels" . | nindent 4 }} annotations: - "helm.sh/hook": test-success + "helm.sh/hook": test + "helm.sh/hook-weight": "50" spec: serviceAccountName: {{ include "lagoon-test.serviceAccountName" . }} containers: From 939f6ad60186fc54b6f0155746356cede6e6a42a Mon Sep 17 00:00:00 2001 From: Toby Bellwood Date: Tue, 14 Nov 2023 14:30:18 +1100 Subject: [PATCH 05/11] use testlagoon/main for test-suite --- .github/workflows/test-suite.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/test-suite.yaml b/.github/workflows/test-suite.yaml index 0e8a526e..76a1c4ae 100644 --- a/.github/workflows/test-suite.yaml +++ b/.github/workflows/test-suite.yaml @@ -123,8 +123,8 @@ jobs: if: | (steps.list-changed.outputs.changed == 'true') || (contains(github.event.pull_request.labels.*.name, 'needs-testing')) - # run: make -j8 -O fill-test-ci-values TESTS=[${{ matrix.test }}] IMAGE_REGISTRY=testlagoon IMAGE_TAG=main - run: make -j8 -O fill-test-ci-values TESTS=[${{ matrix.test }}] + run: make -j8 -O fill-test-ci-values TESTS=[${{ matrix.test }}] IMAGE_REGISTRY=testlagoon IMAGE_TAG=main + # run: make -j8 -O fill-test-ci-values TESTS=[${{ matrix.test }}] - name: Free up some disk space if: | From fbefeaf92400bad6d82b1681711131b88607b61b Mon Sep 17 00:00:00 2001 From: Toby Bellwood Date: Tue, 21 Nov 2023 13:13:02 +1100 Subject: [PATCH 06/11] add CLEAR_API_DATA env to dwp --- Makefile | 5 ++++- charts/lagoon-test/ci/linter-values.yaml.tpl | 2 ++ .../templates/local-api-data-watcher-pusher.job.yaml | 4 ++++ charts/lagoon-test/values.yaml | 3 +++ 4 files changed, 13 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 27e28b75..64088fcf 100644 --- a/Makefile +++ b/Makefile @@ -41,6 +41,9 @@ SKIP_ALL_DEPS = DISABLE_CORE_HARBOR = # Set to `true` to enable the elements of lagoon-core that talk to OpenSearch installs OPENSEARCH_INTEGRATION_ENABLED = false +# Ordinarily we shouldn't need to clear the API data as it's usually a first run. Set this +# variable on a test run to clear (what's clearable) first +CLEAR_API_DATA = false TIMEOUT = 30m HELM = helm @@ -56,7 +59,7 @@ fill-test-ci-values: && export token="$$($(KUBECTL) -n lagoon create token lagoon-build-deploy --duration 3h)" \ && export $$([ $(IMAGE_TAG) ] && echo imageTag='$(IMAGE_TAG)' || echo imageTag='latest') \ && export webhookHandler="lagoon-core-webhook-handler" \ - && export tests='$(TESTS)' imageRegistry='$(IMAGE_REGISTRY)' \ + && export tests='$(TESTS)' imageRegistry='$(IMAGE_REGISTRY)' clearApiData='$(CLEAR_API_DATA)' \ && valueTemplate=charts/lagoon-test/ci/linter-values.yaml \ && envsubst < $$valueTemplate.tpl > $$valueTemplate \ && cat $$valueTemplate diff --git a/charts/lagoon-test/ci/linter-values.yaml.tpl b/charts/lagoon-test/ci/linter-values.yaml.tpl index 221725a5..d2f94f44 100644 --- a/charts/lagoon-test/ci/linter-values.yaml.tpl +++ b/charts/lagoon-test/ci/linter-values.yaml.tpl @@ -14,6 +14,8 @@ localGit: localAPIDataWatcherPusher: image: repository: ${imageRegistry}/local-api-data-watcher-pusher + additonalEnvs: + CLEAR_API_DATA: ${clearApiData} tests: image: diff --git a/charts/lagoon-test/templates/local-api-data-watcher-pusher.job.yaml b/charts/lagoon-test/templates/local-api-data-watcher-pusher.job.yaml index 2efae436..0b35d692 100644 --- a/charts/lagoon-test/templates/local-api-data-watcher-pusher.job.yaml +++ b/charts/lagoon-test/templates/local-api-data-watcher-pusher.job.yaml @@ -50,6 +50,10 @@ spec: value: {{ .Values.minioPass | quote }} - name: TOKEN value: {{ required "A valid .Values.token required!" .Values.token | quote }} + {{- range $key, $val := .Values.localAPIDataWatcherPusher.additionalEnvs }} + - name: {{ $key }} + value: {{ $val | quote }} + {{- end }} resources: {{- toYaml .Values.localAPIDataWatcherPusher.resources | nindent 10 }} {{- with .Values.localAPIDataWatcherPusher.nodeSelector }} diff --git a/charts/lagoon-test/values.yaml b/charts/lagoon-test/values.yaml index 49108445..1a29b857 100644 --- a/charts/lagoon-test/values.yaml +++ b/charts/lagoon-test/values.yaml @@ -119,6 +119,9 @@ localAPIDataWatcherPusher: # Overrides the image tag whose default is the chart appVersion. tag: "" + additionalEnvs: + # CLEAR_API_DATA: true + podSecurityContext: {} securityContext: {} From 66e7268b38f24025439f2981debe503cb122c4ad Mon Sep 17 00:00:00 2001 From: Toby Bellwood Date: Tue, 28 Nov 2023 09:18:07 +1100 Subject: [PATCH 07/11] remove insights-remote service if not enabled --- charts/lagoon-remote/Chart.yaml | 4 ++-- charts/lagoon-remote/templates/insights-remote.service.yaml | 2 ++ 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/charts/lagoon-remote/Chart.yaml b/charts/lagoon-remote/Chart.yaml index 4c02e8e8..da2a29c3 100644 --- a/charts/lagoon-remote/Chart.yaml +++ b/charts/lagoon-remote/Chart.yaml @@ -19,7 +19,7 @@ type: application # This is the chart version. This version number should be incremented each # time you make changes to the chart and its templates, including the app # version. -version: 0.86.1 +version: 0.87.0 dependencies: - name: lagoon-build-deploy @@ -45,4 +45,4 @@ dependencies: annotations: artifacthub.io/changes: | - kind: changed - description: update lagoon-build-deploy to v0.26.2 with updated remote-controller + description: remove insights-remote service if not enabled diff --git a/charts/lagoon-remote/templates/insights-remote.service.yaml b/charts/lagoon-remote/templates/insights-remote.service.yaml index 01f105dd..981022e1 100644 --- a/charts/lagoon-remote/templates/insights-remote.service.yaml +++ b/charts/lagoon-remote/templates/insights-remote.service.yaml @@ -1,3 +1,4 @@ +{{- if .Values.insightsRemote.enabled -}} apiVersion: v1 kind: Service metadata: @@ -13,3 +14,4 @@ spec: name: insights-ws selector: {{- include "lagoon-remote.insightsRemote.selectorLabels" . | nindent 4 }} +{{- end }} From ca39ac004a350a537a81666ab332876ca87c44b0 Mon Sep 17 00:00:00 2001 From: Toby Bellwood Date: Wed, 6 Dec 2023 14:29:22 +1100 Subject: [PATCH 08/11] insights updates --- charts/lagoon-core/Chart.yaml | 4 ++++ charts/lagoon-core/values.yaml | 2 +- charts/lagoon-remote/Chart.yaml | 2 ++ charts/lagoon-remote/values.yaml | 2 +- 4 files changed, 8 insertions(+), 2 deletions(-) diff --git a/charts/lagoon-core/Chart.yaml b/charts/lagoon-core/Chart.yaml index 0db0900c..a9e6ce47 100644 --- a/charts/lagoon-core/Chart.yaml +++ b/charts/lagoon-core/Chart.yaml @@ -42,3 +42,7 @@ annotations: artifacthub.io/changes: | - kind: changed description: bump lagoon-opensearch-sync version to v0.7.1 + - kind: changed + description: updated to insights-handler:v0.0.2 + - kind: changed + description: pinned insights to trivy:0.48.0 diff --git a/charts/lagoon-core/values.yaml b/charts/lagoon-core/values.yaml index 25798e4f..f37eea9a 100644 --- a/charts/lagoon-core/values.yaml +++ b/charts/lagoon-core/values.yaml @@ -620,7 +620,7 @@ insightsHandler: enabled: false image: repository: aquasec/trivy - tag: latest + tag: 0.48.0 service: type: ClusterIP port: 4954 diff --git a/charts/lagoon-remote/Chart.yaml b/charts/lagoon-remote/Chart.yaml index 566f0439..09b9be81 100644 --- a/charts/lagoon-remote/Chart.yaml +++ b/charts/lagoon-remote/Chart.yaml @@ -48,3 +48,5 @@ annotations: description: remove insights-remote service if not enabled - kind: removed description: removed old kubernetes build deploy references + - kind: changed + description: updated to insights-remote:v0.0.8 diff --git a/charts/lagoon-remote/values.yaml b/charts/lagoon-remote/values.yaml index 185057b6..866f1871 100644 --- a/charts/lagoon-remote/values.yaml +++ b/charts/lagoon-remote/values.yaml @@ -220,7 +220,7 @@ insightsRemote: repository: uselagoon/insights-remote pullPolicy: Always # Overrides the image tag whose default is the chart appVersion. - tag: "v0.0.7" + tag: "v0.0.8" imagePullSecrets: [] nameOverride: "" From 274d92c9f0a6e558ae4948130018cd96f119b985 Mon Sep 17 00:00:00 2001 From: Toby Bellwood Date: Wed, 6 Dec 2023 14:31:53 +1100 Subject: [PATCH 09/11] bump chart version to 1.41.0 --- charts/lagoon-core/Chart.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/charts/lagoon-core/Chart.yaml b/charts/lagoon-core/Chart.yaml index a9e6ce47..b09e5fc7 100644 --- a/charts/lagoon-core/Chart.yaml +++ b/charts/lagoon-core/Chart.yaml @@ -21,7 +21,7 @@ type: application # time you make changes to the chart and its templates, including the app # version. # Versions are expected to follow Semantic Versioning (https://semver.org/) -version: 1.40.0 +version: 1.41.0 # This is the version number of the application being deployed. This version # number should be incremented each time you make changes to the application. From 2eb0675acfd7f80c45819cd3c6b17a3fc96f51a7 Mon Sep 17 00:00:00 2001 From: Toby Bellwood Date: Wed, 6 Dec 2023 15:52:06 +1100 Subject: [PATCH 10/11] override task and bd images --- .github/workflows/test-suite.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/test-suite.yaml b/.github/workflows/test-suite.yaml index 76a1c4ae..b91e17b4 100644 --- a/.github/workflows/test-suite.yaml +++ b/.github/workflows/test-suite.yaml @@ -123,7 +123,7 @@ jobs: if: | (steps.list-changed.outputs.changed == 'true') || (contains(github.event.pull_request.labels.*.name, 'needs-testing')) - run: make -j8 -O fill-test-ci-values TESTS=[${{ matrix.test }}] IMAGE_REGISTRY=testlagoon IMAGE_TAG=main + run: make -j8 -O fill-test-ci-values TESTS=[${{ matrix.test }}] IMAGE_REGISTRY=testlagoon IMAGE_TAG=main OVERRIDE_BUILD_DEPLOY_DIND_IMAGE=uselagoon/build-deploy-image:main OVERRIDE_ACTIVE_STANDBY_TASK_IMAGE=testlagoon/task-activestandby:main # run: make -j8 -O fill-test-ci-values TESTS=[${{ matrix.test }}] - name: Free up some disk space From 70b1d5663def39313df79791d1431c88ca05f79b Mon Sep 17 00:00:00 2001 From: Toby Bellwood Date: Fri, 8 Dec 2023 13:19:16 +1100 Subject: [PATCH 11/11] chore: update lagoon-build-deploy chart to v0.26.3 --- charts/lagoon-remote/Chart.lock | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/charts/lagoon-remote/Chart.lock b/charts/lagoon-remote/Chart.lock index 63327a5c..cf4d6808 100644 --- a/charts/lagoon-remote/Chart.lock +++ b/charts/lagoon-remote/Chart.lock @@ -1,7 +1,7 @@ dependencies: - name: lagoon-build-deploy repository: https://uselagoon.github.io/lagoon-charts/ - version: 0.26.2 + version: 0.26.3 - name: dioscuri repository: https://amazeeio.github.io/charts/ version: 0.4.1 @@ -11,5 +11,5 @@ dependencies: - name: nats repository: https://nats-io.github.io/k8s/helm/charts/ version: 0.19.17 -digest: sha256:98fadf31ddfe049c50ff4554a11732ec10a9cf8a41b8e0ce1c33a37af4f81de4 -generated: "2023-11-14T10:12:08.139505+11:00" +digest: sha256:1f873430b35bd44722954a1f02699e32c5e7468a9a77a135e5df3ca3766a8ed2 +generated: "2023-12-08T13:18:21.18121865+11:00"