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

fix: use rabe base image and wheel multistage build #118

Merged
merged 12 commits into from
Nov 29, 2022

Conversation

hairmare
Copy link
Member

@hairmare hairmare commented Jun 25, 2022

Uses radiorabe/python-minimal as the runtime image and radiorabe/s2i-python for building the app.

The new build process first builds a wheel in a first build stage and then installs this wheel into the final app container.

Switching to a RaBe base image helps us with lifecycle management since there is now one entrypoint of new base image versions for all of our python properties. Additionally the shared base image makes pulling this faster run on a shared container host since the base layers will most likely already be there.

@hairmare hairmare requested a review from spameier June 25, 2022 12:47
Dockerfile Outdated Show resolved Hide resolved
@spameier
Copy link
Member

Sorry for the late reply, I like the idea. Will dependabot update to newer tags when new versions of s2i-python are released?

@hairmare
Copy link
Member Author

i updated this to also cover basic, fails to build type errors in ci

let's get it merged this month :)

@hairmare hairmare enabled auto-merge (squash) November 29, 2022 21:34
@hairmare hairmare merged commit d5c1902 into radiorabe:main Nov 29, 2022
@hairmare hairmare deleted the chore/use-rabe-base-images branch November 29, 2022 21:34
@hairmare
Copy link
Member Author

Dec go brrrrrrt

imma be available should this fail

@hairmare hairmare mentioned this pull request Dec 7, 2022
11 tasks
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