Skip to content

Conversation

yxxhero
Copy link
Collaborator

@yxxhero yxxhero commented Jun 5, 2025

This pull request refactors the extractManifestFromHelmUpgradeDryRunOutput function in cmd/helm3.go to improve its handling of edge cases. Specifically, it ensures that the function properly checks for the presence of "HOOKS" and "MANIFEST" sections before attempting to extract them.

Enhancements to edge case handling:

  • cmd/helm3.go: Added conditional checks to verify the existence of "HOOKS" and "MANIFEST" sections before extracting their contents, preventing potential issues with invalid input.

Signed-off-by: yxxhero <aiopsclub@163.com>
@yxxhero
Copy link
Collaborator Author

yxxhero commented Jun 5, 2025

@mumoshu WDYT?

@yxxhero yxxhero merged commit 3c4195f into master Jun 8, 2025
16 checks passed
@yxxhero yxxhero deleted the 795-helm_diff_use_upgrade_dry_runtrue-crashes-helm-when-no-deployment-exists-in-target-namespace branch September 6, 2025 23:01
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.

HELM_DIFF_USE_UPGRADE_DRY_RUN=true crashes helm when no deployment exists in target namespace
1 participant