Provides flags for users to expose and set UnschedulablePodTime…

[aiden-von]

What type of PR is this?

/kind feature

What this PR does / why we need it:

Provides flags for users to expose and set UnschedulablePodTimeBuffer, UnschedulablePodWithGpuTimeBuffer.

Which issue(s) this PR fixes:

Fixes #6087

Does this PR introduce a user-facing change?

NONE

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: aiden-von
Once this PR has been reviewed and has the lgtm label, please assign bigdarkclown for approval. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files:

• cluster-autoscaler/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[aiden-von]

Github Action has not been working for more than 30 minutes and is in a Action required state.
Can someone help me?

[aiden-von]

/test ?

[k8s-ci-robot]

@aiden-von: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.

In response to this:

/test ?

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.

[k8s-ci-robot]

@aiden-von: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.

In response to this:

/ok-to-test

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.

[injae-kim]

Hi @vadasambar, can you review this PR? I need this feature 🙇 thanks!

[vadasambar]

/assign vadasambar

[vadasambar]

/ok-to-test

[vadasambar]

@x13n we might need your approval to run the test workflow 🙏

[x13n]

Actually I'd instead wait for #6235 which goes even further - bypassing not just the buffer time, but even kube-scheduler processing. Would this be an acceptable fix for #6087?

[aiden-von]

Hello, @x13n

If I understand correctly, #6235 seems to ignore kube-scheduler processing.

I think we need to check whether the person who created this issue wants to reduce buffer time or ignore scheduling, but I think #6235 could be an alternative.

If you think the issue(#6087) is resolved by the PR(#6235), I wouldn't mind closing this PR.
Thank you.

[x13n]

@maxpain - does that sound like a good fix for #6087 you created?

[k8s-ci-robot]

PR needs rebase.

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.

[x13n]

I'm guessing the answer is "yes", will close this PR due to inactivity. Please reopen if anything changes.

/close

[k8s-ci-robot]

@x13n: Closed this PR.

In response to this:

I'm guessing the answer is "yes", will close this PR due to inactivity. Please reopen if anything changes.

/close

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.