Skip to content

Commit 8c07f2c

Browse files
committed
tests/uefi_sb: Adapt tests for varstored changes
We're now shipping more Secure Boot variables with varstored. As such, the following tests have changed: - test_start_vm_without_uefi_vars_on_pool_with_only_pk: No longer applicable. - test_clear_custom_pool_certificates: Only check for symlink when clearing certs, as the condition that hosts only have PK after `secureboot-certs clear` is no longer true. Signed-off-by: Tu Dinh <[email protected]>
1 parent 495a6c3 commit 8c07f2c

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

tests/uefi_sb/test_varstored_cert_flow.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,8 +53,6 @@ def test_clear_custom_pool_certificates(self, host):
5353
for h in host.pool.hosts:
5454
logging.info(f"Check host {h} has no custom certificates on disk.")
5555
assert h.is_symlink(host.varstore_dir())
56-
logging.info(f"Check host {h} only has PK, and no other certs.")
57-
assert h.ssh(['ls', '/var/lib/varstored/']) == 'PK.auth'
5856

5957
@pytest.mark.small_vm
6058
@pytest.mark.usefixtures("host_at_least_8_3")
@@ -124,6 +122,7 @@ def test_start_vm_without_uefi_vars(self, uefi_vm):
124122
for key in ['PK', 'KEK', 'db', 'dbx']:
125123
check_vm_cert_md5sum(vm, key, pool_auths[key].auth())
126124

125+
@pytest.mark.skip(reason="No longer applicable since varstored now ships most SB keys")
127126
def test_start_vm_without_uefi_vars_on_pool_with_only_pk(self, uefi_vm):
128127
# When a VM first starts but the pool doesn't have certs configured,
129128
# this used, until late in 8.3 development, to *not* propagate the certs to the VM

0 commit comments

Comments
 (0)