fix: Add missing ai-service to resolve ArgoCD sync failure (#11) #34
+69
−15
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔧 Fix: Missing ai-service Dependency
Resolves #11
Problem Statement
The ArgoCD application
2-broken-appswas failing to sync with the error:Root Cause Analysis
Confidence Level: HIGH (95%)
The manifest
Act-3/argocd/apps/broken-aks-store-all-in-one.yamlcontained a critical configuration error where theproduct-servicedeployment references anai-servicethat was not defined in the manifest:Impact:
Changes Made
1. ✅ Added Missing ai-service (Primary Fix)
ai-servicewith proper resource limitsghcr.io/azure-samples/aks-store-demo/ai-service:2.1.0/healthendpoint2. 🔧 Improved Resource Allocations (Secondary Fix)
Fixed unrealistically low memory requests that would cause OOMKilled issues:
Files Changed
Act-3/argocd/apps/broken-aks-store-all-in-one.yaml(+68, -12)Testing & Validation
After merge, validate the fix with:
Expected Outcome
RunningstateHealthyRollback Plan
If issues arise, rollback is simple:
Safety & Impact
default)Generated by: Cluster Doctor Agent v1.0
Analysis Mode: Passive (manifest-based)
Commit: a8fa2cf