Skip to content

Conversation

@spbolton
Copy link
Contributor

Description

Fix ALB healthcheck template parsing errors by replacing direct nested value access with dig function for safe value retrieval. This provides backwards compatibility with partial ALB override configurations and resolves nil pointer evaluation errors in ingress-dotcms-cluster.yaml template.

Key Changes:

  • Replace .Values.ingress.alb.healthcheck.* with dig function calls
  • Replace .Values.ingress.alb.targetGroup.* with dig function calls
  • Add sensible defaults for all healthcheck and targetGroup parameters
  • Remove duplicate configuration from environments.prod section
  • Maintain full backwards compatibility with existing override files

Testing:

  • Validated template parsing with actual override files from staff-sandbox deployment
  • Confirmed ALB annotations generate correctly with proper defaults
  • Verified StatefulSet, Service, and Ingress resources render without errors

Changes

  • [List the main changes made]

Testing

  • [Describe testing approach]

Closes #61

Issue: Update health checks for consistent ALB and Statef

@spbolton spbolton force-pushed the issue-61-update-health-checks-for-consistent-alb-and branch from e3a66a5 to 34ef8c3 Compare August 27, 2025 18:22
@spbolton spbolton changed the title fix(helm): Update health checks for consistent ALB and Statef (#61) fix(helm): resolve ALB healthcheck template parsing with dig function (#61) Aug 27, 2025
@spbolton spbolton merged commit 6ac563c into main Aug 27, 2025
3 checks passed
@spbolton spbolton deleted the issue-61-update-health-checks-for-consistent-alb-and branch August 27, 2025 18:54
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.

Update health checks for consistent ALB and StatefulSet use with new management port

2 participants