Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

CCv0: SEV/SNP tests should print debug messages #5708

Open
wainersm opened this issue Jul 12, 2023 · 0 comments
Open

CCv0: SEV/SNP tests should print debug messages #5708

wainersm opened this issue Jul 12, 2023 · 0 comments
Labels
enhancement Improvement to an existing feature needs-review Needs to be assessed by the team.

Comments

@wainersm
Copy link
Contributor

Which feature do you think can be improved?

The SEV and SNP integration tests.

How can it be improved?

Currently when a test fail we are left clueless because there aren't debug messages printed. For example:

05:55:34 [init.sh:34] INFO: Taint 'NoSchedule' is found. Untaint the node so pods can be scheduled.
05:55:34 node/amd-milan-coco-ci-ubuntu2004-001 untainted
05:55:34 INFO: Run tests
05:55:34 1..1
05:56:12 not ok 1 [cc][kubernetes][containerd][snp] Test SNP unencrypted container launch success
05:56:12 # (from function `kubernetes_wait_for_pod_ready_state' in file lib.sh, line 42,
05:56:12 #  in test file confidential/snp.bats, line 77)
05:56:12 #   `kubernetes_wait_for_pod_ready_state "$pod_name" 20' failed
05:56:12 # Deleting previous test services...
05:56:12 # No resources found in default namespace.
05:56:12 # service/snp-unencrypted created
05:56:12 # deployment.apps/snp-unencrypted created
05:56:12 # error: timed out waiting for the condition on pods/snp-unencrypted-68dd975f64-g2c4k

Additional Information

Anything else to add?

Before raising this enhancement request

Have you looked at the limitations document?

@wainersm wainersm added enhancement Improvement to an existing feature needs-review Needs to be assessed by the team. labels Jul 12, 2023
wainersm added a commit that referenced this issue Jul 12, 2023
The SEV and SNP tests sometimes fail on CI and it is hard to determine
what went wrong because they don't print debug messages. This changed
the tests so that they behave like many others: print the system's logs
since the test's begin time.

Fixes #5708
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Improvement to an existing feature needs-review Needs to be assessed by the team.
Projects
None yet
Development

No branches or pull requests

1 participant