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 24, 2023
1 parent 1553518 commit 4664350
Show file tree
Hide file tree
Showing 2 changed files with 77 additions and 0 deletions.
1 change: 1 addition & 0 deletions tests/kola/platforms/gcp/data/commonlib.sh
76 changes: 76 additions & 0 deletions tests/kola/platforms/gcp/nvme-symlink
Original file line number Diff line number Diff line change
@@ -0,0 +1,76 @@
#!/bin/bash
## kola:
## description: Verify new GCP udev rules work well on confidential instance.
## # Note: each local SSD is 375 GB in size, refer to https://cloud.google.com/compute/docs/disks/local-ssd
## additionalDisks: ["375G:channel=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.

set -xeuo pipefail

. $KOLA_EXT_DATA/commonlib.sh

# Set global variable with NVME json info
NVME_INFO=$(nvme list-subsys -o json)

# verify the instance is Confidential VM
assert_confidential_vm() {
local sevlog=$(dmesg | grep SEV | head)
if [ -n "${sevlog}" ] && echo "${sevlog}" | grep "Memory Encryption Features active: AMD SEV"; then
ok "instance is Confidential VM"
else
fatal "instance should be Confidential VM"
fi
}

# check instance has 2 disks
assert_two_nvme_disks() {
local 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
assert_nvme_disk_accessible() {
local disk=$1
local nvme_disk=$(jq -r ".[].Subsystems[].Paths[] | select(.Name == \"${disk}\").Name" <<< "${NVME_INFO}")
if [ -n "${nvme_disk}" ]; then
if [ ! -e "/dev/${disk}n1" ]; then
fatal "instance has nvme device but no ${disk} accessible"
fi
fi
}

# check symlink
assert_expected_symlink_exists() {
local device=$1
eval $(/usr/lib/udev/google_nvme_id -d "${device}")
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
}

assert_confidential_vm
assert_two_nvme_disks

for disk in nvme0 nvme1; do
assert_nvme_disk_accessible $disk
assert_expected_symlink_exists "/dev/${disk}n1"
ok "Found /dev/${disk}n1 symlink"
done

0 comments on commit 4664350

Please sign in to comment.