search

kubernetes / test-infra

Test infrastructure for the Kubernetes project.
Apache License 2.0
3.82k stars 2.63k forks source link

crio: use separate yaml to configure crun and rename default crio.conf #32899

Closed sohankunkerkar closed 2 months ago

sohankunkerkar commented 2 months ago

Fixes: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/123303/pull-kubernetes-node-crio-cgrpv2-userns-e2e/1808152463027474432 Ginkgo identified the test filter but did not run the required tests. The CRI-O drop-in config change was not applied to the final configuration, which led to the tests not being run.

sohankunkerkar commented 2 months ago

/retest

haircommander commented 2 months ago

/lgtm /approve

k8s-ci-robot commented 2 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: haircommander, saschagrunert, sohankunkerkar

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[jobs/e2e_node/crio/OWNERS](https://github.com/kubernetes/test-infra/blob/master/jobs/e2e_node/crio/OWNERS)~~ [haircommander,saschagrunert] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment