Skip to content

Conversation

wainersm
Copy link
Contributor

@wainersm wainersm commented Jul 3, 2025

We've seen tests failing occasionally on CoCo non-TEE job. When I try to reproduce them on my environment, even after run 20x, they never fail again. Do they have something against me personally? I don't know, but I know that some concurrency problems can be hard to reproduce...

This PR aims to improve these tests to print more information (notably system's logs) when they fail, so that we can have a chance (even if small) of spotting the problems.

@wainersm wainersm added the area/testing Issues related to testing label Jul 3, 2025
Copy link
Member

@stevenhorsman stevenhorsman left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @wainersm!

Copy link
Member

@BbolroC BbolroC left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @wainersm !

wainersm added 4 commits July 12, 2025 10:13
The teardown_common will print the description of the running pods, kill
them all and print the system's syslogs afterwards.

Signed-off-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
The teardown_common will print the description of the running pods, kill
them all and print the system's syslogs afterwards.

Signed-off-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
Set the node in the spec template of a Job manifest, allowing to use
set_node() on tests like k8s-parallel.bats

Signed-off-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
The teardown_common will print the description of the running pods, kill
them all and print the system's syslogs afterwards.

Signed-off-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
@stevenhorsman stevenhorsman merged commit 0fa2cd8 into kata-containers:main Jul 14, 2025
647 of 716 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/testing Issues related to testing ok-to-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants