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

Build strategy based on containers with cache #214

Merged
merged 2 commits into from
Jun 5, 2024

Conversation

adrianriobo
Copy link
Contributor

Fix #213

Makefile Outdated
# Build the container cache image
.PHONY: build-cache
build-cache:
${CONTAINER_MANAGER} build -t ${IMG_CACHE} -f oci/Containerfile.cache .
Copy link
Contributor

Choose a reason for hiding this comment

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

Who maintains the lifecycle of this cache?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the idea would be self managed on every merge to main a gh action will build the image and push it

@adrianriobo adrianriobo force-pushed the fix-213 branch 3 times, most recently from b621a39 to db71ade Compare May 15, 2024 13:51
@adrianriobo adrianriobo changed the title WIP: Build strategy based on containers with cached layers WIP: Build strategy based on containers with cache May 15, 2024
@adrianriobo adrianriobo changed the title WIP: Build strategy based on containers with cache Build strategy based on containers with cache May 15, 2024
@adrianriobo adrianriobo requested a review from benoitf May 15, 2024 13:53
@adrianriobo
Copy link
Contributor Author

@benoitf added you here as per our discussions about a way to move the cache into a building strategy based on Containerfiles

@adrianriobo
Copy link
Contributor Author

This depends on #222

@@ -24,7 +24,7 @@ on:

jobs:
release:
runs-on: ubuntu-latest
runs-on: ubuntu-22.04
Copy link
Contributor

Choose a reason for hiding this comment

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

24.04 ?

@adrianriobo adrianriobo merged commit 4b322d6 into crc-org:main Jun 5, 2024
3 checks passed
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.

Optimize Building Strategy Based on Containers
3 participants