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.

Sign up for GitHub

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

Jump to bottom

CA: refactor utils related to NodeInfos #7479

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

CA: refactor utils related to NodeInfos #7479

wants to merge 2 commits into from

Commits on Nov 13, 2024

  1. TMP squash: kubernetes#7466 

    DONOTSUBMIT
    @towca
    towca committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    8ad16c3 View commit details
    Browse the repository at this point in the history

  2. DRA: refactor utils related to NodeInfos 

    simulator.BuildNodeInfoForNode, core_utils.GetNodeInfoFromTemplate,
and scheduler_utils.DeepCopyTemplateNode all had very similar logic
for sanitizing and copying NodeInfos. They're all consolidated to
one file in simulator, sharing common logic.

DeepCopyNodeInfo is changed to be a framework.NodeInfo method.

MixedTemplateNodeInfoProvider now correctly uses ClusterSnapshot to
correlate Nodes to scheduled pods, instead of using a live Pod lister.
This means that the snapshot now has to be properly initialized in a
bunch of tests.
    @towca
    towca committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    5a9ff97 View commit details
    Browse the repository at this point in the history