Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests: add nvme-symlink test to verify GCP udev rules #2425

Merged
merged 1 commit into from
Jun 1, 2023

Conversation

HuijingHei
Copy link
Member

@HuijingHei HuijingHei commented May 16, 2023

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

@HuijingHei HuijingHei force-pushed the gcp-confidential-test branch 3 times, most recently from cbb46d8 to e1a8558 Compare May 17, 2023 07:56
@HuijingHei HuijingHei marked this pull request as ready for review May 17, 2023 07:57
@HuijingHei
Copy link
Member Author

HuijingHei commented May 17, 2023

Hold on until #2412 is fixed and sync to openshift/os

tests/kola/files/initrd/expected-contents Outdated Show resolved Hide resolved
tests/kola/platforms/gcp/nvme-symlink Outdated Show resolved Hide resolved
tests/kola/platforms/gcp/nvme-symlink Outdated Show resolved Hide resolved
tests/kola/platforms/gcp/nvme-symlink Outdated Show resolved Hide resolved
tests/kola/platforms/gcp/nvme-symlink Outdated Show resolved Hide resolved
tests/kola/platforms/gcp/nvme-symlink Outdated Show resolved Hide resolved
tests/kola/platforms/gcp/nvme-symlink Outdated Show resolved Hide resolved
tests/kola/platforms/gcp/nvme-symlink Outdated Show resolved Hide resolved
tests/kola/platforms/gcp/nvme-symlink Outdated Show resolved Hide resolved
tests/kola/platforms/gcp/nvme-symlink Outdated Show resolved Hide resolved
@HuijingHei HuijingHei added the WIP PR still being worked on label May 24, 2023
@HuijingHei
Copy link
Member Author

Need to do some tweak when coreos/coreos-assembler#3484 merged, change additionalDisks: ["375G:interface=nvme"] to additionalDisks: ["375G:channel=nvme"]

Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good! A few remaining comments

tests/kola/platforms/gcp/nvme-symlink Show resolved Hide resolved
tests/kola/platforms/gcp/nvme-symlink Show resolved Hide resolved
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
Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@HuijingHei
Copy link
Member Author

Thanks @dustymabe for the review, hold on until coreos/coreos-assembler#3484 merged.

@HuijingHei
Copy link
Member Author

/retest

@dustymabe
Copy link
Member

CI failed because of "downgrade detected" but I think that's just because this PR needs a rebase. I don't think we really need to do that. I'll just merge it.

@dustymabe dustymabe merged commit 075a403 into coreos:testing-devel Jun 1, 2023
@HuijingHei HuijingHei deleted the gcp-confidential-test branch June 1, 2023 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a new GCP udev rules test
2 participants