Skip to content

Add /kuvel loadbalancer register/unregister subcommands and extend repair auto-recovery#69

Draft
Copilot wants to merge 3 commits intodevelop/v3from
copilot/add-loadbalancer-subcommand
Draft

Add /kuvel loadbalancer register/unregister subcommands and extend repair auto-recovery#69
Copilot wants to merge 3 commits intodevelop/v3from
copilot/add-loadbalancer-subcommand

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 4, 2026

/kuvel で Pod だけでなく LoadBalancer(ReplicaSet UID)の手動登録・削除ができるようにし、運用時の復旧導線を揃えました。あわせて /kuvel repair を強化し、壊れた loadbalancer マッピングを削除前に再登録で自己修復できるようにしています。

  • Command surface expansion (/kuvel)

    • Added register loadbalancer subcommand:
      • /kuvel register loadbalancer <replicaSetUid> <serverName>
    • Added unregister loadbalancer subcommand:
      • /kuvel unregister loadbalancer <replicaSetUid>
    • Kept existing pod-oriented subcommands backward compatible (register <podUid> ..., unregister <podUid>).
  • Service-layer support for manual loadbalancer registration

    • Added KuvelServiceHandler#registerLoadBalancer(String replicaSetUid, String serverName).
    • Added KuvelServiceHandler#isLoadBalancerRegistered(String replicaSetUid).
    • Registration flow resolves ReplicaSet by UID under configured selectors, creates the Velocity server entry, and wires LoadBalancer metadata (strategy + initial-server flag).
  • /kuvel repair auto-fix enhancement

    • For missing loadbalancer registrations, repair now attempts re-registration first.
    • Only falls back to cleanup (unregisterLoadBalancer) when re-registration fails.
    • Repair summary now reports repaired loadbalancers separately from cleaned loadbalancers.
  • Docs update

    • Updated command reference in README.md and README-JP.md to include new loadbalancer subcommands and expanded repair behavior.
/kuvel register loadbalancer <replicaSetUid> <serverName>
/kuvel unregister loadbalancer <replicaSetUid>

Copilot AI and others added 3 commits April 4, 2026 02:01
@matsu1213 matsu1213 marked this pull request as ready for review April 5, 2026 12:52
@matsu1213 matsu1213 marked this pull request as draft April 5, 2026 13:04
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