Skip to content

feat(workflow): resolve Helm service variables - #4960

Open
Cynthia-0203 wants to merge 5 commits into
koderover:mainfrom
Cynthia-0203:feat/helm-service-variable-override-kvs
Open

feat(workflow): resolve Helm service variables#4960
Cynthia-0203 wants to merge 5 commits into
koderover:mainfrom
Cynthia-0203:feat/helm-service-variable-override-kvs

Conversation

@Cynthia-0203

@Cynthia-0203 Cynthia-0203 commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

What this PR does / Why we need it:

Enable Helm service variables in container deployment tasks to reference workflow parameters.

Resolved values are written to OverrideKVs and processed by the existing Helm merge logic, ensuring each deployment uses the latest runtime values while preserving historical behavior.

What is changed and how it works?

  • Added OverrideKVs support to Helm deployment task specs and previews.
  • Merged service_variable_config references into Helm task KV values during task creation.
  • Resolved references from the latest workflow GlobalContext at deployment time.
  • Failed the deployment when a referenced variable cannot be resolved.

Does this PR introduce a user-facing change?

  • API change
  • database schema change
  • upgrade assistant change
  • change in non-functional attributes such as efficiency or availability
  • fix of a previous issue

This change is Reviewable

…ntime

Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.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.

1 participant