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
"unbound immediate PersistentVolumeClaims" should be UnschedulableAnd… #93892
Conversation
…Unresolvable error This was fixed in 1.19 by refactoring that part into PreFilter in kubernetes#91775
/assign @cofyc @msau42 @alculquicondor |
@@ -809,9 +809,9 @@ func TestFindPodVolumesWithoutProvisioning(t *testing.T) { | |||
shouldFail: true, | |||
}, | |||
"prebound-pvc": { | |||
podPVCs: []*v1.PersistentVolumeClaim{preboundPVC}, |
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.
@cofyc can you double check if the change to this test case is expected?
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.
yes
@@ -850,9 +850,9 @@ func TestFindPodVolumesWithoutProvisioning(t *testing.T) { | |||
reasons: ConflictReasons{ErrReasonBindConflict}, | |||
}, | |||
"one-prebound,one-unbound": { |
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.
ditto,@cofyc can you double check if the change to this test case is expected?
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.
yes
lgtm from scheduling PoV |
/lgtm |
/approve I think all cherry-picks need a release note. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ahg-g, msau42 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:
Approvers can indicate their approval by writing |
/sig-storage |
/sig storage |
/retest |
2 similar comments
/retest |
/retest |
friendly ping @kubernetes/release-managers |
@justaugustus can you please take a look at this PR? |
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.
/lgtm
/retest |
/retest Review the full test history for this PR. Silence the bot with an |
What type of PR is this?
/kind bug
What this PR does / why we need it:
"unbound immediate PersistentVolumeClaims" should be UnschedulableAndUnresolvable error
This was fixed in 1.19 by refactoring that part into PreFilter in #91775
This is a bug because it is currently causing incorrect reporting in scheduling_attempts metric (internal error vs unschedulable).
Which issue(s) this PR fixes:
Fixes ##91755
Does this PR introduce a user-facing change?: