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

Deploy to ghcr.io #89

Merged
merged 1 commit into from
Jan 16, 2023
Merged

Deploy to ghcr.io #89

merged 1 commit into from
Jan 16, 2023

Conversation

pbrkr
Copy link
Member

@pbrkr pbrkr commented Jan 16, 2023

This deployment process has been tested on my GitHub account using an extra commit to change the destination repo and not push to Docker Hub: https://github.com/pbrkr/yocto-dockerfiles/commit/74021a4badb7d9b447d4a577901da7bc75c54b98

The action results are all good there: https://github.com/pbrkr/yocto-dockerfiles/actions/runs/3928894387

And the containers are visible within my user account: https://github.com/pbrkr/yocto-dockerfiles/pkgs/container/yocto/versions

This should work when merged to the main yocto-dockerfiles repo but deployments are one of those things that's not possible to perfectly test before merging. Please ping me on IRC before merging this so we can check the results and I can jump in and fix things if there is an issue.

Once this deployment process is confirmed working for yocto-dockerfiles I'll move on to looking at the other repositories.

Signed-off-by: Paul Barker <paul.barker@sancloud.com>
Copy link
Member

@moto-timo moto-timo left a comment

Choose a reason for hiding this comment

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

We'll probably want to document that the containers can then be pulled from ghcr.io/crops/yocto:ubuntu-20.04-base for example, but that is a separate issue.

Logical variable names and following the existing paradigm, so LGTM.

@moto-timo moto-timo merged commit c849382 into crops:master Jan 16, 2023
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