-
-
Notifications
You must be signed in to change notification settings - Fork 595
Upon release, publish Docker image to ghcr.io #3027
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
base: develop
Are you sure you want to change the base?
Upon release, publish Docker image to ghcr.io #3027
Conversation
Simplifying deployment/running locally. Using an existing Github Action that works and has tests. It pushes to the current repo's ghcr.io repository and adds a latest tag. I'm not sure how the project would want to handle other tags, so I've not dived into that (yet). Signed-off-by: Rob Grant <rob.grant@nanoporetech.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks... not sure why the DCO bot complains about your signoff.. I will override this as this looks fine otherwise.
@robertlagrant I have been pondering whether merging this or not silently for a long while! It would be great to automate this end-to-end though as a blueprint on how to use ScanCode instead. Do you feel like chipping in on this larger piece of work. (Also as an aside, I might prefer using a Debian base than a Ubuntu base as there could be additional license restrictions attached to Ubuntu base images) |
Hi @pombredanne - FYI this is just to use Ubuntu as the image to run the automation on. All it does is build the existing Dockerfile in the root of the repo, which I think is a Python build. As for the attribution - I know a good tool that can scan a repo for licences etc :D |
@robertlagrant Thanks.
We will run a built image through ScanCode.io for sure as it provide pretty decent support for container images ;) |
@pombredanne I am interested in helping to move this work forward. What do you see as the next step to unblock this PR? |
@pombredanne @robertlagrant Any thoughts on how this PR can be unblocked? I'm happy to help. For context on the need for this work, there are 28 images for scancode-toolkit in the Docker hub that individuals have created in an attempt to fill this need. Advantages to generating an image and storing in GitHub Packages:
The hold up seems to be described in the following statement...
@pombredanne Can you say more about what you think is the solution? I'm happy to meet to discuss this. |
@elrayle sorry for the late reply! I am sold on the benefits of publishing an image alright.
Few folks care to make this for published container images alright, but we should lead by example and there would be some irony to publish an image that's not been scanned given the domain and context of ScanCode ;)
This is not a huge amount of work, but this is real work. I really welcome the help there and can guide you as needed if you are willing to tackle some of it. The community hangs out on https://matrix.to/#/#aboutcode-org_discuss:gitter.im and we can talk or chat here or elsewhere anytime! |
@elrayle It is likely simpler and better to start a new PR BTW |
@pombredanne Thanks for the detailed list of expectations. They all make sense. I'm a big fan of leading by example, so agree with your views on making sure the image is scanned and has attributions. I'm planning to put this on my work list starting as early as next week or possibly in a few weeks depending on how well other work clears up. I'll definitely start a new PR. Is there a Slack channel or other interactive discussion platform for asking questions in real time if clarifications are needed? |
@elrayle We use matrix (with formerly gitter.im service acquired by element) at https://matrix.to/#/#aboutcode-org_discuss:gitter.im (and we are also on IRC libera.chat at #aboutcode but this is seldom used) |
You can use your favorite matrix client or the web one from Element at https://app.element.io/#/room/#aboutcode-org_discuss:gitter.im |
Fixes #3026
Tasks
Run tests locally to check for errors.
Thanks!