Skip to content

Fix cascading-hooks labels#3573

Open
aveyrenc wants to merge 1 commit intosecureCodeBox:mainfrom
aveyrenc:fix-cascading-hooks-labels
Open

Fix cascading-hooks labels#3573
aveyrenc wants to merge 1 commit intosecureCodeBox:mainfrom
aveyrenc:fix-cascading-hooks-labels

Conversation

@aveyrenc
Copy link

@aveyrenc aveyrenc commented Mar 25, 2026

When supplying multiple labels for the cascading scans hook, we end up with an indention error.

Values:

hook:
  labels:
    app: securecodebox
    hooks: cascading

Result:

apiVersion: "execution.securecodebox.io/v1"
kind: ScanCompletionHook
metadata:
  name: release-name-cascading-scans
  labels:
    helm.sh/chart: cascading-scans-5.6.0
    app.kubernetes.io/name: cascading-scans
    app.kubernetes.io/instance: release-name
    app.kubernetes.io/managed-by: Helm
    securecodebox.io/internal: "true"
    app: securecodebox
hooks: cascading
spec:
  priority: 100
  type: ReadOnly

As you can see, the hooks: cascading label is not indented.

This PR fixes labels indentation

@netlify
Copy link

netlify bot commented Mar 25, 2026

Deploy Preview for docs-securecodebox canceled.

Name Link
🔨 Latest commit ee945ec
🔍 Latest deploy log https://app.netlify.com/projects/docs-securecodebox/deploys/69c40027ab9cc30008acfd17

@Weltraumschaf Weltraumschaf moved this from Triage to To Review in secureCodeBox Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: To Review

Development

Successfully merging this pull request may close these issues.

2 participants