Skip to content

Commit

Permalink
Cirrus: Migrate Ubuntu to Debian CI VM Images
Browse files Browse the repository at this point in the history
Signed-off-by: Chris Evich <[email protected]>
  • Loading branch information
cevich committed Apr 4, 2023
1 parent b04c26b commit 1ad45ee
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 18 deletions.
22 changes: 11 additions & 11 deletions .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,15 +17,15 @@ env:
####
#### Cache-image names to test with (double-quotes around names are critical)
###
FEDORA_NAME: "fedora-37" ### 20230120t152650z-f37f36u2204
UBUNTU_NAME: "ubuntu-2204" ### 20230120t152650z-f37f36u2204
FEDORA_NAME: "fedora-37"
DEBIAN_NAME: "debian-12"

# GCE project where images live
IMAGE_PROJECT: "libpod-218412"
# VM Image built in containers/automation_images
IMAGE_SUFFIX: "c20230120t152650z-f37f36u2204"
IMAGE_SUFFIX: "c20230330t153101z-f37f36d12"
FEDORA_CACHE_IMAGE_NAME: "fedora-${IMAGE_SUFFIX}"
UBUNTU_CACHE_IMAGE_NAME: "ubuntu-${IMAGE_SUFFIX}"
DEBIAN_CACHE_IMAGE_NAME: "debian-${IMAGE_SUFFIX}"

####
#### Command variables to help avoid duplication
Expand Down Expand Up @@ -67,7 +67,7 @@ linux_testing: &linux_testing
always:
df_script: '${_DFCMD} || true'
rh_audit_log_script: '${_RAUDITCMD} || true'
ubuntu_audit_log_script: '${_UAUDITCMD} || true'
debian_audit_log_script: '${_UAUDITCMD} || true'
journal_log_script: '${_JOURNALCMD} || true'


Expand Down Expand Up @@ -95,13 +95,13 @@ fedora_testing_task: &fedora_testing


# aufs was dropped between 20.04 and 22.04, can't test it
ubuntu_testing_task: &ubuntu_testing
debian_testing_task: &debian_testing
<<: *linux_testing
alias: ubuntu_testing
alias: debian_testing
name: *std_test_name
env:
OS_NAME: "${UBUNTU_NAME}"
VM_IMAGE: "${UBUNTU_CACHE_IMAGE_NAME}"
OS_NAME: "${DEBIAN_NAME}"
VM_IMAGE: "${DEBIAN_CACHE_IMAGE_NAME}"
# Not all $TEST_DRIVER combinations valid for all $VM_IMAGE types.
matrix:
- env:
Expand Down Expand Up @@ -145,7 +145,7 @@ meta_task:
# Space-separated list of images used by this repository state
IMGNAMES: |-
${FEDORA_CACHE_IMAGE_NAME}
${UBUNTU_CACHE_IMAGE_NAME}
${DEBIAN_CACHE_IMAGE_NAME}
BUILDID: "${CIRRUS_BUILD_ID}"
REPOREF: "${CIRRUS_CHANGE_IN_REPO}"
GCPJSON: ENCRYPTED[244a93fe8b386b48b96f748342bf741350e43805eee81dd04b45093bdf737e540b993fc735df41f131835fa0f9b65826]
Expand Down Expand Up @@ -177,7 +177,7 @@ success_task:
depends_on:
- lint
- fedora_testing
- ubuntu_testing
- debian_testing
- meta
- vendor
- cross
Expand Down
6 changes: 3 additions & 3 deletions contrib/cirrus/build_and_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,8 @@ case $TEST_DRIVER in
showrun make STORAGE_DRIVER=aufs local-test-integration local-test-unit
;;
btrfs)
# Fedora: install btrfs-progs, btrfs-progs-devel
# Ubuntu: install btrfs-progs, libbtrfs-dev
# Fedora: Needs btrfs-progs, btrfs-progs-devel
# Debian: Needs btrfs-progs, libbtrfs-dev
if [[ "$(./hack/btrfs_tag.sh)" =~ exclude_graphdriver_btrfs ]]; then
die "Built without btrfs, so we can't test it"
fi
Expand All @@ -52,7 +52,7 @@ case $TEST_DRIVER in
TMPDIR="$tmpdir" showrun make STORAGE_DRIVER=$TEST_DRIVER local-test-integration local-test-unit
;;
zfs)
# Ubuntu: install zfsutils-linux
# Debian: Needs zfsutils
if ! check_filesystem_supported $TEST_DRIVER ; then
die "This CI VM does not support $TEST_DRIVER in its kernel"
fi
Expand Down
5 changes: 1 addition & 4 deletions contrib/cirrus/setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -14,13 +14,10 @@ case "$OS_RELEASE_ID" in
fedora)
[[ -z "$RPMS_CONFLICTING" ]] || \
$SHORT_DNFY erase $RPMS_CONFLICTING
$SHORT_DNFY install zstd
;;
ubuntu)
debian)
[[ -z "$DEBS_CONFLICTING" ]] || \
$SHORT_APTGET -q remove $DEBS_CONFLICTING
$SHORT_APTGET -q update
$SHORT_APTGET -q install zstd zfsutils-linux
;;
*)
bad_os_id_ver
Expand Down

0 comments on commit 1ad45ee

Please sign in to comment.