Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Build kernel from a specific git tag #725

Merged
merged 2 commits into from
Nov 9, 2020

Conversation

darkowlzz
Copy link
Contributor

@darkowlzz darkowlzz commented Nov 8, 2020

This patch removes fetching of upstream kernel repo updates and gets the target kernel version code, by doing a shallow clone, to reduce the kernel build time. It also removes the dependency and maintenance of luxas/kernel-builder with a local ignite-kernel-builder image that can be updated from within the ignite repo.

  • Adds a local ignite-kernel-builder container image based on luxas/kernel-builder, with all the build dependencies and without any kernel source code in it.
  • For kernel build or config upgrade, use the local ignite-kernel-builder as base and shallow clone the target kernel tag. ignite-kernel-builder is built once and reused.
  • Cleanup the kernel config upgrade container once it exits by adding --rm to the docker run command.

This patch tries to reduce the kernel build process time by shallow
cloning only the target version of the kernel, instead of fetching all
the updates from upstream.
It also creates a local ignite-kernel-builder container image which is
shared between building kernel and kernel config update operations.
This removes the dependency and maintenance of luxas/kernel-builder
image.
Update the upgrade-config.sh script to use the ignite-kernel-builder
image and clone only the required target kernel version.
Also, adds --rm to the docker run command to remove the container when
it exits.
@darkowlzz darkowlzz added the kind/enhancement Categorizes issue or PR as related to improving an existing feature. label Nov 8, 2020
@stealthybox stealthybox merged commit 8463489 into weaveworks:master Nov 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/enhancement Categorizes issue or PR as related to improving an existing feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants