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

test: Cleanup ginkgo test artifacts #25833

Merged
merged 2 commits into from Jun 6, 2023

Conversation

pchaigno
Copy link
Member

@pchaigno pchaigno commented Jun 1, 2023

Clean up various stuff in ginkgo artifacts that are now duplicated because we collect a full sysdump (cf. #25079). See commits for details.

Hopefully, that should bring the average size of ginkgo artifacts below 25MB again, and allow uploads to GitHub.

Since commit e93fdd8 ("test: Collect sysdump as part of artifacts"),
bugtool reports are already collected as part of the sysdump. Now that
we are confident the sysdump collection works, we can drop the separate
collection of bugtool reports.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
Those kubectl commands are already part of the sysdump collected for
artifacts since commit e93fdd8 ("test: Collect sysdump as part of
artifacts").

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
@pchaigno pchaigno added the release-note/ci This PR makes changes to the CI. label Jun 1, 2023
@pchaigno pchaigno requested a review from a team as a code owner June 1, 2023 14:44
@pchaigno pchaigno requested a review from nebril June 1, 2023 14:44
@pchaigno
Copy link
Member Author

pchaigno commented Jun 1, 2023

/test

@pchaigno pchaigno added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 6, 2023
@dylandreimerink dylandreimerink merged commit 96c8b47 into cilium:main Jun 6, 2023
54 checks passed
@pchaigno pchaigno deleted the cleanup-ginkgo-test-artifacts branch June 6, 2023 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants