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

[release-0.17][manual] backport scheduler cache tunings enhancement #274

Merged
merged 3 commits into from
Feb 8, 2024

Conversation

ffromani
Copy link
Collaborator

@ffromani ffromani commented Feb 8, 2024

cumulative backport of PRs #250 and #270

The test
```
[PositiveFlow] Deployer execution with a running cluster without any components
  when deployed with resource-topology-exporter as the updater [It] should verify
  a test pod scheduled with the topology aware scheduler goes running
```

started to flake lately with the symptoms matching a timeout due to slow
CI. Relax the timeout and add more logs to (try to) deflake the issue.

Signed-off-by: Francesco Romani <[email protected]>
Add support to gather and set cache fine-tuning parameters:
ForeignPodsDetectMode and CacheResyncMethod, both exposed
in the `cache` subfield of the plugin parameters (at least
cacheResyncPeriodSeconds also belong there, but this is
another story for another day).

Also add validation support for the parameters.

Signed-off-by: Francesco Romani <[email protected]>
The scheduler plugin gained support for the external
informer, necessary to overcome  mismatch between the scheduler
and the ode views wrt the pods on nodes.
Expose support in the deployer libraries to change this setting.

Signed-off-by: Francesco Romani <[email protected]>
@ffromani ffromani merged commit 487988a into release-0.17 Feb 8, 2024
1 check passed
@ffromani ffromani deleted the sched-enh-0.17 branch February 8, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant