Skip to content

fix: improve native deploy error messages and escape commas in helm values#146

Merged
vigneshrajsb merged 1 commit intomainfrom
fix/native-deploy-error-messages
Mar 25, 2026
Merged

fix: improve native deploy error messages and escape commas in helm values#146
vigneshrajsb merged 1 commit intomainfrom
fix/native-deploy-error-messages

Conversation

@vigneshrajsb
Copy link
Contributor

Summary

  • Human-readable statusMessage: Replace raw log dumps in deploys.statusMessage with short, actionable messages. Non-timeout failures point to Console > Deploy tab; timeout failures point to Console > Pods.
  • Comma escaping: Escape commas in helm --set values (\,) to fix parsing failures when env vars contain comma-separated URLs (e.g. AUTH_JWK_URI).
  • Timeout clarity: When helm upgrade times out due to unhealthy pods, surface a clear message instead of the generic "timed out waiting for the condition".

Test plan

  • Lint passes
  • Tests pass
  • Deploy a service with comma-separated env var values and verify helm upgrade succeeds
  • Trigger a deploy timeout and verify statusMessage shows pod health guidance
  • Trigger a non-timeout deploy failure and verify statusMessage points to Deploy tab

…alues

- Replace verbose log dumps in deploys.statusMessage with short, actionable
  messages directing users to Console > Deploy tab or Console > Pods
- Escape commas in helm --set values to prevent parsing failures when env
  vars contain comma-separated URLs
- Add specific timeout messaging when helm upgrade fails due to unhealthy pods
@vigneshrajsb vigneshrajsb requested a review from a team as a code owner March 25, 2026 19:07
@vigneshrajsb vigneshrajsb merged commit cbac8cd into main Mar 25, 2026
1 check passed
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.

1 participant