diff options
| author | Laurent Bonnans <laurent.bonnans@here.com> | 2019-07-04 15:35:44 +0200 |
|---|---|---|
| committer | Mykhaylo Sul <ext-mykhaylo.sul@here.com> | 2019-07-17 16:06:00 +0300 |
| commit | 14bedef5860abc352d7cfe957f2c3781cf29d52c (patch) | |
| tree | 3da162e9622eaff609aa8fb2d13624fbd26eb3ee | |
| parent | 9da34c89fcb8dc01137c568d2823aedd56a349e1 (diff) | |
| download | meta-updater-14bedef5860abc352d7cfe957f2c3781cf29d52c.tar.gz | |
Gitlab pipeline for building core-image-minimal
Including these points (initially as separate commits):
* Try caching updater-repo in gitlab
* Only patch aktualizr recipe in CI if needed
* Run under user 4321 in bitbake docker image
* Add ability to supply METADIR to envsetup.sh
So that we can bypass the "relative path from the script" trick that
doesn't work with symlinks.
* Checkout the PR base branch
* do not run on pushes
* run on runners with "gitlab" tag
* Print all revisions after repo checkout on CI
* Pull CI image from the base branch instead of master
* More robust checkout in CI
* Only store updater-repo artifacts for one day
* Use "stretch" instead of "stable" for Debian release in Dockerfile
(in fact, stable just changed to buster)
* Move UPDATER_REPO variable out of .gitlab-ci.yml
To be defined in the project's settings
* Make docker CI stage buildable on shared runners
Signed-off-by: Laurent Bonnans <laurent.bonnans@here.com>
| -rw-r--r-- | .gitlab-ci.yml | 63 | ||||
| -rw-r--r-- | scripts/ci/Dockerfile.checkout | 9 | ||||
| -rwxr-xr-x | scripts/ci/checkout-oe.sh | 47 | ||||
| -rwxr-xr-x | scripts/envsetup.sh | 2 |
4 files changed, 120 insertions, 1 deletions
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml new file mode 100644 index 0000000..94a614c --- /dev/null +++ b/.gitlab-ci.yml | |||
| @@ -0,0 +1,63 @@ | |||
| 1 | stages: | ||
| 2 | - docker | ||
| 3 | - checkout | ||
| 4 | - test | ||
| 5 | |||
| 6 | # notes: | ||
| 7 | # - could be useful https://docs.gitlab.com/ee/ci/yaml/#include | ||
| 8 | |||
| 9 | variables: | ||
| 10 | # Needs $PR_BASE_BRANCH to be defined as a pipeline variable to work correctly | ||
| 11 | |||
| 12 | CHECKOUT_MASTER_IMAGE: ${CI_REGISTRY_IMAGE}:ci-${PR_BASE_BRANCH}-checkout | ||
| 13 | CHECKOUT_PR_IMAGE: ${CI_REGISTRY_IMAGE}:ci-${CI_COMMIT_REF_SLUG}-checkout | ||
| 14 | |||
| 15 | BITBAKE_MASTER_IMAGE: ${CI_REGISTRY_IMAGE}:ci-${PR_BASE_BRANCH}-bitbake | ||
| 16 | BITBAKE_PR_IMAGE: ${CI_REGISTRY_IMAGE}:ci-${CI_COMMIT_REF_SLUG}-bitbake | ||
| 17 | |||
| 18 | |||
| 19 | Docker Setup: | ||
| 20 | image: docker:stable | ||
| 21 | stage: docker | ||
| 22 | services: | ||
| 23 | - docker:dind | ||
| 24 | except: | ||
| 25 | - pushes | ||
| 26 | before_script: | ||
| 27 | - docker login -u gitlab-ci-token -p "$CI_JOB_TOKEN" "$CI_REGISTRY" | ||
| 28 | script: | ||
| 29 | - docker pull "$BITBAKE_PR_IMAGE" || docker pull "$BITBAKE_MASTER_IMAGE" || true | ||
| 30 | - docker build --pull --cache-from "$BITBAKE_MASTER_IMAGE" --cache-from "$BITBAKE_PR_IMAGE" -f ./scripts/ci/Dockerfile.bitbake -t "$BITBAKE_PR_IMAGE" ./scripts/ci | ||
| 31 | - docker push "$BITBAKE_PR_IMAGE" | ||
| 32 | |||
| 33 | - docker pull "$CHECKOUT_PR_IMAGE" || docker pull "$CHECKOUT_MASTER_IMAGE" || true | ||
| 34 | - docker build --pull --cache-from "$CHECKOUT_MASTER_IMAGE" --cache-from "$CHECKOUT_PR_IMAGE" -f ./scripts/ci/Dockerfile.checkout -t "$CHECKOUT_PR_IMAGE" ./scripts/ci | ||
| 35 | - docker push "$CHECKOUT_PR_IMAGE" | ||
| 36 | |||
| 37 | Checkout: | ||
| 38 | image: "$CHECKOUT_PR_IMAGE" | ||
| 39 | stage: checkout | ||
| 40 | except: | ||
| 41 | - pushes | ||
| 42 | cache: | ||
| 43 | paths: | ||
| 44 | - updater-repo | ||
| 45 | artifacts: | ||
| 46 | expire_in: "1 day" | ||
| 47 | paths: | ||
| 48 | - updater-repo | ||
| 49 | script: | ||
| 50 | - MANIFEST=$PR_BASE_BRANCH ./scripts/ci/checkout-oe.sh | ||
| 51 | |||
| 52 | Build core-image-minimal: | ||
| 53 | image: "$BITBAKE_PR_IMAGE" | ||
| 54 | stage: test | ||
| 55 | except: | ||
| 56 | - pushes | ||
| 57 | dependencies: | ||
| 58 | - Checkout | ||
| 59 | tags: | ||
| 60 | - bitbake | ||
| 61 | script: | ||
| 62 | - ./scripts/ci/configure.sh | ||
| 63 | - ./scripts/ci/build.sh core-image-minimal | ||
diff --git a/scripts/ci/Dockerfile.checkout b/scripts/ci/Dockerfile.checkout new file mode 100644 index 0000000..efec234 --- /dev/null +++ b/scripts/ci/Dockerfile.checkout | |||
| @@ -0,0 +1,9 @@ | |||
| 1 | FROM debian:stretch-slim | ||
| 2 | LABEL Description="Image for checking out updater-repo" | ||
| 3 | |||
| 4 | RUN sed -i 's#deb http://deb.debian.org/debian stretch main#deb http://deb.debian.org/debian stretch main contrib#g' /etc/apt/sources.list | ||
| 5 | RUN sed -i 's#deb http://deb.debian.org/debian stretch-updates main#deb http://deb.debian.org/debian stretch-updates main contrib#g' /etc/apt/sources.list | ||
| 6 | RUN apt-get update -q && apt-get install -qy \ | ||
| 7 | git \ | ||
| 8 | repo \ | ||
| 9 | xmlstarlet | ||
diff --git a/scripts/ci/checkout-oe.sh b/scripts/ci/checkout-oe.sh new file mode 100755 index 0000000..8744b2f --- /dev/null +++ b/scripts/ci/checkout-oe.sh | |||
| @@ -0,0 +1,47 @@ | |||
| 1 | #!/usr/bin/env bash | ||
| 2 | |||
| 3 | set -euo pipefail | ||
| 4 | |||
| 5 | set -x | ||
| 6 | |||
| 7 | REMOTE_SOURCE=${REMOTE_SOURCE:-https://github.com/advancedtelematic} | ||
| 8 | MANIFEST=${MANIFEST:-master} | ||
| 9 | CURRENT_PROJECT=${CURRENT_PROJECT:-meta-updater} | ||
| 10 | |||
| 11 | #CURRENT_REV=$(git rev-parse HEAD) | ||
| 12 | LOCAL_REPO=$PWD | ||
| 13 | |||
| 14 | mkdir -p updater-repo | ||
| 15 | |||
| 16 | cd updater-repo | ||
| 17 | |||
| 18 | repo init -m "${MANIFEST}.xml" -u "$REMOTE_SOURCE/updater-repo" | ||
| 19 | |||
| 20 | git -C .repo/manifests reset --hard | ||
| 21 | |||
| 22 | # patch manifest: | ||
| 23 | # - add a new "ats" remote that points to "$REMOTE_SOURCE" | ||
| 24 | # - change projects that contain "advancedtelematic" to use the ats remote | ||
| 25 | # - remove the current project from the manifest | ||
| 26 | MANIFEST_FILE=".repo/manifests/${MANIFEST}.xml" | ||
| 27 | xmlstarlet ed --omit-decl -L \ | ||
| 28 | -s "/manifest" -t elem -n "remote" -v "" \ | ||
| 29 | -i "/manifest/remote[last()]" -t attr -n "name" -v "ats" \ | ||
| 30 | -i "/manifest/remote[last()]" -t attr -n "fetch" -v "$REMOTE_SOURCE" \ | ||
| 31 | -i "/manifest/project[contains(@name, 'advancedtelematic')]" -t attr -n "remote" -v "ats" \ | ||
| 32 | -d "/manifest/project[@path=\"$CURRENT_PROJECT\"]" \ | ||
| 33 | "$MANIFEST_FILE" | ||
| 34 | |||
| 35 | # hack: sed on `advancedtelematic/` names, to remove this unwanted prefix | ||
| 36 | sed -i 's#name="advancedtelematic/#name="#g' "$MANIFEST_FILE" | ||
| 37 | |||
| 38 | repo manifest | ||
| 39 | |||
| 40 | repo forall -c 'git reset --hard ; git clean -fdx' | ||
| 41 | |||
| 42 | repo sync -d --force-sync | ||
| 43 | |||
| 44 | rm -f "$CURRENT_PROJECT" | ||
| 45 | ln -s "$LOCAL_REPO" "$CURRENT_PROJECT" | ||
| 46 | |||
| 47 | repo manifest -r | ||
diff --git a/scripts/envsetup.sh b/scripts/envsetup.sh index 6103cf6..5827bc2 100755 --- a/scripts/envsetup.sh +++ b/scripts/envsetup.sh | |||
| @@ -21,7 +21,7 @@ if [[ $SOURCED -ne 1 ]]; then | |||
| 21 | exit 1 | 21 | exit 1 |
| 22 | fi | 22 | fi |
| 23 | 23 | ||
| 24 | METADIR="${SOURCEDIR}/../.." | 24 | METADIR=${METADIR:-${SOURCEDIR}/../..} |
| 25 | 25 | ||
| 26 | if [[ ! -f "${BUILDDIR}/conf/local.conf" ]]; then | 26 | if [[ ! -f "${BUILDDIR}/conf/local.conf" ]]; then |
| 27 | source "$METADIR/poky/oe-init-build-env" "$BUILDDIR" | 27 | source "$METADIR/poky/oe-init-build-env" "$BUILDDIR" |
