-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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 the integration tests for requeueing scenarios #122305
Comments
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. |
/assign @carlory @sanposhiho |
A basic thought that I have now is that we have to cover the most common scenario only in an integration test, we don't have to cover all scenarios per each plugin. Plus, it'd be great if we could have a scenario where preCheck scenario is covered.
|
@sanposhiho There's already an existing intergation test. it follows similar steps. |
Then, we may want to refactor that test to be a more general one. |
As part of #122292, I implemented the e2e test, which would be a good starting point of this. |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale #122292 is implemented, we have to cultivate the test cases there for existing QHints. |
/sig scheduling
/priority important-soon
/kind feature
Part of: #122284 (comment).
We have few tests to check the scheduler's requeueing scenario in the integration test.
We should have the one so that we can catch a bug like #122284, which only happens the scheduling queue is actually working.
The text was updated successfully, but these errors were encountered: