Skip to content

[Bug]: Fix legacy misconfigurations of autoscaling annotations. #1173

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

Merged

Conversation

zhangjyr
Copy link
Collaborator

@zhangjyr zhangjyr commented Jun 3, 2025

Pull Request Description

Due to possible implementation changes, some autoscaling configuration examples used the legacy approach, which placed autoscaling parameters under the label section. This PR scans the whole repository and fixes misconfigurations.

Related Issues

Resolves: #1159

Important: Before submitting, please complete the description above and review the checklist below.


Contribution Guidelines (Expand for Details)

We appreciate your contribution to aibrix! To ensure a smooth review process and maintain high code quality, please adhere to the following guidelines:

Pull Request Title Format

Your PR title should start with one of these prefixes to indicate the nature of the change:

  • [Bug]: Corrections to existing functionality
  • [CI]: Changes to build process or CI pipeline
  • [Docs]: Updates or additions to documentation
  • [API]: Modifications to aibrix's API or interface
  • [CLI]: Changes or additions to the Command Line Interface
  • [Misc]: For changes not covered above (use sparingly)

Note: For changes spanning multiple categories, use multiple prefixes in order of importance.

Submission Checklist

  • PR title includes appropriate prefix(es)
  • Changes are clearly explained in the PR description
  • New and existing tests pass successfully
  • Code adheres to project style and best practices
  • Documentation updated to reflect changes (if applicable)
  • Thorough testing completed, no regressions introduced

By submitting this PR, you confirm that you've read these guidelines and your changes align with the project's contribution standards.

Signed-off-by: Jingyuan Zhang <jingyuan.zhang0929@bytedance.com>
@zhangjyr zhangjyr requested review from Jeffwan, kr11 and nwangfw June 3, 2025 18:41
Copy link
Collaborator

@Jeffwan Jeffwan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

em. this is embarrassing. seems we didn't exam the correctness in the past

@Jeffwan Jeffwan merged commit d00fda3 into vllm-project:main Jun 3, 2025
16 of 17 checks passed
@zhangjyr zhangjyr deleted the issues/1159_scale_down_configurations branch June 4, 2025 17:54
ModiCodeCraftsman pushed a commit to ModiCodeCraftsman/aibrix that referenced this pull request Jun 25, 2025
…-project#1173)

Fix legacy mis-configuration of autoscaling annotations.

Signed-off-by: Jingyuan Zhang <jingyuan.zhang0929@bytedance.com>
Co-authored-by: Jingyuan Zhang <jingyuan.zhang0929@bytedance.com>
Signed-off-by: Modi Tamam <modi.tamam@gmail.com>
Yaegaki1Erika pushed a commit to Yaegaki1Erika/aibrix that referenced this pull request Jul 23, 2025
…-project#1173)

Fix legacy mis-configuration of autoscaling annotations.

Signed-off-by: Jingyuan Zhang <jingyuan.zhang0929@bytedance.com>
Co-authored-by: Jingyuan Zhang <jingyuan.zhang0929@bytedance.com>
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.

Optimizer-based autoscaler is slow to scale down
2 participants