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

Dockefiles should separate the base image from the "build strategy" (aka flavor) #1898

Closed
Tracked by #1914 ...
jimmykarily opened this issue Oct 9, 2023 · 0 comments · Fixed by #1912
Closed
Tracked by #1914 ...
Assignees

Comments

@jimmykarily
Copy link
Contributor

jimmykarily commented Oct 9, 2023

Currently setting the flavor when building a kairos dockerfile, sets both the base image and the building steps (the "strategy"). In other words, we decide what packages to install and what modifications to make based on the flavor.

We should allow the users to specify a base image and a strategy. For example:

"Build on top of ubuntu_my_custom_build as if you were building ubuntu 20.04. In this example, "ubuntu 20.04" becomes the build strategy, and ubuntu_my_custom_build is the base image.

Part of: #116

@jimmykarily jimmykarily converted this from a draft issue Oct 9, 2023
@jimmykarily jimmykarily moved this from Todo 🖊 to In Progress 🏃 in 🧙Issue tracking board Oct 9, 2023
@jimmykarily jimmykarily moved this from In Progress 🏃 to Todo 🖊 in 🧙Issue tracking board Oct 9, 2023
@mauromorales mauromorales moved this from Todo 🖊 to In Progress 🏃 in 🧙Issue tracking board Oct 9, 2023
@mauromorales mauromorales mentioned this issue Oct 9, 2023
@jimmykarily jimmykarily mentioned this issue Oct 11, 2023
jimmykarily pushed a commit that referenced this issue Oct 19, 2023
Fixes #1898

Signed-off-by: Dimitris Karakasilis <[email protected]>
jimmykarily pushed a commit that referenced this issue Oct 19, 2023
Fixes #1898

Signed-off-by: Dimitris Karakasilis <[email protected]>
jimmykarily pushed a commit that referenced this issue Oct 20, 2023
Fixes #1898

Signed-off-by: Dimitris Karakasilis <[email protected]>
jimmykarily pushed a commit that referenced this issue Oct 23, 2023
Fixes #1898

Signed-off-by: Dimitris Karakasilis <[email protected]>
jimmykarily pushed a commit that referenced this issue Oct 23, 2023
Fixes #1898

Signed-off-by: Dimitris Karakasilis <[email protected]>
jimmykarily pushed a commit that referenced this issue Oct 24, 2023
Fixes #1898

Signed-off-by: Dimitris Karakasilis <[email protected]>
@jimmykarily jimmykarily moved this from In Progress 🏃 to Under review 🔍 in 🧙Issue tracking board Oct 24, 2023
jimmykarily added a commit that referenced this issue Oct 24, 2023
* Refactor flavors and dockerfiles

Fixes #1898

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Use FRAMEWORK_FLAVOR variable to avoid confusion with FLAVOR

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Fix pipeline by adding mandatory value

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Fix ubuntu Dockerfile

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Apply suggestions from code review

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Remove TODO (already implemented) and non-used ARG "IMAGE"

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Remove TODOs

moved here: #1897 (comment)

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Remove unecessary env variables and fix VARIANT_RELEASE missing

Signed-off-by: Dimitris Karakasilis <[email protected]>

* PR comments

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Remove naming.sh script from base image

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Make sure we use the "fixed" kairos version with the remove "+" symbols

when creating a tag for container images.

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Setup qemu and docker buildx in reusable-build-flavor workflow

because otherwise it fails to build rpi images

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Remove not needed env variables

Signed-off-by: Dimitris Karakasilis <[email protected]>

* Remove temporary change

Signed-off-by: Dimitris Karakasilis <[email protected]>

---------

Signed-off-by: Dimitris Karakasilis <[email protected]>
Signed-off-by: Dimitris Karakasilis <[email protected]>
Co-authored-by: Mauro Morales <[email protected]>
@github-project-automation github-project-automation bot moved this from Under review 🔍 to Done ✅ in 🧙Issue tracking board Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants