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

e2e: add facilities to improve troubleshooting #296

Merged
merged 7 commits into from
Mar 5, 2024

Conversation

ffromani
Copy link
Collaborator

@ffromani ffromani commented Mar 5, 2024

Add helpers to make it easier to understand the test failures.

@ffromani ffromani force-pushed the debug-ci branch 7 times, most recently from 2f2854b to 2098527 Compare March 5, 2024 15:48
ffromani added 3 commits March 5, 2024 16:52
make sure to set -v=5 for both scheduler and RTE to catch
all the debug messages for better troubleshooting.

Signed-off-by: Francesco Romani <[email protected]>
dump kernel version to help troubleshooting

Signed-off-by: Francesco Romani <[email protected]>
move utility function in the main suite file,
to foster code reuse.

Trivial code movement, no expected changes in behavior.

Signed-off-by: Francesco Romani <[email protected]>
ffromani added 2 commits March 5, 2024 17:34
Improve the reusability of the helpers, accepting
clients and contexts instead of recreating them every single time.

Signed-off-by: Francesco Romani <[email protected]>
extract a helper function to ensure the NRT data
is correctly populated.

Signed-off-by: Francesco Romani <[email protected]>
make sure the NRT data looks correct before attempting
to run a pod which consumes it.

Signed-off-by: Francesco Romani <[email protected]>
make sure to dump the components state on error,
to make troubleshooting easier. This includes
pod statuses and component logs first and foremost.

Signed-off-by: Francesco Romani <[email protected]>
@ffromani ffromani merged commit 1f994c8 into k8stopologyawareschedwg:main Mar 5, 2024
4 checks passed
@ffromani ffromani deleted the debug-ci branch March 5, 2024 17:51
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.

1 participant