Skip to content

fix(helm): remove legacy migration job and references #723

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vk-playground
Copy link
Contributor

This PR removes the deprecated Helm-based migration step and related configuration, as migrations are now fully automated. Resolves #383.

Changes:

  • Deleted templates/job-migration.yaml
  • Removed migration-specific startupProbe logic from deployment-mcpgateway.yaml
  • Set migration.enabled=false by default in values.yaml
  • Updated values.schema.json with default=false and deprecation note
  • Cleaned migration references from NOTES.txt, README.md, CONTRIBUTING.md, and CHANGELOG.md

Testing:

  • helm lint passes with no errors.
  • helm template mcp-stack . --debug renders with no errors.
  • helm template output contains no migration jobs, scripts, or hooks.
  • Verified no hits for migrat|alembic|helm\.sh/hook in rendered manifests.

…ation.* — closes IBM#383

Signed-off-by: Vicky <90204593+vicky-playground@users.noreply.github.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.

[Bug]: Remove migration step from Helm chart (now automated, no longer needed)
1 participant