Skip to content

feat(account): migrate account-service deployment to Helm chart#6646

Merged
cuisongliu merged 1 commit intolabring:mainfrom
bxy4543:charts/account-service
Feb 2, 2026
Merged

feat(account): migrate account-service deployment to Helm chart#6646
cuisongliu merged 1 commit intolabring:mainfrom
bxy4543:charts/account-service

Conversation

@bxy4543
Copy link
Member

@bxy4543 bxy4543 commented Feb 2, 2026

Summary

Migrate account-service deployment from manifest-based approach to Helm chart infrastructure, aligning with the project's standard deployment pattern.

Key Changes

  • Replace manifests with Helm chart: Complete Helm chart structure with standard templates (deployment, service, configmap)
  • Create entrypoint script: account-service-entrypoint.sh for automated Helm installation/upgrade
  • Update Kubefile: Switch from manifests to Helm chart approach
  • Simplify deployment: Reuse account-controller-manager service account instead of creating separate RBAC
  • Remove unnecessary components:
    • No webhook configuration (service doesn't need admission control)
    • No dedicated RBAC (shares permissions with account-controller)
    • No metrics service
    • No cert-manager certificates
  • Ensure backward compatibility: Use simplified selector labels for seamless migration from existing manifest deployments
  • Comprehensive documentation: Detailed README.md with configuration examples, troubleshooting guide, and usage patterns

Benefits

  • Consistency: Aligns with account-controller deployment pattern
  • Maintainability: Helm chart provides better lifecycle management
  • Flexibility: Easy to customize via HELM_OPTS and values.yaml
  • Migration path: Automatic adoption of existing resources

Test Plan

  • Verify Helm chart templates render correctly with helm template
  • Test entrypoint script execution
  • Confirm backward compatibility with existing deployments
  • Test in staging environment
  • Verify service functionality after deployment

Breaking Changes

None. The change is backward compatible - existing deployments will be automatically adopted by Helm.

🤖 Generated with Claude Code

@bxy4543 bxy4543 requested a review from a team as a code owner February 2, 2026 12:01
@cuisongliu cuisongliu added this to the v5.2 milestone Feb 2, 2026
- Replace manifest-based deployment with Helm chart structure
- Add Helm chart with standard templates (deployment, service, configmap)
- Create account-service-entrypoint.sh for Helm installation
- Update Kubefile to use Helm chart instead of manifests
- Simplify deployment by reusing account-controller-manager service account
- Remove unnecessary components (no webhook, RBAC, metrics, certificates)
- Use compatible selector labels for seamless migration from manifests
- Add comprehensive README.md with configuration examples and troubleshooting guide

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@bxy4543 bxy4543 force-pushed the charts/account-service branch from 22937bc to 5b71f6a Compare February 2, 2026 13:04
@cuisongliu cuisongliu merged commit 0b6e6b4 into labring:main Feb 2, 2026
13 of 14 checks passed
@bxy4543 bxy4543 deleted the charts/account-service branch February 2, 2026 13:16
@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2026

🤖 says: cherry pick action finished successfully 🎉!
See: https://github.com/labring/sealos/actions/runs/21591643109

github-actions bot pushed a commit that referenced this pull request Feb 2, 2026
- Replace manifest-based deployment with Helm chart structure
- Add Helm chart with standard templates (deployment, service, configmap)
- Create account-service-entrypoint.sh for Helm installation
- Update Kubefile to use Helm chart instead of manifests
- Simplify deployment by reusing account-controller-manager service account
- Remove unnecessary components (no webhook, RBAC, metrics, certificates)
- Use compatible selector labels for seamless migration from manifests
- Add comprehensive README.md with configuration examples and troubleshooting guide

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude <noreply@anthropic.com>
cuisongliu pushed a commit that referenced this pull request Feb 2, 2026
feat(account): migrate account-service deployment to Helm chart (#6646)

- Replace manifest-based deployment with Helm chart structure
- Add Helm chart with standard templates (deployment, service, configmap)
- Create account-service-entrypoint.sh for Helm installation
- Update Kubefile to use Helm chart instead of manifests
- Simplify deployment by reusing account-controller-manager service account
- Remove unnecessary components (no webhook, RBAC, metrics, certificates)
- Use compatible selector labels for seamless migration from manifests
- Add comprehensive README.md with configuration examples and troubleshooting guide

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Jiahui <4543bxy@gmail.com>
Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants