Skip to content

Commit

Permalink
Cirrus: Update VM Images + env. var. name
Browse files Browse the repository at this point in the history
For consistency with CI in other repositories, update the variable name.
Also, remove some variable definitions which are not included in the
common automation library.  Finally, strip off the special Fedora
partition setup, as it's no-longer supported in the VM images.

Signed-off-by: Chris Evich <cevich@redhat.com>
  • Loading branch information
cevich committed Mar 29, 2022
1 parent 00d5f3b commit 2011975
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 76 deletions.
10 changes: 5 additions & 5 deletions .cirrus.yml
Expand Up @@ -24,10 +24,10 @@ env:
# GCE project where images live
IMAGE_PROJECT: "libpod-218412"
# VM Image built in containers/automation_images
_BUILT_IMAGE_SUFFIX: "c6431352024203264"
FEDORA_CACHE_IMAGE_NAME: "fedora-${_BUILT_IMAGE_SUFFIX}"
PRIOR_FEDORA_CACHE_IMAGE_NAME: "prior-fedora-${_BUILT_IMAGE_SUFFIX}"
UBUNTU_CACHE_IMAGE_NAME: "ubuntu-${_BUILT_IMAGE_SUFFIX}"
IMAGE_SUFFIX: "c4512539143831552"
FEDORA_CACHE_IMAGE_NAME: "fedora-${IMAGE_SUFFIX}"
PRIOR_FEDORA_CACHE_IMAGE_NAME: "prior-fedora-${IMAGE_SUFFIX}"
UBUNTU_CACHE_IMAGE_NAME: "ubuntu-${IMAGE_SUFFIX}"

####
#### Command variables to help avoid duplication
Expand Down Expand Up @@ -132,7 +132,7 @@ lint_task:
meta_task:

container:
image: "quay.io/libpod/imgts:${_BUILT_IMAGE_SUFFIX}"
image: "quay.io/libpod/imgts:${IMAGE_SUFFIX}"
cpu: 1
memory: 1

Expand Down
62 changes: 0 additions & 62 deletions contrib/cirrus/add_second_partition.sh

This file was deleted.

7 changes: 0 additions & 7 deletions contrib/cirrus/lib.sh
Expand Up @@ -67,13 +67,6 @@ EPOCH_TEST_COMMIT="$CIRRUS_BASE_SHA"
# Unsafe env. vars for display
SECRET_ENV_RE='(IRCID)|(ACCOUNT)|(^GC[EP]..+)|(SSH)'

# GCE image-name compatible string representation of distribution name
OS_RELEASE_ID="$(source /etc/os-release; echo $ID)"
# GCE image-name compatible string representation of distribution _major_ version
OS_RELEASE_VER="$(source /etc/os-release; echo $VERSION_ID | tr -d '.')"
# Combined to ease soe usage
OS_REL_VER="${OS_RELEASE_ID}-${OS_RELEASE_VER}"

# Working with dnf + timeout/retry
SHORT_DNFY='lilto dnf -y'
LONG_DNFY='bigto dnf -y'
Expand Down
2 changes: 0 additions & 2 deletions contrib/cirrus/setup.sh
Expand Up @@ -12,8 +12,6 @@ cd $GOSRC
msg "Setting up $OS_RELEASE_ID $OS_RELEASE_VER"
case "$OS_RELEASE_ID" in
fedora)
# Required on Fedora VM images
bash "$SCRIPT_BASE/add_second_partition.sh"
[[ -z "$RPMS_CONFLICTING" ]] || \
$SHORT_DNFY erase $RPMS_CONFLICTING
$SHORT_DNFY install zstd
Expand Down

0 comments on commit 2011975

Please sign in to comment.