Merge pull request 'fix: use Tekton compute resources' (#6) from fix/tekton-compute-resources into main

Reviewed-on: #6

catalog/kustomization.yaml

@@ -2,4 +2,5 @@ apiVersion: kustomize.config.k8s.io/v1beta1
 kind: Kustomization
 resources:
   - maidn-node-static-image.yaml
+  - maidn-node-runtime-image.yaml
   - maidn-preview-orphan-reconciler.yaml

catalog/maidn-node-runtime-image.yaml

@@ -0,0 +1,136 @@
+apiVersion: tekton.dev/v1
+kind: Task
+metadata:
+  name: maidn-node-runtime-image
+  namespace: tekton-pipelines
+spec:
+  params:
+    - name: url
+      type: string
+    - name: revision
+      type: string
+    - name: image
+      type: string
+  volumes:
+    - name: work
+      emptyDir: {}
+    - name: git-credentials
+      secret:
+        secretName: forgejo-git-credentials
+    - name: registry-credentials
+      secret:
+        secretName: forgejo-registry-credentials
+        items:
+          - key: .dockerconfigjson
+            path: config.json
+  stepTemplate:
+    securityContext:
+      runAsNonRoot: true
+      runAsUser: 1000
+      runAsGroup: 1000
+      allowPrivilegeEscalation: false
+      capabilities:
+        drop: [ALL]
+      seccompProfile:
+        type: RuntimeDefault
+  steps:
+    - name: clone
+      image: alpine/git:2.47.2
+      env:
+        - name: REPOSITORY_URL
+          value: $(params.url)
+        - name: REVISION
+          value: $(params.revision)
+        - name: IMAGE
+          value: $(params.image)
+      volumeMounts:
+        - name: work
+          mountPath: /work
+        - name: git-credentials
+          mountPath: /credentials
+          readOnly: true
+      script: |
+        #!/bin/sh
+        set -eu
+        case "$REPOSITORY_URL" in https://git.pingu.pw/*) ;; *) exit 1 ;; esac
+        repository_path=${REPOSITORY_URL#https://git.pingu.pw/}
+        case "$repository_path" in [A-Za-z0-9][A-Za-z0-9._-]*/[A-Za-z0-9][A-Za-z0-9._-]*.git) ;; *) exit 1 ;; esac
+        case "$repository_path" in *[!A-Za-z0-9._/-]*|*..*|*//*|*/|*/*/*) exit 1 ;; esac
+        case "$REVISION" in [A-Za-z0-9]*) ;; *) exit 1 ;; esac
+        case "$REVISION" in *[!A-Za-z0-9._/-]*|*..*|*//*|*/) exit 1 ;; esac
+        case "$IMAGE" in git.pingu.pw/*) ;; *) exit 1 ;; esac
+        image_path=${IMAGE#git.pingu.pw/}
+        case "$image_path" in [A-Za-z0-9][A-Za-z0-9._-]*/[A-Za-z0-9][A-Za-z0-9._-]*) ;; *) exit 1 ;; esac
+        case "$image_path" in *[!A-Za-z0-9._/-]*|*..*|*//*|*/|*/*/*) exit 1 ;; esac
+        cat >/work/askpass <<'EOF'
+        #!/bin/sh
+        case "$1" in *Username*) cat /credentials/username ;; *) cat /credentials/password ;; esac
+        EOF
+        chmod 0700 /work/askpass
+        GIT_ASKPASS=/work/askpass GIT_TERMINAL_PROMPT=0 git clone "$REPOSITORY_URL" /work/source
+        git -C /work/source checkout "$REVISION"
+        rm -f /work/askpass
+    - name: build-layer
+      image: node:22-alpine
+      computeResources:
+        requests:
+          memory: 256Mi
+        limits:
+          memory: 512Mi
+      volumeMounts:
+        - name: work
+          mountPath: /work
+      script: |
+        #!/bin/sh
+        set -eu
+        cd /work/source
+        npm install --ignore-scripts --no-audit --no-fund --package-lock=false
+        npm run build
+        npm prune --omit=dev --ignore-scripts --no-audit --no-fund
+        test -f package.json
+        test -d node_modules
+        test -d dist
+        mkdir -p /work/layer/app
+        cp package.json /work/layer/app/
+        cp -R node_modules dist /work/layer/app/
+        tar -C /work/layer -cf /work/layer.tar app
+        rm -rf /work/source /work/layer
+    - name: append
+      image: gcr.io/go-containerregistry/crane:v0.21.7
+      env:
+        - name: HOME
+          value: /tekton/home
+        - name: DOCKER_CONFIG
+          value: /tekton/home/.docker
+      args:
+        - append
+        - --base=node:22-alpine
+        - --new_layer=/work/layer.tar
+        - --new_tag=$(params.image):$(params.revision)
+      volumeMounts:
+        - name: work
+          mountPath: /work
+        - name: registry-credentials
+          mountPath: /tekton/home/.docker
+          readOnly: true
+    - name: mutate
+      image: gcr.io/go-containerregistry/crane:v0.21.7
+      env:
+        - name: HOME
+          value: /tekton/home
+        - name: DOCKER_CONFIG
+          value: /tekton/home/.docker
+      args:
+        - mutate
+        - $(params.image):$(params.revision)
+        - --entrypoint=node
+        - --cmd=dist/index.js
+        - --workdir=/app
+        - --user=node
+        - --env=PORT=8080
+        - --exposed-ports=8080/tcp
+        - --tag=$(params.image):$(params.revision)
+      volumeMounts:
+        - name: registry-credentials
+          mountPath: /tekton/home/.docker
+          readOnly: true

catalog/maidn-node-static-image.yaml

@@ -4,20 +4,6 @@ metadata:
   name: maidn-node-static-image
   namespace: tekton-pipelines
 spec:
-  stepTemplate:
-    env:
-      - name: HOME
-        value: /tekton/home
-    securityContext:
-      runAsNonRoot: true
-      runAsUser: 1000
-      runAsGroup: 1000
-      allowPrivilegeEscalation: false
-      capabilities:
-        drop:
-          - ALL
-      seccompProfile:
-        type: RuntimeDefault
   params:
     - name: url
       type: string
@@ -33,18 +19,28 @@ spec:
     - name: base-image
       type: string
       default: nginx:1.27-alpine
-    - name: target-directory
-      type: string
-      default: /www/target
-  workspaces:
-    - name: source
   volumes:
+    - name: work
+      emptyDir: {}
+    - name: git-credentials
+      secret:
+        secretName: forgejo-git-credentials
     - name: registry-credentials
       secret:
         secretName: forgejo-registry-credentials
         items:
           - key: .dockerconfigjson
             path: config.json
+  stepTemplate:
+    securityContext:
+      runAsNonRoot: true
+      runAsUser: 1000
+      runAsGroup: 1000
+      allowPrivilegeEscalation: false
+      capabilities:
+        drop: [ALL]
+      seccompProfile:
+        type: RuntimeDefault
   steps:
     - name: clone
       image: alpine/git:2.47.2
@@ -53,62 +49,71 @@ spec:
           value: $(params.url)
         - name: REVISION
           value: $(params.revision)
-        - name: SOURCE_PATH
-          value: $(workspaces.source.path)
+      volumeMounts:
+        - name: work
+          mountPath: /work
+        - name: git-credentials
+          mountPath: /credentials
+          readOnly: true
       script: |
         #!/bin/sh
         set -eu
-        case "$REPOSITORY_URL" in https://git.pingu.pw/*) ;; *) exit 1 ;; esac
-        repository_path=${REPOSITORY_URL#https://git.pingu.pw/}
-        case "$repository_path" in *.git) ;; *) exit 1 ;; esac
-        case "$repository_path" in ''|*[!A-Za-z0-9._/-]*|/*|*//*|*..*) exit 1 ;; esac
+        case "$REPOSITORY_URL" in https://git.pingu.pw/*.git) ;; *) exit 1 ;; esac
         case "$REVISION" in [A-Za-z0-9]*) ;; *) exit 1 ;; esac
         case "$REVISION" in *[!A-Za-z0-9._/-]*|*..*|*//*|*/) exit 1 ;; esac
-        git clone "$REPOSITORY_URL" "$SOURCE_PATH"
-        git config --global --add safe.directory "$SOURCE_PATH"
-        git -C "$SOURCE_PATH" checkout "$REVISION"
+        cat >/work/askpass <<'EOF'
+        #!/bin/sh
+        case "$1" in *Username*) cat /credentials/username ;; *) cat /credentials/password ;; esac
+        EOF
+        chmod 0700 /work/askpass
+        GIT_ASKPASS=/work/askpass GIT_TERMINAL_PROMPT=0 git clone "$REPOSITORY_URL" /work/source
+        git -C /work/source checkout "$REVISION"
+        rm -f /work/askpass
     - name: build-layer
       image: node:22-alpine
-      workingDir: $(workspaces.source.path)
+      computeResources:
+        requests:
+          memory: 256Mi
+        limits:
+          memory: 512Mi
       env:
-        - name: IMAGE
-          value: $(params.image)
-        - name: REVISION
-          value: $(params.revision)
+        - name: NODE_OPTIONS
+          value: --max-old-space-size=384
         - name: OUTPUT_DIRECTORY
           value: $(params.output-directory)
-        - name: TARGET_DIRECTORY
-          value: $(params.target-directory)
         - name: BUILD_CONFIGURATION
           value: $(params.build-configuration)
+      volumeMounts:
+        - name: work
+          mountPath: /work
       script: |
         #!/bin/sh
         set -eu
-        case "$IMAGE" in git.pingu.pw/*) ;; *) exit 1 ;; esac
-        image_path=${IMAGE#git.pingu.pw/}
-        case "$image_path" in ''|*[!A-Za-z0-9._/-]*|/*|*//*|*..*|*/) exit 1 ;; esac
-        case "$REVISION" in [A-Za-z0-9]*) ;; *) exit 1 ;; esac
-        case "$REVISION" in *[!A-Za-z0-9._/-]*|*..*|*//*|*/) exit 1 ;; esac
         case "$OUTPUT_DIRECTORY" in ''|/*|-*|*[!A-Za-z0-9._/-]*|*..*|*//*|*/) exit 1 ;; esac
-        case "$TARGET_DIRECTORY" in /*) ;; *) exit 1 ;; esac
-        target_path=${TARGET_DIRECTORY#/}
-        case "$target_path" in ''|-*|*[!A-Za-z0-9._/-]*|*..*|*//*|*/) exit 1 ;; esac
-        # ponytail: no lockfile; skip install scripts and use npm ci when package-lock.json is committed.
+        case "$BUILD_CONFIGURATION" in ''|-*|*[!A-Za-z0-9._-]*) exit 1 ;; esac
+        cd /work/source
         npm install --ignore-scripts --no-audit --no-fund --package-lock=false
         npm run build -- --configuration "$BUILD_CONFIGURATION"
         test -d "$OUTPUT_DIRECTORY"
-        layer_dir=$(mktemp -d)
-        trap 'rm -rf "$layer_dir"' EXIT
-        mkdir -p "$layer_dir/$target_path"
-        cp -R "$OUTPUT_DIRECTORY"/. "$layer_dir/$target_path/"
-        tar -C "$layer_dir" -cf layer.tar "$target_path"
+        mkdir -p /work/layer/www
+        cp -R "$OUTPUT_DIRECTORY"/. /work/layer/www/
+        tar -C /work/layer -cf /work/layer.tar www
+        rm -rf /work/source /work/layer
     - name: push
       image: gcr.io/go-containerregistry/crane:v0.21.7
+      env:
+        - name: HOME
+          value: /tekton/home
+        - name: DOCKER_CONFIG
+          value: /tekton/home/.docker
       args:
         - append
         - --base=$(params.base-image)
-        - --new_layer=$(workspaces.source.path)/layer.tar
+        - --new_layer=/work/layer.tar
         - --new_tag=$(params.image):$(params.revision)
       volumeMounts:
+        - name: work
+          mountPath: /work
         - name: registry-credentials
           mountPath: /tekton/home/.docker
+          readOnly: true

catalog/test-node-runtime-image.sh

@@ -0,0 +1,53 @@
+#!/bin/sh
+set -eu
+
+root=$(CDPATH= cd -- "$(dirname -- "$0")/.." && pwd)
+task="$root/catalog/maidn-node-runtime-image.yaml"
+
+contains() {
+  grep -qF -- "$1" "$task"
+}
+
+for value in \
+  'apiVersion: tekton.dev/v1' \
+  'kind: Task' \
+  'name: maidn-node-runtime-image' \
+  'emptyDir: {}' \
+  'forgejo-git-credentials' \
+  'forgejo-registry-credentials' \
+  'case "$REPOSITORY_URL" in https://git.pingu.pw/*)' \
+  'case "$REVISION" in [A-Za-z0-9]*)' \
+  'case "$IMAGE" in git.pingu.pw/*)' \
+  'GIT_ASKPASS=/work/askpass' \
+  'runAsNonRoot: true' \
+  'allowPrivilegeEscalation: false' \
+  'drop: [ALL]' \
+  'type: RuntimeDefault' \
+  'npm install --ignore-scripts --no-audit --no-fund --package-lock=false' \
+  'memory: 256Mi' \
+  'memory: 512Mi' \
+  'computeResources:' \
+  'npm run build' \
+  'npm prune --omit=dev --ignore-scripts --no-audit --no-fund' \
+  'cp package.json /work/layer/app/' \
+  'cp -R node_modules dist /work/layer/app/' \
+  'tar -C /work/layer -cf /work/layer.tar app' \
+  '--base=node:22-alpine' \
+  '--new_tag=$(params.image):$(params.revision)' \
+  '- mutate' \
+  '--entrypoint=node' \
+  '--cmd=dist/index.js' \
+  '--workdir=/app' \
+  '--user=node' \
+  '--env=PORT=8080' \
+  '--exposed-ports=8080/tcp' \
+  '--tag=$(params.image):$(params.revision)'; do
+  contains "$value"
+done
+
+for param in url revision image; do
+  contains "name: $param"
+done
+
+! grep -qiE 'kaniko|privileged: true' "$task"
+! grep -q '^[[:space:]]*resources:' "$task"

catalog/test-node-static-image.sh

@@ -0,0 +1,42 @@
+#!/bin/sh
+set -eu
+
+root=$(CDPATH= cd -- "$(dirname -- "$0")/.." && pwd)
+task="$root/catalog/maidn-node-static-image.yaml"
+
+contains() {
+  grep -qF -- "$1" "$task"
+}
+
+for value in \
+  'apiVersion: tekton.dev/v1' \
+  'kind: Task' \
+  'name: maidn-node-static-image' \
+  'emptyDir: {}' \
+  'forgejo-git-credentials' \
+  'forgejo-registry-credentials' \
+  'case "$REPOSITORY_URL" in https://git.pingu.pw/*.git)' \
+  'case "$REVISION" in [A-Za-z0-9]*)' \
+  'case "$OUTPUT_DIRECTORY" in' \
+  'case "$BUILD_CONFIGURATION" in' \
+  'GIT_ASKPASS=/work/askpass' \
+  'runAsNonRoot: true' \
+  'allowPrivilegeEscalation: false' \
+  'drop: [ALL]' \
+  'type: RuntimeDefault' \
+  'npm install --ignore-scripts --no-audit --no-fund --package-lock=false' \
+  'memory: 256Mi' \
+  'memory: 512Mi' \
+  'computeResources:' \
+  'tar -C /work/layer -cf /work/layer.tar www' \
+  '--base=$(params.base-image)' \
+  '--new_tag=$(params.image):$(params.revision)'; do
+  contains "$value"
+done
+
+for param in url revision image output-directory build-configuration base-image; do
+  contains "name: $param"
+done
+
+! grep -qiE 'kaniko|privileged: true' "$task"
+! grep -q '^[[:space:]]*resources:' "$task"