Skip to content

Commit

Permalink
tests: add test nvme-symlink to verify GCP udev rules
Browse files Browse the repository at this point in the history
Force this test to not run by default unless named specifically
or `--tag confidential` is passed to `kola run`, also requires
`--gcp-machinetype n2d-standard-2 --gcp-confidential-vm`

It will create confidential instance on GCP with 1 nvme persistent disk
and 1 local ssd disk, then check the new udev rules make effect.

Based on coreos/coreos-assembler#3474
and coreos/coreos-assembler#3477.

Fix coreos/fedora-coreos-tracker#1457
  • Loading branch information
HuijingHei committed May 19, 2023
1 parent 0e3d1db commit 6f9e69b
Show file tree
Hide file tree
Showing 2 changed files with 61 additions and 0 deletions.
1 change: 1 addition & 0 deletions tests/kola/platforms/gcp/data/commonlib.sh
60 changes: 60 additions & 0 deletions tests/kola/platforms/gcp/nvme-symlink
Original file line number Diff line number Diff line change
@@ -0,0 +1,60 @@
#!/bin/bash
## kola:
## description: Verify new GCP udev rules work well on confidential instance.
## additionalDisks: ["375G:interface=NVME"]
## platforms: gcp
## requiredTag: confidential

# See https://issues.redhat.com/browse/OCPBUGS-7582
# https://github.com/coreos/fedora-coreos-tracker/issues/1457
#
# Force this test to not run by default unless named specifically
# or `--tag confidential` is passed to `kola run`, also requires
# `--gcp-machinetype n2d-standard-2 --gcp-confidential-vm`
#
# It will create confidential instance on GCP with 1 nvme persistent disk
# and 1 local ssd disk, then check the new udev rules make effect.
# Note: each local SSD is 375 GB in size, refer to https://cloud.google.com/compute/docs/disks/local-ssd

set -xeuo pipefail

. $KOLA_EXT_DATA/commonlib.sh

# check instance has 2 disks
nvme_info=$(nvme list-subsys -o json)
nvme_count=$(jq -r ".[].Subsystems | length" <<< "$nvme_info")
if [ $nvme_count -ne 2 ]; then
fatal "instance does not have 2 disks"
fi

# check nvme device
has_nvme() {
local nvme_disk=$(jq -r ".[].Subsystems[].Paths[] | select(.Name == \"${disk}\").Name" <<< "$nvme_info")
if [ -n "${nvme_disk}" ]; then
if [ ! -e "${device}" ]; then
fatal "instance has nvme device but no ${disk} accessible"
fi
fi
}

# check symlink
check_symlink() {
local tmpfile=$(mktemp)
/usr/lib/udev/google_nvme_id -d "${device}" > ${tmpfile}
source ${tmpfile}
if [ ! -n "${ID_SERIAL_SHORT:-}" ]; then
fatal "can not get nvme ${device} ID_SERIAL_SHORT"
fi

local link="/dev/disk/by-id/google-${ID_SERIAL_SHORT}"
if ! ls -l "${link}"; then
fatal "can not find ${device} symlink ${link}"
fi
}

for disk in nvme0 nvme1; do
device="/dev/${disk}n1"
has_nvme
check_symlink
ok "Find ${device} symlink"
done

0 comments on commit 6f9e69b

Please sign in to comment.