Skip to content

add secrets block to workflow#23

Merged
mbaldessari merged 1 commit intovalidatedpatterns:mainfrom
dminnear-rh:add-secret-for-chart-token
Oct 6, 2025
Merged

add secrets block to workflow#23
mbaldessari merged 1 commit intovalidatedpatterns:mainfrom
dminnear-rh:add-secret-for-chart-token

Conversation

@dminnear-rh
Copy link
Contributor

@mbaldessari This enables us to prpoerly pass the chart repo token secret:

name: vp-patterns/update-helm-repo

on:
  push:
    tags:
      - "v[0-9]+.[0-9]+.[0-9]+"

permissions:
  contents: read

jobs:
  helmlint:
    uses: dminnear-rh/helm-charts/.github/workflows/helmlint.yml@29d4c96c70faf37d0cf8f48206e598ce45a7b77e # September 8, 2025
    permissions:
      contents: read

  update-helm-repo:
    needs: [helmlint]
    uses: dminnear-rh/helm-charts/.github/workflows/update-helm-repo.yml@29d4c96c70faf37d0cf8f48206e598ce45a7b77e # September 8, 2025
    permissions:
      contents: read
    secrets:
      CHARTS_REPOS_TOKEN: ${{ secrets.CHARTS_REPOS_TOKEN }}

I tested it with a new chart (llm-inference-service). We'll need to add this before we can update the workflow SHA in validatedpatterns/vp-template-chart#4 and enable new charts to pass the zizmor linter

@mbaldessari mbaldessari merged commit 69fd10e into validatedpatterns:main Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments