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

export the logs of Volcano components when the e2e test workflow fails #3429

Closed
panoswoo opened this issue Apr 19, 2024 · 4 comments · Fixed by #3438
Closed

export the logs of Volcano components when the e2e test workflow fails #3429

panoswoo opened this issue Apr 19, 2024 · 4 comments · Fixed by #3438
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@panoswoo
Copy link
Contributor

What would you like to be added:

print the logs of volcano components in clean up stage

# clean up
function cleanup {
uninstall-volcano
echo "Running kind: [kind delete cluster ${CLUSTER_CONTEXT}]"
kind delete cluster ${CLUSTER_CONTEXT}
if [[ ${SHOW_VOLCANO_LOGS} -eq 1 ]]; then
#TODO: Add volcano logs support in future.
echo "Volcano logs are currently not supported."
fi
}

, the logs of volcano components are generated by
function generate-log {
echo "Generating volcano log files"
kubectl logs deployment/${CLUSTER_NAME}-admission -n ${NAMESPACE} > volcano-admission.log
kubectl logs deployment/${CLUSTER_NAME}-controllers -n ${NAMESPACE} > volcano-controller.log
kubectl logs deployment/${CLUSTER_NAME}-scheduler -n ${NAMESPACE} > volcano-scheduler.log
}

when e2e test is failed
if [[ $? -ne 0 ]]; then
generate-log
exit 1
fi

Why is this needed:

when the e2e test is failed, we need those logs for troube shorting

@panoswoo panoswoo added the kind/feature Categorizes issue or PR as related to a new feature. label Apr 19, 2024
@Monokaix
Copy link
Member

/assign @panoswoo

@Monokaix
Copy link
Member

You're welcome to do that.

@calvin0327
Copy link

@panoswoo @Monokaix Is this at work? I can do this

@panoswoo
Copy link
Contributor Author

@panoswoo @Monokaix Is this at work? I can do this

sorry, I have already started this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants