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 default alpine and fallback debian docker image builds (GSI-876) #216

Merged
merged 5 commits into from
Aug 6, 2024

Conversation

lkuchenb
Copy link
Member

@lkuchenb lkuchenb commented Aug 5, 2024

This patch changes the template such that it contains two Dockerfiles:

  1. Dockerfile which builds a runner container based on python-alpine
  2. Dockerfile.debian which builds a runner container based on python-debian

Both images are pushed to Dockerhub, once with the tag suffix -debian once without (this makes use of the recently added support for "flavors" in PR ghga-de/gh-action-ci#4).

The alpine Dockerfile has been tested on all currently deployed services to the extend that they start and fail due to missing configuration.

A separate PR will be opened at https://github.com/ghga-de/file-services-backend.

Copy link
Member

@TheByronHimes TheByronHimes left a comment

Choose a reason for hiding this comment

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

Looks alright as far as I can tell (it ran and only mentioned the things below). I'll let Christoph take the lead on approval because I'm not knowledgeable enough at the moment to be able to point out any potential issues if they exist.

Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
lkuchenb and others added 2 commits August 5, 2024 22:44
Co-authored-by: Byron Himes <[email protected]>
Co-authored-by: Byron Himes <[email protected]>
@coveralls
Copy link

coveralls commented Aug 5, 2024

Pull Request Test Coverage Report for Build 10263934747

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 41.27%

Totals Coverage Status
Change from base Build 9645904659: 0.0%
Covered Lines: 26
Relevant Lines: 63

💛 - Coveralls

Cito
Cito previously approved these changes Aug 5, 2024
Copy link
Member

@Cito Cito left a comment

Choose a reason for hiding this comment

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

Looks good to me, just small suggestions.

Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
Dockerfile Show resolved Hide resolved
lkuchenb and others added 2 commits August 6, 2024 11:27
Co-authored-by: Christoph Zwerschke <[email protected]>
Co-authored-by: Christoph Zwerschke <[email protected]>
@lkuchenb lkuchenb requested review from TheByronHimes and Cito and removed request for TheByronHimes August 6, 2024 12:01
@lkuchenb lkuchenb merged commit 0f121c1 into main Aug 6, 2024
9 checks passed
@lkuchenb lkuchenb deleted the feature/multi-flavor-docker-builds branch August 6, 2024 15:07
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.

4 participants