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

Enable publishing a Docker image from a branch #913

Merged
merged 11 commits into from
Dec 13, 2024

Conversation

palinatolmach
Copy link
Collaborator

@palinatolmach palinatolmach commented Dec 12, 2024

Related: https://github.com/runtimeverification/kaas/pull/996.

This PR adds a workflow that allows publishing kontrol image to Dockerhub ghrc.io, i.e., our GitHub org packages, from a specific branch. The image will be named runtimeverification/kontrol:ubuntu-jammy-{branch_name}.

Additionally, it sanitizes a branch name first: for example, github actions disallow the / symbol that appears in branch names — it gets replaced with -.

Update: after a discussion with @yiyi-wang-rv, we decided to instead push the images to our private github packages, so they don't appear in our Dockerhub.

@palinatolmach
Copy link
Collaborator Author

After a discussion with @yiyi-wang-rv, we decided to instead push the images to our private github packages, so they don't appear in our Dockerhub.

.github/workflows/docker-push.yml Outdated Show resolved Hide resolved
.github/workflows/docker-push.yml Outdated Show resolved Hide resolved
.github/workflows/docker-push.yml Outdated Show resolved Hide resolved
.github/workflows/docker-push.yml Outdated Show resolved Hide resolved
.github/workflows/docker-push.yml Outdated Show resolved Hide resolved
@automergerpr-permission-manager automergerpr-permission-manager bot merged commit 0ec85db into master Dec 13, 2024
12 checks passed
@automergerpr-permission-manager automergerpr-permission-manager bot deleted the publish-branch-to-docker branch December 13, 2024 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants