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

conformance-k8s-kind: disable kindnet, enable log dumping #24982

Merged
merged 1 commit into from Apr 20, 2023

Conversation

squeed
Copy link
Contributor

@squeed squeed commented Apr 19, 2023

While debugging a test failure, I noticed that this workflow missed disabling the kindnet CNI (every other workflow already does this).

Separately, enable log dumping on failure, otherwise there isn't enough to go on when tests fail.

While debugging a test failure, I noticed that this workflow missed
disabling the kindnet CNI (every other workflow already does this).

Separately, enable log dumping on failure, otherwise there isn't enough
to go on when tests fail.

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 Apr 19, 2023
@squeed squeed requested review from a team as code owners April 19, 2023 14:25
@squeed squeed requested a review from brlbil April 19, 2023 14:25
@squeed
Copy link
Contributor Author

squeed commented Apr 19, 2023

fyi @aojea, as the author of this job, you might be interested.

@squeed
Copy link
Contributor Author

squeed commented Apr 19, 2023

The affected job has passed.

I'm going to re-trigger it in an attempt to get a failure :-).

@aojea
Copy link
Contributor

aojea commented Apr 20, 2023

/lgtm

@squeed
Copy link
Contributor Author

squeed commented Apr 20, 2023

marking as ready-to-merge; the only failing test is a different github run and there's no point in running Jenkins.

@squeed squeed added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Apr 20, 2023
@ldelossa ldelossa merged commit a9291f6 into cilium:main Apr 20, 2023
43 of 44 checks passed
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

4 participants