-
Notifications
You must be signed in to change notification settings - Fork 9
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: improve log collection #985
Conversation
e46a38c
to
d6b9c9c
Compare
E2E runs that use the new log collection: https://github.com/edgelesssys/contrast/actions/runs/11799318639?pr=985 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great - I did not think the implementation could be this concise!
62bfc50
to
6a6b45c
Compare
Run with new script structure and working log collection: https://github.com/edgelesssys/contrast/actions/runs/11912439438?pr=985 |
6a6b45c
to
fe1570d
Compare
Improve the log collection to also collect init container logs. This is done by adding a
log-collector
daemon set to every e2e cluster that saves logs from the host node which then are downloaded to the local machine through the updatedget-logs
script.