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

build docker images on PR #2053

Closed
wants to merge 13 commits into from

Conversation

kradalby
Copy link
Collaborator

Sometimes we want people to test features in PRs and not everyone is used to using git, build go and docker.

This commit builds docker containers and pushes them to GHCR (not dockerhub) for testing on pushes to branches that has open pull requests to main using Ko.
This is configured to mimic the debug images produced by goreleaser.

Sometimes we want people to test features in PRs and
not everyone is used to using git, build go and docker.

This commit builds docker containers and pushes them to
GHCR (not dockerhub) for testing on pushes to branches
that has open pull requests to main using Ko.
This is configured to mimic the debug images produced
by goreleaser.

Signed-off-by: Kristoffer Dalby <[email protected]>
Signed-off-by: Kristoffer Dalby <[email protected]>
Signed-off-by: Kristoffer Dalby <[email protected]>
Signed-off-by: Kristoffer Dalby <[email protected]>
Signed-off-by: Kristoffer Dalby <[email protected]>
Signed-off-by: Kristoffer Dalby <[email protected]>
Signed-off-by: Kristoffer Dalby <[email protected]>
Signed-off-by: Kristoffer Dalby <[email protected]>
Signed-off-by: Kristoffer Dalby <[email protected]>
Signed-off-by: Kristoffer Dalby <[email protected]>
@kradalby
Copy link
Collaborator Author

I think we need more permissions to the GITHUB_TOKEN to allow for pushes from PRs, sensibly disabled by default since it might be code executed by random people.

I think it is fine to enable this because we have "approve to run workflows" enabled, so we can do a quick review to check if something looks suspicious before allowing a pr container to be published.

@juanfont @ohdearaugustin

@kradalby
Copy link
Collaborator Author

Think we also need a read access to look up PRs too so we can get the PR number for tag name.

Signed-off-by: Kristoffer Dalby <[email protected]>
@kradalby kradalby closed this Aug 13, 2024
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.

1 participant