Skip to content

Update Autoscaler Warmup deployment path#607

Merged
sumanthreddy29 merged 1 commit into
mainfrom
fix-nap-test
Apr 29, 2025
Merged

Update Autoscaler Warmup deployment path#607
sumanthreddy29 merged 1 commit into
mainfrom
fix-nap-test

Conversation

@sumanthreddy29
Copy link
Copy Markdown
Collaborator

@sumanthreddy29 sumanthreddy29 commented Apr 29, 2025

This pull request updates the autoscale.py script and its associated configuration to support a configurable directory for ClusterLoader2 (CL2) configuration files. The changes ensure that paths to configuration files are dynamically determined based on a new cl2_config_dir parameter, improving flexibility and usability.

Updates to autoscale.py:

  • Parameterization of CL2 configuration directory:

    • Modified warmup_deployment_for_karpeneter and cleanup_warmup_deployment_for_karpeneter to accept cl2_config_dir as a parameter, replacing hardcoded paths to the warmup deployment file.
    • Updated calculate_cpu_request_for_clusterloader2 and override_config_clusterloader2 to include cl2_config_dir as an argument, ensuring all related functions use the configurable directory. [1] [2]
    • Added a new command-line argument, cl2_config_dir, to the main function for specifying the CL2 configuration directory.
  • Integration with the main function:

    • Updated the call to override_config_clusterloader2 in the main function to pass the new cl2_config_dir argument.

Updates to execute.yml:

  • Pipeline script compatibility:
    • Updated the steps/engine/clusterloader2/autoscale/execute.yml file to pass the CL2_CONFIG_DIR environment variable as the cl2_config_dir parameter when invoking the override command.This pull request updates the file paths in the autoscale.py script to reflect a new directory structure for deployment configuration files.

Path updates for deployment files:

  • In the warmup_deployment_for_karpeneter and cleanup_warmup_deployment_for_karpeneter functions, the deployment_file path was updated from "autoscale/config/warmup_deployment.yaml" to "clusterloader2/autoscale/config/warmup_deployment.yaml", ensuring compatibility with the updated directory structure.

@sumanthreddy29 sumanthreddy29 changed the title Update Warmup Deployment Path for NAP tests Update Autoscaler Warmup deployment path Apr 29, 2025
@sumanthreddy29 sumanthreddy29 merged commit 12a54e5 into main Apr 29, 2025
3 checks passed
@sumanthreddy29 sumanthreddy29 deleted the fix-nap-test branch July 8, 2025 15:37
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.

2 participants