WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

e2e: verify podvm console logs #2640

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.

Sign up for GitHub

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

Jump to bottom
Draft
PrakashMuddana wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

e2e: verify podvm console logs #2640

PrakashMuddana wants to merge 5 commits into from

Conversation

@PrakashMuddana
Copy link
Contributor

@PrakashMuddana PrakashMuddana commented Oct 16, 2025

validating whether console logs contains expected string or not and failing the testcase based on results

stevenhorsman
stevenhorsman reviewed Oct 23, 2025
View reviewed changes
@PrakashMuddana PrakashMuddana force-pushed the podvm-console-validation branch 2 times, most recently from 1796c1d to e4662a8 Compare October 28, 2025 04:53
@stevenhorsman stevenhorsman added the test_e2e_libvirt Run Libvirt e2e tests label Oct 28, 2025
stevenhorsman
stevenhorsman reviewed Oct 28, 2025
View reviewed changes
@PrakashMuddana PrakashMuddana force-pushed the podvm-console-validation branch from e4662a8 to ccbb321 Compare October 28, 2025 09:15
@stevenhorsman
Copy link
Member

stevenhorsman commented Oct 28, 2025

We need a go mod tidy on the cloud-api-adaptor here (for reasons not clear to me!)

@PrakashMuddana PrakashMuddana force-pushed the podvm-console-validation branch 2 times, most recently from 293aaf1 to 4e7608a Compare October 28, 2025 12:11
@stevenhorsman
Copy link
Member

stevenhorsman commented Oct 29, 2025

There is some issue with the code as the tests are all timing out with a stack trace of:

github.com/confidential-containers/cloud-api-adaptor/src/cloud-api-adaptor/test/e2e.LibvirtAssert.VerifyPodvmConsole({{0x7faa140047f0}}, 0xc000cb28c0, {0xc000c09500, 0x1e}, {0x357e025, 0x5})
	/home/runner/work/cloud-api-adaptor/cloud-api-adaptor/src/cloud-api-adaptor/test/e2e/libvirt_common.go:132 +0x3a5 fp=0xc000fcb310 sp=0xc000fcb1d8 pc=0x2db7145
github.com/confidential-containers/cloud-api-adaptor/src/cloud-api-adaptor/test/e2e.(*LibvirtAssert).VerifyPodvmConsole(0x399b328?, 0xc00107e120?, {0xc000c09500?, 0xc0003fef70?}, {0x357e025?, 0xa7374707665643a?})
	<autogenerated>:1 +0x2e fp=0xc000fcb350 sp=0xc000fcb310 pc=0x2dbccee
github.com/confidential-containers/cloud-api-adaptor/src/cloud-api-adaptor/test/e2e.(*TestCase).Run.func2({0x399b328, 0xc00107e120}, 0xc000cb28c0, 0x0?)
	/home/runner/work/cloud-api-adaptor/cloud-api-adaptor/src/cloud-api-adaptor/test/e2e/assessment_runner.go:322 +0x4eb fp=0xc000fcbeb0 sp=0xc000fcb350 pc=0x2da60eb

@PrakashMuddana PrakashMuddana force-pushed the podvm-console-validation branch from 4e7608a to 24b5d56 Compare November 3, 2025 18:37
@PrakashMuddana
e2e: verify podvm console logs
86c2313 
validating whether console logs contains expected string or not and failing the testcase based on results

Signed-off-by: Jyoti-Prakash-Muddanna <[email protected]>
@PrakashMuddana PrakashMuddana force-pushed the podvm-console-validation branch from 24b5d56 to 86c2313 Compare November 4, 2025 00:16
@PrakashMuddana PrakashMuddana force-pushed the podvm-console-validation branch from bafe096 to 0997e4c Compare December 3, 2025 08:48
@PrakashMuddana
debugging : testing with simple pod
62e9b0f 
testing with simple pod

Signed-off-by: Jyoti-Prakash-Muddanna <[email protected]>
@PrakashMuddana PrakashMuddana force-pushed the podvm-console-validation branch from 0997e4c to 62e9b0f Compare December 3, 2025 09:39
@PrakashMuddana
debugging: check console logs for 3 min
c36dbb6 
print if couldn't find expected string

Signed-off-by: Jyoti-Prakash-Muddanna <[email protected]>
@PrakashMuddana
debugging: added few logs
92bb138 
logs added to check the flow

Signed-off-by: Jyoti-Prakash-Muddanna <[email protected]>
@PrakashMuddana PrakashMuddana force-pushed the podvm-console-validation branch 2 times, most recently from 0901266 to b521d26 Compare December 4, 2025 10:41
@PrakashMuddana
debug: minor changes in code
a2e8b96 
code

Signed-off-by: Jyoti-Prakash-Muddanna <[email protected]>
@PrakashMuddana PrakashMuddana force-pushed the podvm-console-validation branch from b521d26 to a2e8b96 Compare December 5, 2025 04:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stevenhorsman stevenhorsman stevenhorsman left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

test_e2e_libvirt Run Libvirt e2e tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@PrakashMuddana @stevenhorsman