Skip to content

Add bitdeps/helm-oci-charts-releaser#261

Open
muffl0n wants to merge 1 commit intosubshell:mainfrom
NorddeutscherRundfunk:add-oci-releaser
Open

Add bitdeps/helm-oci-charts-releaser#261
muffl0n wants to merge 1 commit intosubshell:mainfrom
NorddeutscherRundfunk:add-oci-releaser

Conversation

@muffl0n
Copy link
Contributor

@muffl0n muffl0n commented Feb 25, 2026

No description provided.

Comment on lines +3 to +6
permissions:
contents: read
packages: write

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Für die Tests in NorddeutscherRundfunk#2 brauchte ich das. Kann aber gut sein, dass das hier anders ist. 🤔

oci_username: xxx
oci_password: '${{ secrets.GITHUB_TOKEN }}'
github_token: '${{ secrets.GITHUB_TOKEN }}'
skip_gh_release: 'true'
Copy link
Contributor Author

@muffl0n muffl0n Feb 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Die Releases werden ja schon in Run chart-releaser gemacht, deswegen muss das hier geskipped werden. Es kann aber gut sein, dass hier das Erkennen von latest nicht mehr richtig funktioniert, weil Run chart-releaser schon ein Tag gesetzt hat.

Also vielleicht wäre es das beste, hier skip_gh_release und auch den Step Run chart-releaser zu entfernen. Dann macht Run chart-releaser - oci alles und die beiden kommen sich nicht in die Quere.

uses: bitdeps/helm-oci-charts-releaser@v0.1.5
with:
oci_registry: ghcr.io/subshell/helm-charts
oci_username: xxx
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hier hatte ich in meinen Tests meinen Username gesetzt. Richtig erscheint mir das aber nicht. Vielleicht reicht da ein dummy?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant