chore: Update renku actions to v1.15.1

RenkuBot · aledegano · commit fed07a37ad23 · 2025-03-13T08:20:06.000+01:00
* Define defaults requests/limits in values file
* Use local minimal-deployment values for CI deployments
* Remove `graph.enabled` switch from ingress Helm chart template
diff --git a/.github/workflows/cron-jobs.yaml b/.github/workflows/cron-jobs.yaml
@@ -3,15 +3,15 @@ name: Cron jobs
 on:
   schedule:
     # Every day at 1am
-    - cron: '0 1 * * *'
+    - cron: "0 1 * * *"
   workflow_dispatch:
 
 jobs:
   remove-CI-deployments:
     runs-on: ubuntu-24.04
     steps:
       - name: renku teardown
-        uses: SwissDataScienceCenter/renku-actions/cleanup-renku-ci-deployments@v1.14.1
+        uses: SwissDataScienceCenter/renku-actions/cleanup-renku-ci-deployments@v1.15.1
         env:
           GITLAB_TOKEN: ${{ secrets.DEV_GITLAB_TOKEN }}
           RENKUBOT_KUBECONFIG: ${{ secrets.RENKUBOT_DEV_KUBECONFIG }}
diff --git a/.github/workflows/publish-helm-chart.yml b/.github/workflows/publish-helm-chart.yml
@@ -15,7 +15,7 @@ jobs:
       - name: Set version
         id: vars
         run: echo "tag=${GITHUB_REF#refs/*/}" >> $GITHUB_OUTPUT
-      - uses: SwissDataScienceCenter/renku-actions/publish-chart@v1.14.1
+      - uses: SwissDataScienceCenter/renku-actions/publish-chart@v1.15.1
         env:
           CHART_DIR: helm-chart/
           CHART_NAME: renku
diff --git a/.github/workflows/publish-master-merges.yaml b/.github/workflows/publish-master-merges.yaml
@@ -19,7 +19,7 @@ jobs:
           fetch-depth: 0
       - uses: azure/setup-helm@v4
         with:
-          version: '3.8.0'
+          version: "3.8.0"
         id: setup-helm
       - id: get-latest-version
         run: |
@@ -35,7 +35,7 @@ jobs:
       - id: set-version
         run: |
           echo "publish_version=${{ steps.bump-semver.outputs.new_version }}.$(echo ${{ github.sha }} | cut -c 1-7)" >> $GITHUB_ENV
-      - uses: SwissDataScienceCenter/renku-actions/publish-chart@v1.14.1
+      - uses: SwissDataScienceCenter/renku-actions/publish-chart@v1.15.1
         env:
           CHART_DIR: helm-chart/
           CHART_TAG: "--tag ${{env.publish_version}}"
@@ -62,7 +62,6 @@ jobs:
       - uses: actions-ecosystem/action-push-tag@v1
         with:
           tag: ${{ env.publish_version }}
-          message:
-            "${{ env.publish_version }}: PR #${{
+          message: "${{ env.publish_version }}: PR #${{
             github.event.pull_request.number }} ${{
             github.event.pull_request.title }}"
diff --git a/.github/workflows/pull-request-test.yml b/.github/workflows/pull-request-test.yml
@@ -66,7 +66,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4.1.7
       - id: deploy-comment
-        uses: SwissDataScienceCenter/renku-actions/check-pr-description@v1.14.1
+        uses: SwissDataScienceCenter/renku-actions/check-pr-description@v1.15.1
         with:
           string: /deploy
           pr_ref: ${{ github.event.number }}
@@ -83,7 +83,7 @@ jobs:
       - uses: actions/checkout@v4.1.7
       - name: renku build and deploy
         if: needs.check-deploy.outputs.pr-contains-string == 'true'
-        uses: SwissDataScienceCenter/renku-actions/deploy-renku@v1.14.1
+        uses: SwissDataScienceCenter/renku-actions/deploy-renku@v1.15.1
         env:
           DOCKER_PASSWORD: ${{ secrets.RENKU_DOCKER_PASSWORD }}
           DOCKER_USERNAME: ${{ secrets.RENKU_DOCKER_USERNAME }}
@@ -92,7 +92,7 @@ jobs:
           RENKU_ANONYMOUS_SESSIONS: true
           RENKU_RELEASE: ci-renku-${{ github.event.number }}
           RENKU_VALUES_FILE: "${{ github.workspace }}/values.yaml"
-          RENKU_VALUES: ${{ secrets.COMBINED_CHARTS_CI_RENKU_VALUES }}
+          RENKU_VALUES: minimal-deployment/minimal-deployment-values.yaml
           RENKUBOT_KUBECONFIG: ${{ secrets.RENKUBOT_DEV_KUBECONFIG }}
           TEST_ARTIFACTS_PATH: "tests-artifacts-${{ github.sha }}"
           renku: "@${{ github.head_ref }}"
@@ -128,7 +128,7 @@ jobs:
     needs: [check-deploy, deploy-pr]
     runs-on: ubuntu-24.04
     steps:
-      - uses: SwissDataScienceCenter/renku-actions/test-renku@v1.14.1
+      - uses: SwissDataScienceCenter/renku-actions/test-renku@v1.15.1
         with:
           kubeconfig: ${{ secrets.RENKUBOT_DEV_KUBECONFIG }}
           renku-release: ci-renku-${{ github.event.number }}
@@ -156,7 +156,7 @@ jobs:
             rstudioSession,
           ]
     steps:
-      - uses: SwissDataScienceCenter/renku-actions/test-renku-cypress@v1.14.1
+      - uses: SwissDataScienceCenter/renku-actions/test-renku-cypress@v1.15.1
         if: github.event.action != 'closed' && needs.check-deploy.outputs.pr-contains-string == 'true' && needs.check-deploy.outputs.test-enabled == 'true'
         with:
           e2e-target: ${{ matrix.tests }}
@@ -197,17 +197,17 @@ jobs:
     runs-on: ubuntu-24.04
     steps:
       - name: Check deploy string
-        if: ${{ 
-            needs.check-deploy.outputs.pr-contains-string == 'true' 
-            && startsWith(github.base_ref, 'release-') 
-            && (
-               needs.check-deploy.outputs.renku-core != null
-               || needs.check-deploy.outputs.renku-gateway != null
-               || needs.check-deploy.outputs.renku-graph != null
-               || needs.check-deploy.outputs.renku-notebooks != null
-               || needs.check-deploy.outputs.renku-ui != null
-               || needs.check-deploy.outputs.renku-data-services != null
-            )
+        if: ${{
+          needs.check-deploy.outputs.pr-contains-string == 'true'
+          && startsWith(github.base_ref, 'release-')
+          && (
+          needs.check-deploy.outputs.renku-core != null
+          || needs.check-deploy.outputs.renku-gateway != null
+          || needs.check-deploy.outputs.renku-graph != null
+          || needs.check-deploy.outputs.renku-notebooks != null
+          || needs.check-deploy.outputs.renku-ui != null
+          || needs.check-deploy.outputs.renku-data-services != null
+          )
           }}
         uses: actions/github-script@v7
         with:
@@ -218,7 +218,7 @@ jobs:
     runs-on: ubuntu-24.04
     steps:
       - name: renku teardown
-        uses: SwissDataScienceCenter/renku-actions/cleanup-renku-ci-deployments@v1.14.1
+        uses: SwissDataScienceCenter/renku-actions/cleanup-renku-ci-deployments@v1.15.1
         env:
           HELM_RELEASE_REGEX: "^ci-renku-${{ github.event.number }}$"
           GITLAB_TOKEN: ${{ secrets.DEV_GITLAB_TOKEN }}
diff --git a/.github/workflows/renku-dev-test.yaml b/.github/workflows/renku-dev-test.yaml
@@ -8,7 +8,7 @@ jobs:
       github.event.client_payload.message == 'Helm test succeeded' }}
     runs-on: ubuntu-24.04
     steps:
-      - uses: SwissDataScienceCenter/renku-actions/test-renku@v1.14.1
+      - uses: SwissDataScienceCenter/renku-actions/test-renku@v1.15.1
         with:
           kubeconfig: ${{ secrets.RENKUBOT_DEV_KUBECONFIG }}
           renku-release: renku
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -3,6 +3,22 @@
 0.65.1
 ------
 
+Renku 0.65.1 refactors the code used to deploy CI installations used to test Pull Requests.
+NOTE to administrators: The default values file now specify several resource requests and limits
+that were not specified before that are tuned for a small deployment, if the values file
+for your deployment does not override these requests and limits (which it should!) then
+you might end up with services being killed by Kubernetes (for Out Of Memory errors).
+
+Internal Changes
+~~~~~~~~~~~~~~~~
+
+**Improvements**
+
+- Many resources requests and limits have been defined in the default values file.
+- Now the values file used in the CI deployments is taken from this repository in `minimal-deployment`
+  which also serves as a reference for the default values file.
+- Removed a check in the Helm template for the ingress that could result in
+  dataset search not working properly if disabled.
 
 
 0.65.0
diff --git a/cypress-tests/cypress/e2e/useSession.cy.ts b/cypress-tests/cypress/e2e/useSession.cy.ts
@@ -146,7 +146,6 @@ describe("Basic public project functionality", () => {
     cy.getDataCy("workflows-browser")
       .should("be.visible")
       .children()
-      .should("have.length", 1)
       .contains(workflow.name)
       .should("be.visible")
       .click();
diff --git a/helm-chart/renku/templates/ingress.yaml b/helm-chart/renku/templates/ingress.yaml
@@ -8,15 +8,12 @@
 {{- $uiServicePort := .Values.ui.client.service.port -}}
 {{- $uiserverFullname := include "uiserver.fullname" . -}}
 {{- $uiserverServicePort := .Values.ui.server.service.port -}}
-{{- $graphEnabled := .Values.graph.enabled -}}
 {{- $webhookServiceFullname := include "renku.graph.webhookService.fullname" . -}}
 {{- $knowledgeGraphFullname := include "renku.graph.knowledgeGraph.fullname" . -}}
 {{- $searchApiFullname := include "renku.search.searchApi.fullname" . -}}
 {{- $renkuFullname := include "renku.fullname" . -}}
-{{- if $graphEnabled }}
 {{- $jenaFullname := include "renku-jena.fullname" . -}}
 {{- $jenaServicePort := .Values.jena.service.port -}}
-{{- end }}
 {{- $swaggerEnabled := .Values.swagger.enabled -}}
 ---
 apiVersion: networking.k8s.io/v1
@@ -104,7 +101,6 @@ spec:
             name: {{ template "renku.fullname" $ }}-gateway
             port:
               number: {{ $uiserverServicePort }}
-      {{- if $graphEnabled }}
       - path: /webhooks/events
         pathType: Prefix
         backend:
@@ -119,7 +115,6 @@ spec:
             name: {{ template "renku.fullname" $ }}-gateway
             port:
               number: 80
-      {{- end }}
       {{- if $swaggerEnabled }}
       - path: /swagger
         pathType: Prefix
diff --git a/helm-chart/renku/values.yaml b/helm-chart/renku/values.yaml
diff --git a/helm-chart/values.yaml.changelog.md b/helm-chart/values.yaml.changelog.md
diff --git a/minimal-deployment/minimal-deployment-values.yaml b/minimal-deployment/minimal-deployment-values.yaml
