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

[Flaking Test] [sig-scheduling] TestSchedulerSchedulePod #122312

Closed
pacoxu opened this issue Dec 14, 2023 · 6 comments · Fixed by #122317
Closed

[Flaking Test] [sig-scheduling] TestSchedulerSchedulePod #122312

pacoxu opened this issue Dec 14, 2023 · 6 comments · Fixed by #122317
Labels
kind/flake Categorizes issue or PR as related to a flaky test. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/scheduling Categorizes an issue or PR as relevant to SIG Scheduling.

Comments

@pacoxu
Copy link
Member

pacoxu commented Dec 14, 2023

Which jobs are flaking?

https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/122027/pull-kubernetes-unit/1735186633038237696

Which tests are flaking?

k8s.io/kubernetes/pkg: scheduler

{Failed;Failed;  === RUN   TestSchedulerSchedulePod/test_no_score_plugin,_prefilter_plugin_returning_2_nodes,_only_1_node_is_evaluated
    node_tree.go:65: I1214 06:59:28.565440] Added node in listed group to NodeTree node="node1" zone=""
    node_tree.go:65: I1214 06:59:28.565571] Added node in listed group to NodeTree node="node2" zone=""
    node_tree.go:65: I1214 06:59:28.565780] Added node in listed group to NodeTree node="node3" zone=""
W1214 06:59:28.565971   50310 mutation_detector.go:53] Mutation detector is enabled, this will result in memory leakage.
    framework.go:368: I1214 06:59:28.566481] the scheduler starts to work with those plugins Plugins={"PreEnqueue":{"Enabled":null,"Disabled":null},"QueueSort":{"Enabled":[{"Name":"PrioritySort","Weight":0}],"Disabled":null},"PreFilter":{"Enabled":[{"Name":"FakePreFilter","Weight":0}],"Disabled":null},"Filter":{"Enabled":null,"Disabled":null},"PostFilter":{"Enabled":null,"Disabled":null},"PreScore":{"Enabled":null,"Disabled":null},"Score":{"Enabled":null,"Disabled":null},"Reserve":{"Enabled":null,"Disabled":null},"Permit":{"Enabled":null,"Disabled":null},"PreBind":{"Enabled":null,"Disabled":null},"Bind":{"Enabled":[{"Name":"DefaultBinder","Weight":0}],"Disabled":null},"PostBind":{"Enabled":null,"Disabled":null},"MultiPoint":{"Enabled":null,"Disabled":null}}
    schedule_one_test.go:2437: Expected: map[node1:{} node3:{}], got: node2
    --- FAIL: TestSchedulerSchedulePod/test_no_score_plugin,_prefilter_plugin_returning_2_nodes,_only_1_node_is_evaluated (0.10s)
;=== RUN   TestSchedulerSchedulePod
--- FAIL: TestSchedulerSchedulePod (2.78s)
;}

Since when has it been flaking?

today after #122022 is merged. Edited

caused by #122058

Testgrid link

NA

Reason for failure (if possible)

N/A

Anything else we need to know?

/cc @sanposhiho

Relevant SIG(s)

/sig scheduling

@pacoxu pacoxu added the kind/flake Categorizes issue or PR as related to a flaky test. label Dec 14, 2023
@k8s-ci-robot k8s-ci-robot added sig/scheduling Categorizes an issue or PR as relevant to SIG Scheduling. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Dec 14, 2023
@k8s-ci-robot
Copy link
Contributor

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.

@AxeZhan
Copy link
Member

AxeZhan commented Dec 14, 2023

/cc

@sanposhiho
Copy link
Member

I believe, after merging #122058 is more correct.
The flaky test is introduced by that PR.

@sanposhiho
Copy link
Member

cc @aleksandra-malinowska

@pacoxu
Copy link
Member Author

pacoxu commented Dec 14, 2023

I believe, after merging #122058 is more correct. The flaky test is introduced by that PR.

Thanks for correcting me. I updated the issue description.

@pacoxu
Copy link
Member Author

pacoxu commented Dec 14, 2023

https://prow.k8s.io/job-history/gs/kubernetes-jenkins/pr-logs/directory/pull-kubernetes-unit

  • failed 4 in last ~20 runs, a very high flake rate.

I would suggest reverting it or fixing it ASAP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/flake Categorizes issue or PR as related to a flaky test. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/scheduling Categorizes an issue or PR as relevant to SIG Scheduling.
Projects
Development

Successfully merging a pull request may close this issue.

4 participants