☂️Slow Unit Test 🌂 1.30/1.31

[pacoxu]

https://prow.k8s.io/job-history/gs/kubernetes-jenkins/logs/ci-kubernetes-unit-1-27 takes ~35min
https://prow.k8s.io/job-history/gs/kubernetes-jenkins/logs/ci-kubernetes-unit taks ~40min

There are some adding UT. New below is comparing with v1.27. (The test or package may be old, but it takes more than 1m in UT in master, and not take more than 1m in v1.27.)

• New k8s.io/apiserver/pkg/admission/plugin/policy: validating 1m24s apiserver/pkg/admission/plugin/policy/validating make test parallel in for loop #124036
• k8s.io/apiserver/pkg: server apiserver/pkg/server decrease mock handler sleep time from 60s to 10s #124037
• k8s.io/apiserver/pkg/server: filters
• k8s.io/apiserver/pkg/server: options apiserver: decrease timeout for EncryptionConfiguration tests #124082 apiserver: decrease timeout for TestKMSHealthzEndpoint #124087
• New k8s.io/apiserver/pkg/server/options: encryptionconfig
• k8s.io/apiserver/pkg/storage: cacher ~3m. still one of the slowest package
  • Timeout is fixed: we have a 3 minute timeout [Flaking Test] UT k8s.io/apiserver/pkg/storage cacher #123850.
• k8s.io/apiserver/pkg/storage: etcd3
• k8s.io/apiserver/pkg/util: flowcontrol
• New k8s.io/apiserver/pkg/util: peerproxy
• New k8s.io/kubectl/pkg/cmd: apply Improve performance of TestForceApply unit test (k8s.io/kubectl/pkg/cmd/apply) #124494
• k8s.io/kubernetes/pkg/controller: endpoint
• k8s.io/kubernetes/pkg/controller: endpointslice
• k8s.io/kubernetes/pkg/controller: nodeipam
• k8s.io/kubernetes/pkg/controller: statefulset
• New k8s.io/kubernetes/pkg/kubelet/cm: dra ~3m ⚠️ we have a 3 minute timeout kubelet: fix slow dra unit test #123720
• k8s.io/kubernetes/pkg/kubelet/volumemanager: reconciler
• k8s.io/kubernetes/pkg/registry/core/pod: storage
• k8s.io/kubernetes/pkg/registry/core/service: ipallocator
• k8s.io/kubernetes/pkg/registry/core/service: storage
• k8s.io/kubernetes/pkg/scheduler/framework/plugins: volumebinding
• k8s.io/kubernetes/pkg/volume: csi

/sig node api-machinery cli
for new ones.

#98486 we did a similar check in 2021 before.

---

#123850 (comment) We have a 3 minute timeout for a package.

kubernetes/hack/make-rules/test.sh

Lines 60 to 64 in ebc1a7b

	# TODO: This timeout should really be lower, this is a *long* time to test one
	# package, however pkg/api/testing in particular will fail with a lower timeout
	# currently. We should attempt to lower this over time.
	KUBE_TIMEOUT=${KUBE_TIMEOUT:--timeout=180s}
	KUBE_COVER=${KUBE_COVER:-n} # set to 'y' to enable coverage collection

[k8s-ci-robot]

This issue is currently awaiting triage.

If a SIG or subproject determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[pacoxu]

/area test
/kind cleanup
/help wanted

[pacoxu]

/sig node api-machinery cli

[HirazawaUi]

/help

[k8s-ci-robot]

@HirazawaUi:
This request has been marked as needing help from a contributor.

Guidelines

Please ensure that the issue body includes answers to the following questions:

• Why are we solving this issue?
• To address this issue, are there any code changes? If there are code changes, what needs to be done in the code and what places can the assignee treat as reference points?
• Does this issue have zero to low barrier of entry?
• How can the assignee reach out to you for help?

For more details on the requirements of such an issue, please see here and ensure that they are met.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

/help

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[HirazawaUi]

k8s.io/kubernetes/pkg/kubelet/cm/dra #123720

time required before modification:

go test ./... -cover -count=1
ok  	k8s.io/kubernetes/pkg/kubelet/cm/dra	94.097s	coverage: 83.8% of statements

time required after modification

go test ./... -cover -count=1
ok  	k8s.io/kubernetes/pkg/kubelet/cm/dra	1.356s	coverage: 83.8% of statements

[p0lyn0mial]

I will try to decrease the running time of the apiserver/pkg/storage/cacher pkg.

[leilajal]

/remove-sig api-machinery

[SergeyKanzhelev]

/assign @bart0sh

for k8s.io/kubernetes/pkg/kubelet/cm: dra

[HirazawaUi]

/assign @bart0sh

for k8s.io/kubernetes/pkg/kubelet/cm: dra

Fixed in #123720

[brianpursley]

This one has been fixed by #124494

[ ] New k8s.io/kubectl/pkg/cmd: apply