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: improve log collection #985

Merged
merged 22 commits into from
Nov 21, 2024
Merged

Conversation

miampf
Copy link
Contributor

@miampf miampf commented Nov 7, 2024

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 updated get-logs script.

@miampf miampf requested a review from burgerdev November 7, 2024 12:26
@miampf miampf added the no changelog PRs not listed in the release notes label Nov 7, 2024
@miampf miampf force-pushed the miampf/also-collect-init-container-logs branch from e46a38c to d6b9c9c Compare November 12, 2024 10:20
@miampf miampf marked this pull request as ready for review November 12, 2024 14:47
@miampf miampf requested a review from katexochen as a code owner November 12, 2024 14:47
@miampf
Copy link
Contributor Author

miampf commented Nov 12, 2024

E2E runs that use the new log collection: https://github.com/edgelesssys/contrast/actions/runs/11799318639?pr=985

Copy link
Contributor

@burgerdev burgerdev left a 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!

packages/scripts.nix Outdated Show resolved Hide resolved
@miampf miampf force-pushed the miampf/also-collect-init-container-logs branch 4 times, most recently from 62bfc50 to 6a6b45c Compare November 19, 2024 11:51
@miampf
Copy link
Contributor Author

miampf commented Nov 19, 2024

Run with new script structure and working log collection: https://github.com/edgelesssys/contrast/actions/runs/11912439438?pr=985

@miampf miampf requested a review from burgerdev November 19, 2024 12:03
packages/scripts.nix Outdated Show resolved Hide resolved
@miampf miampf force-pushed the miampf/also-collect-init-container-logs branch from 6a6b45c to fe1570d Compare November 21, 2024 10:38
@miampf miampf merged commit ba7fb9d into main Nov 21, 2024
11 checks passed
@miampf miampf deleted the miampf/also-collect-init-container-logs branch November 21, 2024 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants