Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get Release Helm chart Github action working #69

Merged
merged 10 commits into from
Apr 23, 2024
Merged

Conversation

MichelHollands
Copy link
Contributor

@MichelHollands MichelHollands commented Apr 22, 2024

  • Use the Github app for permissions
  • Add documentation

This based on #66 which took the Helm chart publishing code from https://github.com/grafana/k8s-monitoring-helm.

Signed-off-by: Michel Hollands <[email protected]>
Signed-off-by: Michel Hollands <[email protected]>
Signed-off-by: Michel Hollands <[email protected]>
Signed-off-by: Michel Hollands <[email protected]>
Signed-off-by: Michel Hollands <[email protected]>
Signed-off-by: Michel Hollands <[email protected]>
@MichelHollands MichelHollands changed the title Use uppercase keys Fix CI Apr 22, 2024
Signed-off-by: Michel Hollands <[email protected]>
Signed-off-by: Michel Hollands <[email protected]>
Signed-off-by: Michel Hollands <[email protected]>
Signed-off-by: Michel Hollands <[email protected]>
@MichelHollands MichelHollands changed the title Fix CI Get Release Helm chart Github action working Apr 22, 2024
@MichelHollands MichelHollands requested review from a team, DylanGuedes and slim-bean April 22, 2024 13:03
@@ -8,9 +8,6 @@ env:
CR_PACKAGE_PATH: "${{ github.workspace }}/.cr-release-packages"
CR_TOOL_PATH: "${{ github.workspace }}/.cr-tool"
CR_VERSION: "1.5.0"
permissions:
contents: read
id-token: write
Copy link
Collaborator

Choose a reason for hiding this comment

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

don't you need the id-token for the github app token to work?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, it works without. A v0.0.1 release was built.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am using the actions/create-github-app-token@v1 action. That creates a token from the github app app-ID and private key.

@MichelHollands MichelHollands merged commit e81b124 into main Apr 23, 2024
1 check passed
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