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

Use Docker files for Linux workflows #666

Merged
merged 2 commits into from
Feb 19, 2024
Merged

Conversation

zjeffer
Copy link
Collaborator

@zjeffer zjeffer commented Feb 18, 2024

This PR converts the rest of the linux workflows to use docker images, speeding up the CI/CD.

@nuttyartist
Copy link
Owner

Awesomee, thanks, @zjeffer!

Copy link
Collaborator

@guihkx guihkx left a comment

Choose a reason for hiding this comment

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

This is really nice work, thank you!

I have a few questions, that I should've asked in the previous PR, actually.

I'm not 100% sure how Docker works, but from I understand, ideally, these images have to be kept constantly up to date, right?

For example, in a scenario where a package in Ubuntu 20.04 (that we ship to users in the AppImage) has a serious CVE, do you have to rebuild and republish these images to the Docker website? Or, when we pull these images from our CI, the base image is automatically updated?

Based on a local test, I assume it's the first option: I pulled zjeffer/notes:ubuntu-22_04 (which was last updated two months ago), and there were 43 packages out of date.

If that's the case, can we somehow automate the updating and publishing of these images?

I understand GitHub itself also has it's own OCI Registry, correct? So, it'd be really awesome if we could integrate that here, instead of having to dump all this maintenance work on you.

.github/workflows/linux.yml Outdated Show resolved Hide resolved
@zjeffer
Copy link
Collaborator Author

zjeffer commented Feb 19, 2024

I'm not 100% sure how Docker works, but from I understand, ideally, these images have to be kept constantly up to date, right?

Correct.

If that's the case, can we somehow automate the updating and publishing of these images?

Yes, that should be possible. We should be able to automatically build these every week or so, for example.
Will look into this maybe next weekend when I have more time.

I understand GitHub itself also has it's own OCI Registry, correct? So, it'd be really awesome if we could integrate that here, instead of having to dump all this maintenance work on you.

In my experence, the GitHub container registry is extremely slow to upload/download from. That's why I always use the official Docker registry. It should still be possible to automate the builds with this, so there won't be any maintanance work from my side.

Copy link
Collaborator

@guihkx guihkx left a comment

Choose a reason for hiding this comment

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

Gotcha, thanks for clarifying!

@zjeffer zjeffer merged commit 419d232 into master Feb 19, 2024
17 checks passed
@zjeffer zjeffer deleted the feat/zjeffer/dockerfiles branch February 19, 2024 21:37
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.

3 participants