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
Implement QueueingHintFn in VolumeRestriction #119405
base: master
Are you sure you want to change the base?
Implement QueueingHintFn in VolumeRestriction #119405
Conversation
This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The 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. |
Hi @HirazawaUi. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
/ok-to-test |
/retest |
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
cc417a3
to
bb4c899
Compare
@AxeZhan thanks for review, I've made changes to the suggest |
/retest |
2 similar comments
/retest |
/retest |
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
187283c
to
8da238a
Compare
/retest |
@AxeZhan Can you take the first reviewer on this? Then I'll go through after your |
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
156f844
to
6e7f6f7
Compare
I have rebased and replaced QueueAfterBackoff with Queue. |
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
34710b0
to
f2063d1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Back from Kubecon + post-Kubecon days with tons of tasks made during Kubecon. 😩
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Show resolved
Hide resolved
d0ce032
to
a189004
Compare
/retest |
a189004
to
6b76cfb
Compare
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
6b76cfb
to
701046a
Compare
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Show resolved
Hide resolved
701046a
to
107ed2d
Compare
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/volumerestrictions/volume_restrictions.go
Show resolved
Hide resolved
107ed2d
to
1e3fca7
Compare
The Kubernetes project currently lacks enough contributors to adequately respond to all PRs. This bot triages PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Which issue(s) this PR fixes:
Part of #118893
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: