Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
6123f469f6
|
||
|
|
7d0c1b9c22
|
||
|
|
b82d0a2155 |
No files matched your search
@@ -28,16 +28,37 @@ on:
|
|||||||
required: false
|
required: false
|
||||||
default: '0'
|
default: '0'
|
||||||
type: string
|
type: string
|
||||||
|
# Bewusst `type: string` und nicht `boolean` — siehe Issue #4. Auf dieser
|
||||||
|
# Gitea-Version (gemessen an 1.27.0) kommt eine boolesche
|
||||||
|
# workflow_call-Eingabe im aufgerufenen Workflow gar nicht an: weder ihre
|
||||||
|
# Vorgabe noch ein ausdruecklich uebergebenes `push: true`. `inputs.push`
|
||||||
|
# war in beiden Faellen leer, der Baustein baute immer im Pruefmodus und
|
||||||
|
# meldete dabei Erfolg. Alle Eingaben vom Typ string desselben Laufs
|
||||||
|
# (platforms, registry, image-owner, container-name) kamen korrekt an;
|
||||||
|
# push war die einzige boolesche und die einzige, die versagte.
|
||||||
|
#
|
||||||
|
# Wer das beim naechsten Aufraeumen "korrekt" auf boolean zurueckstellt,
|
||||||
|
# holt den Fehler still zurueck: nichts wird veroeffentlicht, und der
|
||||||
|
# Lauf bleibt gruen.
|
||||||
|
#
|
||||||
|
# Die Bedingung `inputs.push && inputs.push != 'false'` musste dafuer
|
||||||
|
# nicht angefasst werden. Sie wurde seinerzeit ausdruecklich so gebaut,
|
||||||
|
# dass sie Boolean und String vertraegt (Begruendung am
|
||||||
|
# 1Password-Schritt), und traegt die Umstellung unveraendert: 'true' ist
|
||||||
|
# truthy und ungleich 'false', 'false' ist truthy aber gleich 'false'.
|
||||||
push:
|
push:
|
||||||
description: >-
|
description: >-
|
||||||
Whether the built image is pushed. With push: false no credentials are
|
Whether the built image is pushed, as the string 'true' or 'false'.
|
||||||
fetched from 1Password and no registry login happens, so the workflow
|
Deliberately a string, not a boolean: boolean workflow_call inputs do
|
||||||
doubles as a build check. Caveat: the checkout token comes from the
|
not reach the called workflow on this Gitea version (see issue #4).
|
||||||
same 1Password step, so without push the checkout falls back to the
|
With push: 'false' no credentials are fetched from 1Password and no
|
||||||
job token — which cannot clone private submodules.
|
registry login happens, so the workflow doubles as a build check.
|
||||||
|
Caveat: the checkout token comes from the same 1Password step, so
|
||||||
|
without push the checkout falls back to the job token — which cannot
|
||||||
|
clone private submodules.
|
||||||
required: false
|
required: false
|
||||||
default: true
|
default: 'true'
|
||||||
type: boolean
|
type: string
|
||||||
container-name:
|
container-name:
|
||||||
required: true
|
required: true
|
||||||
type: string
|
type: string
|
||||||
@@ -113,6 +134,24 @@ jobs:
|
|||||||
# gesetzt, bleibt die Variable leer und Docker nutzt seinen Default.
|
# gesetzt, bleibt die Variable leer und Docker nutzt seinen Default.
|
||||||
DOCKER_HOST: ${{ vars.DOCKER_HOST }}
|
DOCKER_HOST: ${{ vars.DOCKER_HOST }}
|
||||||
steps:
|
steps:
|
||||||
|
# Issue #4 blieb so lange unbemerkt, weil nirgends sichtbar war, was
|
||||||
|
# ueberhaupt ankommt — und weil ein Lauf ohne Push genauso gruen aussieht
|
||||||
|
# wie einer mit. Der Schritt macht beides sichtbar: den rohen Wert der
|
||||||
|
# Eingabe und den daraus abgeleiteten Modus. Gleiches Muster wie
|
||||||
|
# "Docker-Socket ermitteln": rein informativ, mit `|| true` abgesichert,
|
||||||
|
# damit er den Job unter keinen Umstaenden scheitern laesst.
|
||||||
|
#
|
||||||
|
# Der Wert wird ueber die Umgebung gereicht statt in die Kommandozeile
|
||||||
|
# interpoliert; so bleibt er auch dann harmlos, wenn ein Aufrufer etwas
|
||||||
|
# Unerwartetes uebergibt.
|
||||||
|
- name: Eingaben melden
|
||||||
|
env:
|
||||||
|
PUSH_ROH: ${{ inputs.push }}
|
||||||
|
PUSH_MODUS: ${{ (inputs.push && inputs.push != 'false') && 'veroeffentlichen' || 'nur bauen' }}
|
||||||
|
run: |
|
||||||
|
echo "push (roh): '${PUSH_ROH}'" || true
|
||||||
|
echo "Modus: ${PUSH_MODUS}" || true
|
||||||
|
|
||||||
# Der Token, den Gitea jedem Job stellt, taugt fuer zwei Dinge nicht:
|
# Der Token, den Gitea jedem Job stellt, taugt fuer zwei Dinge nicht:
|
||||||
# Er gilt nur fuer das ausfuehrende Repository, waehrend die Submodule in
|
# Er gilt nur fuer das ausfuehrende Repository, waehrend die Submodule in
|
||||||
# eigenen, privaten Repositories liegen — deren Klon scheitert an
|
# eigenen, privaten Repositories liegen — deren Klon scheitert an
|
||||||
@@ -135,6 +174,12 @@ jobs:
|
|||||||
# Strings sind truthy), `inputs.push == 'true'` waere beim echten Boolean
|
# Strings sind truthy), `inputs.push == 'true'` waere beim echten Boolean
|
||||||
# true falsch (ein Vergleich ueber Typgrenzen laeuft ueber Zahlen, 'true'
|
# true falsch (ein Vergleich ueber Typgrenzen laeuft ueber Zahlen, 'true'
|
||||||
# wird dabei zu NaN). Die Kombination aus beidem deckt alle vier Faelle ab.
|
# wird dabei zu NaN). Die Kombination aus beidem deckt alle vier Faelle ab.
|
||||||
|
#
|
||||||
|
# Genau deshalb blieb sie bei der Umstellung von push auf `type: string`
|
||||||
|
# (Issue #4) unveraendert: sie war von Anfang an fuer beide Schreibweisen
|
||||||
|
# gebaut. 'true' ergibt wahr, 'false' ergibt falsch, ein leerer Wert
|
||||||
|
# ergibt falsch — und die Boolean-Faelle gelten unveraendert weiter,
|
||||||
|
# falls ein Aufrufer doch einmal ein YAML-Boolean uebergibt.
|
||||||
- name: Zugangsdaten aus 1Password laden
|
- name: Zugangsdaten aus 1Password laden
|
||||||
id: op-secrets
|
id: op-secrets
|
||||||
if: ${{ inputs.push && inputs.push != 'false' }}
|
if: ${{ inputs.push && inputs.push != 'false' }}
|
||||||
|
|||||||
@@ -0,0 +1,249 @@
|
|||||||
|
name: Publish Helm Chart with patched Application Version
|
||||||
|
|
||||||
|
# Called through "uses:" only, never triggered here. The thin wrappers in the
|
||||||
|
# calling repository carry their own "workflow_dispatch" and "on: push", so the
|
||||||
|
# workflow_dispatch block this file used to have is gone: a dispatch in THIS
|
||||||
|
# repository would run against this repository's files, where there is no chart.
|
||||||
|
#
|
||||||
|
# The comments below are kept from the copy this file replaces, in English as
|
||||||
|
# they were written. Each of them records something that was paid for once -
|
||||||
|
# none of it is restating what the YAML already says.
|
||||||
|
on:
|
||||||
|
workflow_call:
|
||||||
|
inputs:
|
||||||
|
chart-location:
|
||||||
|
required: false
|
||||||
|
type: string
|
||||||
|
default: "."
|
||||||
|
description: "location of the helm chart directory"
|
||||||
|
chart-name:
|
||||||
|
required: true
|
||||||
|
type: string
|
||||||
|
description: "name of the helm chart"
|
||||||
|
# Deliberately required and without a default, following the house rule
|
||||||
|
# of this repository: an owner segment is never guessed. The caller
|
||||||
|
# passes "${{ github.repository_owner }}", where the value is measured
|
||||||
|
# (jlab-cluster, run 282 of validate-charts.yml) - inside a workflow
|
||||||
|
# called from another repository it is documented to stay the caller's
|
||||||
|
# owner, but that is not measured on this Gitea version, and a published
|
||||||
|
# chart landing under the wrong owner is not worth relying on a context
|
||||||
|
# semantic for. As an input the value is evaluated in the caller, where
|
||||||
|
# it is known to be right.
|
||||||
|
#
|
||||||
|
# type: string, like every input here. On this Gitea version (measured on
|
||||||
|
# 1.27.0, issue #4) a boolean workflow_call input does not reach the
|
||||||
|
# called workflow at all - neither its default nor an explicitly passed
|
||||||
|
# value - while the run stays green. Every input of this block is
|
||||||
|
# therefore a string, even one that is a switch in meaning.
|
||||||
|
registry-owner:
|
||||||
|
required: true
|
||||||
|
type: string
|
||||||
|
description: >-
|
||||||
|
Owner segment of the chart registry URL, i.e. the <owner> in
|
||||||
|
<base>/<owner>/helm/api/charts. Deliberately has no default: an owner
|
||||||
|
segment is never guessed. Callers pass github.repository_owner, which
|
||||||
|
resolves in the calling repository, where the value is measured.
|
||||||
|
outputs:
|
||||||
|
chart-version:
|
||||||
|
description: "the version of the published helm chart"
|
||||||
|
value: ${{ jobs.publish-helm-chart.outputs.chart-version }}
|
||||||
|
secrets:
|
||||||
|
OP_CONNECT_TOKEN:
|
||||||
|
required: true
|
||||||
|
jobs:
|
||||||
|
publish-helm-chart:
|
||||||
|
outputs:
|
||||||
|
chart-version: ${{ steps.get-chart-version.outputs.CHART_VERSION }}
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- name: "Checkout Repository"
|
||||||
|
uses: actions/checkout@v7
|
||||||
|
- name: "Install Helm"
|
||||||
|
uses: azure/setup-helm@v5
|
||||||
|
# The Connect host is a variable of the CALLING repository, not a value
|
||||||
|
# wired into this file: a shared building block must not carry one
|
||||||
|
# tenant's infrastructure address. The price of that is a prerequisite
|
||||||
|
# that can be forgotten, and an unset variable arrives as an empty
|
||||||
|
# string. An empty connect-host does not fail here - it fails two steps
|
||||||
|
# later, inside the loading action, against a URL nobody can see. So the
|
||||||
|
# guard comes first and names the setting, the place to set it and the
|
||||||
|
# shape of the value.
|
||||||
|
- name: "Check the calling repository's prerequisites"
|
||||||
|
env:
|
||||||
|
OP_CONNECT_HOST: ${{ vars.OP_CONNECT_HOST }}
|
||||||
|
run: |-
|
||||||
|
if [ -z "$OP_CONNECT_HOST" ]; then
|
||||||
|
echo "::error::the variable OP_CONNECT_HOST is not set in the calling repository (Settings -> Actions -> Variables). It must hold the address of the 1Password Connect server, e.g. https://op-connect.example.com. This workflow deliberately does not hardcode a host."
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
- name: "Configure OP-Connect"
|
||||||
|
uses: 1password/load-secrets-action/configure@v5
|
||||||
|
with:
|
||||||
|
connect-host: ${{ vars.OP_CONNECT_HOST }}
|
||||||
|
# helm package exits 1 with "found in Chart.yaml, but missing in charts/
|
||||||
|
# directory" while a declared dependency has not been fetched, so this
|
||||||
|
# comes first. It is a no-op for every chart without a dependencies:
|
||||||
|
# block; ci/helm-dependencies.sh explains why it resolves from Chart.yaml
|
||||||
|
# instead of a committed Chart.lock. Note the consequence for the
|
||||||
|
# published artifact: the subchart .tgz is packaged INTO ours, so the
|
||||||
|
# version pinned in Chart.yaml is the version operators get, and nothing
|
||||||
|
# downstream ever sees the dependency again.
|
||||||
|
#
|
||||||
|
# The script belongs to the CALLING repository and that is unchanged by
|
||||||
|
# the move: the checkout above clones the repository the run belongs to -
|
||||||
|
# the caller - so ${{ github.workspace }} is the caller's tree and the
|
||||||
|
# script is the caller's copy. It is therefore a prerequisite of every
|
||||||
|
# caller, the same way OP_CONNECT_HOST is, and the guard below says so
|
||||||
|
# rather than letting the shell report a missing file.
|
||||||
|
- name: "Fetch chart dependencies"
|
||||||
|
env:
|
||||||
|
HELM_DEPENDENCIES_SCRIPT: ${{ github.workspace }}/ci/helm-dependencies.sh
|
||||||
|
CHART_NAME: ${{ inputs.chart-name }}
|
||||||
|
run: |-
|
||||||
|
test -x "$HELM_DEPENDENCIES_SCRIPT" || {
|
||||||
|
echo "::error::${HELM_DEPENDENCIES_SCRIPT} is missing or not executable - ci/helm-dependencies.sh is a script of the calling repository and this workflow runs it before helm package"
|
||||||
|
exit 1
|
||||||
|
}
|
||||||
|
"$HELM_DEPENDENCIES_SCRIPT" "$CHART_NAME"
|
||||||
|
working-directory: ${{ inputs.chart-location }}
|
||||||
|
- name: "Package Helm Chart"
|
||||||
|
run: helm package ${{ inputs.chart-name }}
|
||||||
|
working-directory: ${{ inputs.chart-location }}
|
||||||
|
- name: "Get Helm Chart Version"
|
||||||
|
id: get-chart-version
|
||||||
|
run: |-
|
||||||
|
CHART_VERSION=$(grep '^version:' ${{ inputs.chart-name}}/Chart.yaml | awk '{print $2}')
|
||||||
|
echo $CHART_VERSION
|
||||||
|
echo "CHART_VERSION=$CHART_VERSION" >> $GITHUB_OUTPUT
|
||||||
|
working-directory: ${{ inputs.chart-location }}
|
||||||
|
- name: "Load Chart Registry Credentials"
|
||||||
|
id: load-registry-creds
|
||||||
|
uses: 1password/load-secrets-action@v5
|
||||||
|
env:
|
||||||
|
OP_CONNECT_TOKEN: ${{ secrets.OP_CONNECT_TOKEN }}
|
||||||
|
REGISTRY_PASSWORD: op://GitHub-Actions/jlab-registry-access/password
|
||||||
|
REGISTRY_USERNAME: op://GitHub-Actions/jlab-registry-access/username
|
||||||
|
REGISTRY_CHARTS_URL: op://GitHub-Actions/jlab-registry-access/charts-url
|
||||||
|
# The target is a classic, index-based Helm registry, not an OCI one, so
|
||||||
|
# there is no "helm registry login" session and no "helm push": the upload
|
||||||
|
# is a single HTTP POST of the packaged .tgz to
|
||||||
|
# <charts-url>/<owner>/helm/api/charts, authenticated per request.
|
||||||
|
#
|
||||||
|
# That URL is composed, not stored (jlab-cluster/helm-charts#63).
|
||||||
|
# charts-url holds only the base up to "/api/packages"; the owner segment
|
||||||
|
# comes from the registry-owner input, which the caller fills with
|
||||||
|
# github.repository_owner - measured as "jlab-cluster" in run 282 of
|
||||||
|
# validate-charts.yml, on a temporary step that was removed again. So no
|
||||||
|
# value pointing at the registry - not even a path segment - lives in the
|
||||||
|
# calling repository, and a second owner on the same instance needs no
|
||||||
|
# second setting. Two guards make the switch from the old shape loud
|
||||||
|
# instead of puzzling: charts-url is rejected unless it ends in
|
||||||
|
# "/api/packages" (an old full URL still carrying "/helm", "/api/charts"
|
||||||
|
# or the owner is named explicitly), and the owner is rejected when empty
|
||||||
|
# or when it holds characters that are not a path segment - an empty one
|
||||||
|
# would build "<base>//helm/api/charts", i.e. another bare 404 against a
|
||||||
|
# masked URL.
|
||||||
|
#
|
||||||
|
# Three further properties of this step are load-bearing and each of them
|
||||||
|
# was paid for once already.
|
||||||
|
#
|
||||||
|
# 1) The values are mapped into the step's own env block and read as quoted
|
||||||
|
# shell variables - never interpolated into the command line. load-secrets-
|
||||||
|
# action v5 no longer exports the loaded values as environment variables
|
||||||
|
# for later steps the way v4 did; only the step outputs carry them. Every
|
||||||
|
# "${{ env.* }}" reference therefore resolved to an empty string from that
|
||||||
|
# bump on, with nothing reporting an error - the loading step logs
|
||||||
|
# "Populating variable" and stays green either way. The ":?" guards fail
|
||||||
|
# loudly and name the op:// field that is missing, and quoting keeps an
|
||||||
|
# empty value from silently shifting arguments.
|
||||||
|
#
|
||||||
|
# 2) The credentials never reach the process list. "curl --user $U:$P"
|
||||||
|
# would put them in argv; "curl --config -" reads its options from stdin
|
||||||
|
# instead, so nothing sensitive appears in any argument. The price of that
|
||||||
|
# config file is its value syntax: an unquoted value ends at the first
|
||||||
|
# space (measured on curl 8.5.0 - "has space" arrived as "has"), so the
|
||||||
|
# step refuses whitespace in the credentials up front rather than letting
|
||||||
|
# a truncated password come back as a puzzling 401. A Gitea access token
|
||||||
|
# is 40 hex characters, so this only ever fires on a mis-entered item.
|
||||||
|
#
|
||||||
|
# 3) "--fail-with-body" is what makes a failed upload fail the job. Unlike
|
||||||
|
# "helm push", curl exits 0 on HTTP 401/403/409 unless asked not to - the
|
||||||
|
# publish would go green while nothing was uploaded. Measured against Gitea
|
||||||
|
# 1.24.7: a 401 without the flag is exit 0, with the flag exit 22, and the
|
||||||
|
# response body is still printed so the log names the cause.
|
||||||
|
#
|
||||||
|
# The file name is built from the chart name and the version read above
|
||||||
|
# instead of a "<chart>-*.tgz" glob: --upload-file takes exactly one file,
|
||||||
|
# and a glob matching two turns the second into curl's URL argument, while
|
||||||
|
# a glob matching none is passed through literally. Both fail obscurely.
|
||||||
|
# "helm package" keeps the "+" of a "1.4.0+up1.9.2" version verbatim in the
|
||||||
|
# file name, so "<name>-<version>.tgz" is an exact match (measured).
|
||||||
|
- name: "Upload Helm Chart to the Chart Registry"
|
||||||
|
env:
|
||||||
|
REGISTRY_PASSWORD: ${{ steps.load-registry-creds.outputs.REGISTRY_PASSWORD }}
|
||||||
|
REGISTRY_USERNAME: ${{ steps.load-registry-creds.outputs.REGISTRY_USERNAME }}
|
||||||
|
REGISTRY_CHARTS_URL: ${{ steps.load-registry-creds.outputs.REGISTRY_CHARTS_URL }}
|
||||||
|
REGISTRY_OWNER: ${{ inputs.registry-owner }}
|
||||||
|
CHART_NAME: ${{ inputs.chart-name }}
|
||||||
|
CHART_VERSION: ${{ steps.get-chart-version.outputs.CHART_VERSION }}
|
||||||
|
run: |-
|
||||||
|
: "${REGISTRY_CHARTS_URL:?empty - check op://GitHub-Actions/jlab-registry-access/charts-url}"
|
||||||
|
: "${REGISTRY_USERNAME:?empty - check op://GitHub-Actions/jlab-registry-access/username}"
|
||||||
|
: "${REGISTRY_PASSWORD:?empty - check op://GitHub-Actions/jlab-registry-access/password}"
|
||||||
|
: "${REGISTRY_OWNER:?empty - the registry-owner input did not arrive; the caller passes it from github.repository_owner, do not hardcode an owner}"
|
||||||
|
: "${CHART_NAME:?empty - the chart-name input did not arrive}"
|
||||||
|
: "${CHART_VERSION:?empty - could not read version: from ${CHART_NAME}/Chart.yaml}"
|
||||||
|
# 1Password masks the full URL, but a curl connection error prints only
|
||||||
|
# "<host>:<port>", which that mask would not cover. Mask the host too.
|
||||||
|
CHARTS_HOST="${REGISTRY_CHARTS_URL#*://}"
|
||||||
|
echo "::add-mask::${CHARTS_HOST%%/*}"
|
||||||
|
# charts-url is the base only; this workflow appends the owner, the
|
||||||
|
# package type and the endpoint. Reject the old, full shape by name so
|
||||||
|
# the failure says what to change in the vault item instead of 404ing
|
||||||
|
# against a URL the log has masked.
|
||||||
|
CHARTS_BASE="${REGISTRY_CHARTS_URL%/}"
|
||||||
|
case "$CHARTS_BASE" in
|
||||||
|
*/api/charts)
|
||||||
|
echo "::error::op://GitHub-Actions/jlab-registry-access/charts-url still ends in /api/charts - it must hold only the base up to /api/packages (https://<host>/api/packages); this workflow appends /<owner>/helm/api/charts itself"
|
||||||
|
exit 1
|
||||||
|
;;
|
||||||
|
*/helm|*/helm/*)
|
||||||
|
echo "::error::op://GitHub-Actions/jlab-registry-access/charts-url still contains the /helm path segment - it must hold only the base up to /api/packages (https://<host>/api/packages); this workflow appends /<owner>/helm/api/charts itself"
|
||||||
|
exit 1
|
||||||
|
;;
|
||||||
|
*/api/packages) ;;
|
||||||
|
*)
|
||||||
|
echo "::error::op://GitHub-Actions/jlab-registry-access/charts-url must end in /api/packages (https://<host>/api/packages) - no owner, no package type, no endpoint; this workflow appends /<owner>/helm/api/charts itself"
|
||||||
|
exit 1
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
# An empty owner would build "<base>//helm/api/charts", a stray "/" or
|
||||||
|
# "?" would build something else entirely - both are bare 404s.
|
||||||
|
case "$REGISTRY_OWNER" in
|
||||||
|
*[!A-Za-z0-9._-]*)
|
||||||
|
echo "::error::the registry-owner input is not usable as a URL path segment: '${REGISTRY_OWNER}'"
|
||||||
|
exit 1
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
CHARTS_API_URL="${CHARTS_BASE}/${REGISTRY_OWNER}/helm/api/charts"
|
||||||
|
case "${REGISTRY_USERNAME}${REGISTRY_PASSWORD}" in
|
||||||
|
*[[:space:]]*)
|
||||||
|
echo "::error::username/password in op://GitHub-Actions/jlab-registry-access must not contain whitespace - curl --config truncates an unquoted value at the first space"
|
||||||
|
exit 1
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
CHART_PACKAGE="${CHART_NAME}-${CHART_VERSION}.tgz"
|
||||||
|
test -f "$CHART_PACKAGE" || {
|
||||||
|
echo "::error::${CHART_PACKAGE} does not exist - helm package produced: $(ls -1 ./*.tgz 2>&1)"
|
||||||
|
exit 1
|
||||||
|
}
|
||||||
|
echo "uploading ${CHART_PACKAGE}"
|
||||||
|
printf 'user = %s:%s\nurl = %s\n' \
|
||||||
|
"$REGISTRY_USERNAME" "$REGISTRY_PASSWORD" "$CHARTS_API_URL" |
|
||||||
|
curl --config - \
|
||||||
|
--fail-with-body \
|
||||||
|
--silent --show-error \
|
||||||
|
--request POST \
|
||||||
|
--upload-file "$CHART_PACKAGE" \
|
||||||
|
--write-out '\nHTTP %{http_code}\n'
|
||||||
|
working-directory: ${{ inputs.chart-location }}
|
||||||
@@ -4,6 +4,12 @@ Geteilte CI/CD-Bausteine für die Gitea-Instanz `git.jk-dev.de`. Die Workflows
|
|||||||
hier werden nicht selbst ausgelöst, sondern von anderen Repositories über
|
hier werden nicht selbst ausgelöst, sondern von anderen Repositories über
|
||||||
`uses:` aufgerufen (`on: workflow_call`).
|
`uses:` aufgerufen (`on: workflow_call`).
|
||||||
|
|
||||||
|
Diese Datei ist die **Kurzreferenz für Aufrufer**: was ein Baustein tut, seine
|
||||||
|
Eingaben, ein Aufrufbeispiel, die Voraussetzungen. Wer einen Aufruf schreibt,
|
||||||
|
braucht nur das. Die Langform — warum die Dinge so sind und was dafür gemessen
|
||||||
|
wurde — steht im [Wiki](https://git.jk-dev.de/workflow-util/pipelines/wiki),
|
||||||
|
eine Seite je Baustein und eine für die Eigenheiten dieser Forge.
|
||||||
|
|
||||||
## Bausteine
|
## Bausteine
|
||||||
|
|
||||||
### `.gitea/workflows/docker-build-push.yml`
|
### `.gitea/workflows/docker-build-push.yml`
|
||||||
@@ -25,7 +31,7 @@ Gepusht wird unter fünf Tags:
|
|||||||
Der Checkout holt Submodule mit; die Zugangsdaten dafür und für die Registry
|
Der Checkout holt Submodule mit; die Zugangsdaten dafür und für die Registry
|
||||||
kommen zur Laufzeit aus 1Password Connect.
|
kommen zur Laufzeit aus 1Password Connect.
|
||||||
|
|
||||||
Mit `push: false` entfällt der Push — der Baustein baut dann nur. Damit nutzt
|
Mit `push: 'false'` entfällt der Push — der Baustein baut dann nur. Damit nutzt
|
||||||
die CI eines Projekts dieselbe Bau-Logik wie das Deployment, statt sie zu
|
die CI eines Projekts dieselbe Bau-Logik wie das Deployment, statt sie zu
|
||||||
duplizieren. Siehe [Prüfmodus](#prüfmodus-ohne-push).
|
duplizieren. Siehe [Prüfmodus](#prüfmodus-ohne-push).
|
||||||
|
|
||||||
@@ -37,7 +43,7 @@ duplizieren. Siehe [Prüfmodus](#prüfmodus-ohne-push).
|
|||||||
| `dockerfile` | ja | — | Pfad zum Dockerfile |
|
| `dockerfile` | ja | — | Pfad zum Dockerfile |
|
||||||
| `docker-context` | ja | — | Build-Kontext |
|
| `docker-context` | ja | — | Build-Kontext |
|
||||||
| `image-owner` | ja | — | Eigentümer-Segment des Images, also der mittlere Teil von `<registry>/<owner>/<name>` |
|
| `image-owner` | ja | — | Eigentümer-Segment des Images, also der mittlere Teil von `<registry>/<owner>/<name>` |
|
||||||
| `push` | nein | `true` | Ob das Image gepusht wird. Bei `false` werden keine Zugangsdaten aus 1Password geholt und es findet kein Registry-Login statt — siehe [Prüfmodus](#prüfmodus-ohne-push) |
|
| `push` | nein | `'true'` | Zeichenkette `'true'` oder `'false'`. Ob das Image gepusht wird. Bei `'false'` werden keine Zugangsdaten aus 1Password geholt und es findet kein Registry-Login statt — siehe [Prüfmodus](#prüfmodus-ohne-push) |
|
||||||
| `registry` | nein | `git.jk-dev.de` | Registry, für die getaggt und — beim Push — bei der angemeldet wird |
|
| `registry` | nein | `git.jk-dev.de` | Registry, für die getaggt und — beim Push — bei der angemeldet wird |
|
||||||
| `version-number` | nein | `0.0.0` | Vollständige Version, wird als Tag gesetzt |
|
| `version-number` | nein | `0.0.0` | Vollständige Version, wird als Tag gesetzt |
|
||||||
| `version-number-major` | nein | `0` | Major-Anteil für die mitwandernden Tags |
|
| `version-number-major` | nein | `0` | Major-Anteil für die mitwandernden Tags |
|
||||||
@@ -52,6 +58,23 @@ ist je Projekt verschieden und soll nie stillschweigend geraten werden — genau
|
|||||||
das war der Fehler des früher fest verdrahteten `suer`, unter dem die Images
|
das war der Fehler des früher fest verdrahteten `suer`, unter dem die Images
|
||||||
aller Projekte landeten. Die Registry ist dagegen praktisch immer dieselbe.
|
aller Projekte landeten. Die Registry ist dagegen praktisch immer dieselbe.
|
||||||
|
|
||||||
|
`push` ist eine Zeichenkette und kein Boolean — und das ist Absicht. Auf dieser
|
||||||
|
Gitea-Version (gemessen an 1.27.0) werden **boolesche `workflow_call`-Eingaben
|
||||||
|
nicht an den aufgerufenen Workflow durchgereicht**: Weder ihre Vorgabe noch ein
|
||||||
|
ausdrücklich übergebenes `push: true` kamen an, `inputs.push` blieb leer, und
|
||||||
|
der Baustein baute immer im Prüfmodus — bei grünem Lauf. Alle Eingaben vom Typ
|
||||||
|
`string` kamen im selben Lauf korrekt an. Details in
|
||||||
|
[Issue #4](https://git.jk-dev.de/workflow-util/pipelines/issues/4).
|
||||||
|
|
||||||
|
Deshalb bitte immer in Anführungszeichen schreiben: `push: 'true'` bzw.
|
||||||
|
`push: 'false'`. Ein unquotiertes `push: false` ist im YAML ein Boolean und
|
||||||
|
wird verschluckt.
|
||||||
|
|
||||||
|
> **Brechende Änderung gegenüber `v1`.** Aufrufer, die `push` als YAML-Boolean
|
||||||
|
> übergeben, müssen auf `@v2` wechseln und dabei auf die Schreibweise mit
|
||||||
|
> Anführungszeichen umstellen. Wer `@v1` weiterverwendet, behält das alte
|
||||||
|
> Verhalten samt Fehler.
|
||||||
|
|
||||||
Die vier Versionsangaben dienen ausschließlich der Tag-Liste. Sie sind keine
|
Die vier Versionsangaben dienen ausschließlich der Tag-Liste. Sie sind keine
|
||||||
Pflicht mehr, damit ein Prüflauf keine Platzhalter erfinden muss; wer pusht,
|
Pflicht mehr, damit ein Prüflauf keine Platzhalter erfinden muss; wer pusht,
|
||||||
übergibt sie wie bisher. Ihre Vorgabe ist bewusst `0.0.0` und nicht leer — die
|
übergibt sie wie bisher. Ihre Vorgabe ist bewusst `0.0.0` und nicht leer — die
|
||||||
@@ -63,7 +86,7 @@ Tag-Liste wird auch ohne Push gebildet, und ein leerer Wert ergäbe das ungülti
|
|||||||
```yaml
|
```yaml
|
||||||
jobs:
|
jobs:
|
||||||
image:
|
image:
|
||||||
uses: workflow-util/pipelines/.gitea/workflows/docker-build-push.yml@v1
|
uses: workflow-util/pipelines/.gitea/workflows/docker-build-push.yml@v2
|
||||||
with:
|
with:
|
||||||
version-number: 2026.09.1
|
version-number: 2026.09.1
|
||||||
version-number-major: '2026'
|
version-number-major: '2026'
|
||||||
@@ -73,6 +96,7 @@ jobs:
|
|||||||
dockerfile: ./Dockerfile
|
dockerfile: ./Dockerfile
|
||||||
docker-context: .
|
docker-context: .
|
||||||
image-owner: bekemeyer-klein
|
image-owner: bekemeyer-klein
|
||||||
|
push: 'true'
|
||||||
secrets: inherit
|
secrets: inherit
|
||||||
```
|
```
|
||||||
|
|
||||||
@@ -81,7 +105,7 @@ dem aufrufenden Repository.
|
|||||||
|
|
||||||
#### Prüfmodus ohne Push
|
#### Prüfmodus ohne Push
|
||||||
|
|
||||||
Mit `push: false` baut der Baustein nur und veröffentlicht nichts. Es werden
|
Mit `push: 'false'` baut der Baustein nur und veröffentlicht nichts. Es werden
|
||||||
dann keine Zugangsdaten aus 1Password geholt und es findet kein Registry-Login
|
dann keine Zugangsdaten aus 1Password geholt und es findet kein Registry-Login
|
||||||
statt. Die CI eines Projekts nutzt damit dieselbe Bau-Logik wie das Deployment,
|
statt. Die CI eines Projekts nutzt damit dieselbe Bau-Logik wie das Deployment,
|
||||||
statt sie zu duplizieren.
|
statt sie zu duplizieren.
|
||||||
@@ -91,15 +115,21 @@ Weder Secrets noch Versionsangaben sind dafür nötig:
|
|||||||
```yaml
|
```yaml
|
||||||
jobs:
|
jobs:
|
||||||
build-check:
|
build-check:
|
||||||
uses: workflow-util/pipelines/.gitea/workflows/docker-build-push.yml@v1
|
uses: workflow-util/pipelines/.gitea/workflows/docker-build-push.yml@v2
|
||||||
with:
|
with:
|
||||||
container-name: bk-dev-website
|
container-name: bk-dev-website
|
||||||
dockerfile: ./Dockerfile
|
dockerfile: ./Dockerfile
|
||||||
docker-context: .
|
docker-context: .
|
||||||
image-owner: bekemeyer-klein
|
image-owner: bekemeyer-klein
|
||||||
push: false
|
push: 'false'
|
||||||
```
|
```
|
||||||
|
|
||||||
|
Die Anführungszeichen sind nicht kosmetisch: `push: false` ohne sie ist ein
|
||||||
|
YAML-Boolean und kommt im Baustein nicht an (siehe oben). Es griffe dann die
|
||||||
|
Vorgabe `'true'`, der Lauf versuchte also trotz `push: false` zu
|
||||||
|
veröffentlichen — sofern Gitea die Typabweichung nicht ohnehin als Fehler
|
||||||
|
zurückweist.
|
||||||
|
|
||||||
> **Private Submodule funktionieren in diesem Modus nicht.**
|
> **Private Submodule funktionieren in diesem Modus nicht.**
|
||||||
> Der Checkout-Token stammt aus demselben 1Password-Schritt, der ohne Push
|
> Der Checkout-Token stammt aus demselben 1Password-Schritt, der ohne Push
|
||||||
> übersprungen wird. Der Checkout fällt deshalb auf den Standard-Token des Jobs
|
> übersprungen wird. Der Checkout fällt deshalb auf den Standard-Token des Jobs
|
||||||
@@ -124,12 +154,127 @@ Die eigentlichen Zugänge für Checkout und Registry liegen in 1Password
|
|||||||
(`GitHub-Actions/jlab-git-checkout-token`, `GitHub-Actions/jlab-registry-access`)
|
(`GitHub-Actions/jlab-git-checkout-token`, `GitHub-Actions/jlab-registry-access`)
|
||||||
und müssen in Gitea nicht hinterlegt werden.
|
und müssen in Gitea nicht hinterlegt werden.
|
||||||
|
|
||||||
|
### `.gitea/workflows/publish-helm-chart.yml`
|
||||||
|
|
||||||
|
Packt ein Helm-Chart und lädt es in eine **klassische, indexbasierte**
|
||||||
|
Helm-Registry. Das ist kein OCI-`helm push`: Es gibt keine Registry-Sitzung,
|
||||||
|
sondern ein einzelnes HTTP-`POST` des gepackten `.tgz`, je Anfrage
|
||||||
|
authentifiziert. Läuft auf `ubuntu-latest`.
|
||||||
|
|
||||||
|
Die Ziel-URL wird zusammengesetzt, nicht gespeichert:
|
||||||
|
|
||||||
|
```
|
||||||
|
<charts-url>/<registry-owner>/helm/api/charts
|
||||||
|
```
|
||||||
|
|
||||||
|
`<charts-url>` ist die Basis bis `/api/packages` und liegt in 1Password
|
||||||
|
(`GitHub-Actions/jlab-registry-access`, Feld `charts-url`), `<registry-owner>`
|
||||||
|
ist die Pflichteingabe. Damit steht im aufrufenden Repository kein Wert, der auf
|
||||||
|
die Registry zeigt — nicht einmal ein Pfadsegment —, und ein zweiter Eigentümer
|
||||||
|
auf derselben Instanz braucht keine zweite Einstellung.
|
||||||
|
|
||||||
|
#### Eingaben
|
||||||
|
|
||||||
|
| Eingabe | Pflicht | Vorgabe | Bedeutung |
|
||||||
|
| ---------------- | ------- | ------- | --------- |
|
||||||
|
| `chart-name` | ja | — | Name des Charts, zugleich der Name seines Verzeichnisses unterhalb von `chart-location` |
|
||||||
|
| `registry-owner` | ja | — | Eigentümer-Segment der Ziel-URL, also der `<owner>` in `<basis>/<owner>/helm/api/charts` |
|
||||||
|
| `chart-location` | nein | `.` | Verzeichnis, in dem das Chart-Verzeichnis liegt |
|
||||||
|
|
||||||
|
Dass `registry-owner` Pflicht ist und keine Vorgabe hat, folgt derselben
|
||||||
|
Hausregel wie `image-owner` oben: Ein Eigentümer-Segment wird nie geraten.
|
||||||
|
Aufrufer übergeben `${{ github.repository_owner }}`; der Wert wird damit im
|
||||||
|
**aufrufenden** Repository ausgewertet, wo er gemessen ist (`jlab-cluster`, Lauf
|
||||||
|
282 von `validate-charts.yml`). Dass der Kontext auch in einem von fremd
|
||||||
|
aufgerufenen Workflow der des Aufrufers bleibt, ist dokumentiert, auf dieser
|
||||||
|
Gitea-Fassung aber nicht gemessen — und ein Chart, das unter dem falschen
|
||||||
|
Eigentümer veröffentlicht wird, ist nichts, wofür man sich auf eine
|
||||||
|
Kontext-Semantik verlassen müsste. Als Eingabe hängt der Baustein an keiner.
|
||||||
|
|
||||||
|
Alle Eingaben sind `type: string`, auch dort, wo künftig ein Schalter
|
||||||
|
hinzukommt: Boolesche `workflow_call`-Eingaben werden auf dieser Gitea-Version
|
||||||
|
nicht durchgereicht — bei grünem Lauf. Siehe
|
||||||
|
[Issue #4](https://git.jk-dev.de/workflow-util/pipelines/issues/4) und die
|
||||||
|
Begründung im Abschnitt zu `push` oben.
|
||||||
|
|
||||||
|
#### Ausgaben
|
||||||
|
|
||||||
|
| Ausgabe | Bedeutung |
|
||||||
|
| --------------- | --------- |
|
||||||
|
| `chart-version` | Die veröffentlichte Chart-Version, gelesen aus `version:` in `Chart.yaml` |
|
||||||
|
|
||||||
|
#### Aufrufbeispiel
|
||||||
|
|
||||||
|
```yaml
|
||||||
|
jobs:
|
||||||
|
publish-outline:
|
||||||
|
name: Publish Outline Helm Chart
|
||||||
|
uses: workflow-util/pipelines/.gitea/workflows/publish-helm-chart.yml@v3
|
||||||
|
with:
|
||||||
|
chart-name: 'outline'
|
||||||
|
registry-owner: ${{ github.repository_owner }}
|
||||||
|
secrets: inherit
|
||||||
|
```
|
||||||
|
|
||||||
|
`secrets: inherit` ist notwendig — der Workflow braucht `OP_CONNECT_TOKEN` aus
|
||||||
|
dem aufrufenden Repository.
|
||||||
|
|
||||||
|
Der Baustein löst sich nicht selbst aus. Wann veröffentlicht wird, entscheidet
|
||||||
|
die Hülle im aufrufenden Repository über ihr eigenes `on:` — üblicherweise ein
|
||||||
|
`push` auf `main`, eingeschränkt auf das Chart-Verzeichnis, dazu ein
|
||||||
|
`workflow_dispatch` für den Fall, dass von Hand nachgeschoben werden muss.
|
||||||
|
|
||||||
|
#### Form des Tresor-Felds `charts-url`
|
||||||
|
|
||||||
|
Das Feld enthält **nur** die Basis:
|
||||||
|
|
||||||
|
```
|
||||||
|
https://<host>/api/packages # so
|
||||||
|
https://<host>/api/packages/<owner>/helm/api/charts # nicht so
|
||||||
|
```
|
||||||
|
|
||||||
|
Drei Wächter weisen eine andere Form namentlich zurück: wenn der Wert auf
|
||||||
|
`/api/charts` endet, wenn er das Segment `/helm` enthält, und wenn er nicht auf
|
||||||
|
`/api/packages` endet. Der Grund ist die Lesbarkeit des Fehlers — 1Password
|
||||||
|
maskiert die URL im Protokoll, ein 404 gegen eine maskierte Adresse wäre nicht zu
|
||||||
|
deuten. Der Eigentümer wird aus demselben Grund geprüft: Ein leerer Wert ergäbe
|
||||||
|
`<basis>//helm/api/charts`, also wieder einen nackten 404.
|
||||||
|
|
||||||
|
#### Voraussetzungen im aufrufenden Repository
|
||||||
|
|
||||||
|
| Name | Art | Zweck |
|
||||||
|
| ------------------------- | ----------------- | ----- |
|
||||||
|
| `OP_CONNECT_HOST` | Variable | Adresse des 1Password-Connect-Servers im Cluster, etwa `https://op-connect.example.com` |
|
||||||
|
| `OP_CONNECT_TOKEN` | Secret | Zugang zu diesem Connect-Server, kommt über `secrets: inherit` |
|
||||||
|
| `ci/helm-dependencies.sh` | Datei im Repository | Ausführbares Skript, das die deklarierten Abhängigkeiten eines Charts nach `charts/` holt |
|
||||||
|
|
||||||
|
`OP_CONNECT_HOST` ist bewusst eine Variable des aufrufenden Repositories und
|
||||||
|
kein fest verdrahteter Wert: Ein geteilter Baustein trägt nicht die
|
||||||
|
Infrastruktur-Adresse eines einzelnen Mandanten. Der Preis dafür ist eine
|
||||||
|
Voraussetzung, die man vergessen kann, und eine nicht gesetzte Variable kommt als
|
||||||
|
leere Zeichenkette an. Deshalb prüft der erste Schritt sie und benennt beim
|
||||||
|
Abbruch die Einstellung samt Ort — statt zwei Schritte später an einer leeren
|
||||||
|
Adresse zu scheitern.
|
||||||
|
|
||||||
|
`ci/helm-dependencies.sh` ist ein Skript des **aufrufenden** Repositories, und
|
||||||
|
das bleibt nach dem Umzug hierher so: Der Checkout holt das Repository, zu dem
|
||||||
|
der Lauf gehört — das des Aufrufers —, also ist `${{ github.workspace }}` dessen
|
||||||
|
Arbeitsverzeichnis. Der Aufruf steht vor `helm package`, weil `helm package` mit
|
||||||
|
`found in Chart.yaml, but missing in charts/ directory` abbricht, solange eine
|
||||||
|
deklarierte Abhängigkeit nicht geholt ist. Für ein Chart ohne `dependencies:`
|
||||||
|
tut das Skript nichts, vorhanden und ausführbar muss es trotzdem sein; fehlt es,
|
||||||
|
sagt der Lauf genau das.
|
||||||
|
|
||||||
|
Der Registry-Zugang selbst liegt in 1Password
|
||||||
|
(`GitHub-Actions/jlab-registry-access`: `username`, `password`, `charts-url`) und
|
||||||
|
muss in Gitea nicht hinterlegt werden.
|
||||||
|
|
||||||
## Auf einen Tag zeigen, nicht auf `@main`
|
## Auf einen Tag zeigen, nicht auf `@main`
|
||||||
|
|
||||||
Aufrufe bitte immer an einen Tag binden:
|
Aufrufe bitte immer an einen Tag binden:
|
||||||
|
|
||||||
```yaml
|
```yaml
|
||||||
uses: workflow-util/pipelines/.gitea/workflows/docker-build-push.yml@v1 # so
|
uses: workflow-util/pipelines/.gitea/workflows/docker-build-push.yml@v2 # so
|
||||||
uses: workflow-util/pipelines/.gitea/workflows/docker-build-push.yml@main # nicht so
|
uses: workflow-util/pipelines/.gitea/workflows/docker-build-push.yml@main # nicht so
|
||||||
```
|
```
|
||||||
|
|
||||||
@@ -140,3 +285,24 @@ im jeweiligen Projekt.
|
|||||||
|
|
||||||
Änderungen hier bekommen deshalb einen neuen Tag. Bricht eine Änderung
|
Änderungen hier bekommen deshalb einen neuen Tag. Bricht eine Änderung
|
||||||
bestehende Aufrufe, wird der Major-Anteil erhöht.
|
bestehende Aufrufe, wird der Major-Anteil erhöht.
|
||||||
|
|
||||||
|
`v2` ist genau so ein Fall: Die Umstellung von `push` auf `type: string` bricht
|
||||||
|
Aufrufe, die den Wert als YAML-Boolean übergeben. `v1` bleibt unverändert
|
||||||
|
stehen; wer umstellt, wechselt auf `@v2` und schreibt `push: 'true'` bzw.
|
||||||
|
`push: 'false'`.
|
||||||
|
|
||||||
|
### Die Nummer bezeichnet den Zustand des Repositories
|
||||||
|
|
||||||
|
Tags gelten hier **repository-weit**, nicht je Datei. `v3` ist der Stand, ab dem
|
||||||
|
es `publish-helm-chart.yml` gibt; unter `v1` und `v2` existiert die Datei nicht,
|
||||||
|
ein Aufruf auf `@v2` findet sie also nicht.
|
||||||
|
|
||||||
|
Das hat eine Folge, die man kennen muss, weil sie sonst als Fehler gelesen wird:
|
||||||
|
Wer künftig `docker-build-push.yml` ändert, erhöht auf `v4` — und ein Aufruf von
|
||||||
|
`publish-helm-chart.yml@v3` bleibt gültig und richtig, obwohl sich an dieser
|
||||||
|
Datei nichts geändert hat. Die Nummer ist keine Dateiversion. Umgekehrt zwingt
|
||||||
|
eine brechende Änderung an einem Baustein die Aufrufer der anderen zu nichts:
|
||||||
|
Sie bleiben auf ihrem Tag stehen, bis sie bewusst nachziehen.
|
||||||
|
|
||||||
|
`v3` selbst ist additiv. Der neue Baustein ändert an `docker-build-push.yml`
|
||||||
|
nichts, bestehende Aufrufer auf `@v2` bleiben unberührt.
|
||||||
Reference in new issue
Block a user