Skip to content

chore: move h-scale lifecycle into ITS#10105

Open
leon-ape wants to merge 3 commits intomainfrom
support/hscale-to-its
Open

chore: move h-scale lifecycle into ITS#10105
leon-ape wants to merge 3 commits intomainfrom
support/hscale-to-its

Conversation

@leon-ape
Copy link
Copy Markdown
Contributor

@leon-ape leon-ape commented Mar 25, 2026

Summary

  • Move horizontal scale lifecycle ownership into the workload controller.
  • Add pod-centric lifecycle state and execution for data load, member join, and member leave in InstanceSet.
  • Turn component into a compatibility shell that translates intent and consumes InstanceSet state, while removing the old replicas-status and task-event based execution path.

@leon-ape leon-ape requested a review from a team as a code owner March 25, 2026 03:08
@apecloud-bot
Copy link
Copy Markdown
Collaborator

Auto Cherry-pick Instructions

Usage:
  - /nopick: Not auto cherry-pick when PR merged.
  - /pick: release-x.x [release-x.x]: Auto cherry-pick to the specified branch when PR merged.

Example:
  - /nopick
  - /pick release-1.1

@github-actions github-actions bot added the size/XXL Denotes a PR that changes 1000+ lines. label Mar 25, 2026
@leon-ape
Copy link
Copy Markdown
Contributor Author

/nopick

@leon-ape leon-ape changed the title feat: move h-scale lifecycle into instanceset feat: move h-scale lifecycle into ITS Mar 25, 2026
@apecloud-bot apecloud-bot added the nopick Not auto cherry-pick when PR merged label Mar 25, 2026
@leon-ape leon-ape added this to the Release 1.2.0 milestone Mar 25, 2026
@leon-ape leon-ape changed the title feat: move h-scale lifecycle into ITS chore: move h-scale lifecycle into ITS Mar 25, 2026
@leon-ape leon-ape force-pushed the support/hscale-to-its branch from 3d1db64 to 5c8daef Compare March 25, 2026 03:37
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 25, 2026

Codecov Report

❌ Patch coverage is 63.13725% with 94 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.86%. Comparing base (edbff89) to head (5c8daef).

Files with missing lines Patch % Lines
...oller/instanceset/reconciler_instance_alignment.go 57.14% 50 Missing and 19 partials ⚠️
apis/workloads/v1/instanceset_types.go 0.00% 10 Missing ⚠️
...ers/apps/component/transformer_component_status.go 25.00% 4 Missing and 2 partials ⚠️
pkg/controller/builder/builder_instance_set.go 0.00% 3 Missing ⚠️
pkg/controller/lifecycle/lfa_data.go 66.66% 2 Missing and 1 partial ⚠️
...ps/component/transformer_component_workload_ops.go 71.42% 1 Missing and 1 partial ⚠️
...s/apps/component/transformer_component_workload.go 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10105      +/-   ##
==========================================
+ Coverage   51.60%   51.86%   +0.25%     
==========================================
  Files         525      524       -1     
  Lines       58397    57980     -417     
==========================================
- Hits        30138    30069      -69     
+ Misses      25305    24970     -335     
+ Partials     2954     2941      -13     
Flag Coverage Δ
unittests 51.86% <63.13%> (+0.25%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

nopick Not auto cherry-pick when PR merged size/XXL Denotes a PR that changes 1000+ lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants