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: gather containerd logs on failure #24133

Merged
merged 1 commit into from Mar 7, 2023

Conversation

squeed
Copy link
Contributor

@squeed squeed commented Mar 2, 2023

To aid in debugging.

Sometimes containerd gets sad, and the only way to feel better is to talk about it.

To aid in debugging. Sometimes containerd gets sad, and the only way to
feel better is to talk about it.

Signed-off-by: Casey Callendrello <cdc@isovalent.com>
@squeed squeed added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Mar 2, 2023
@squeed squeed requested a review from a team as a code owner March 2, 2023 10:51
@squeed squeed requested a review from tklauser March 2, 2023 10:51
@squeed
Copy link
Contributor Author

squeed commented Mar 3, 2023

/test

@squeed
Copy link
Contributor Author

squeed commented Mar 7, 2023

It seems some of the github actions just... disappeared. However, this extremely minor PR only affects Jenkins, which passed completely. Manually marking as ready-to-merge

@squeed squeed added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 7, 2023
@YutaroHayakawa YutaroHayakawa merged commit 6a1a0c5 into cilium:master Mar 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake 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