Skip to content

fix: EC3 image handling, preflight, and chart correctness#44

Merged
aa-replicated merged 1 commit intomainfrom
demo/tier4
Apr 17, 2026
Merged

fix: EC3 image handling, preflight, and chart correctness#44
aa-replicated merged 1 commit intomainfrom
demo/tier4

Conversation

@aa-replicated
Copy link
Copy Markdown
Owner

Summary

  • Use EC3 ReplicatedImageRegistry/ReplicatedImageRepository functions for proper online vs airgap image routing
  • Remove broken IsEmbeddedCluster KOTS function that was blocking EC installs
  • Fix preflight: change reachability check from fail→warn so airgap installs aren't blocked
  • Fix db-connection-check fileName path (flat, not subdirectory)
  • Fix support bundle logs fileName (*/gameshelf.log not gameshelf-app/*.log)
  • Fix support bundle SDK log selector (app.kubernetes.io/name=replicated)
  • Clear postgresql volumePermissions.image.registry for EC installs
  • Remove integrationLicenseID from EC values (SDK gets license via proper injection)
  • Fix PR workflow image tag mismatch (push versioned tag alongside pr-N)
  • Align replicated-actions versions between release.yml and pr.yml

Test plan

  • Fresh airgap EC install completes without image pull errors
  • Online EC install routes images through proxy correctly
  • Direct Helm install still works via imageProxy fallback
  • Preflights pass on airgap (reachability shows as warn, not fail)
  • Support bundle DB error analyzer finds logs correctly

🤖 Generated with Claude Code

- helmchart.yaml: use ReplicatedImageRegistry/Repository for proper
  online vs airgap image routing; remove broken IsEmbeddedCluster
  function and preflight.excludeReachability block
- deployment.yaml: support image.registry/repository split (EC path)
  with imageProxy fallback for direct Helm installs
- values.yaml: add image.registry default, remove excludeReachability
- preflight.yaml: remove excludeReachability excludes, change
  reachability fail->warn so airgap installs are not blocked

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@aa-replicated aa-replicated merged commit fa1b6df into main Apr 17, 2026
2 checks 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