fix: bootstrap local app store platform dependencies - #96
Merged
hsluoyz merged 1 commit intoJul 9, 2026
Merged
Conversation
bugkeep
force-pushed
the
fix/app-store-platform-bootstrap-upstream
branch
from
July 9, 2026 00:58
5b7fdb6 to
83e67af
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Problem
A number of App Store charts can reach Helm resource creation but still fail in a local CasOS cluster because the cluster bootstrap path is missing platform dependencies that standard Kubernetes distributions usually provide by default.
Observed symptoms included:
Root Cause
CasOS already has the Kubernetes API/resource shape, but the local runtime layer did not consistently provide DNS and default storage capabilities expected by common Helm charts. This made failures look like chart/network issues even when the underlying gap was in the local cluster substrate.
Fix
kube-system/kube-dnsService is not managed by CasOS.local-pathStorageClass for local PVC workloads.local-pathalready exists, bootstrap only reconciles default-class annotations instead of overwriting mutable UI-edited fields like reclaim policy or expansion.Review Follow-ups Addressed
master, including the StorageClass CRUD work.local-pathStorageClass fields on restart.kube-dnsServices to avoid silently taking over a distribution-provided DNS Service.Validation
go test ./server ./controllersocr review --from origin/master --to HEAD --format json --audience agentmaster.OCR Notes
Final OCR completed successfully. No Critical/High findings remained.
Remaining Medium/Low items are follow-up/product tradeoffs rather than required fixes for this PR:
Scope
This PR targets local App Store platform dependencies. It intentionally does not try to solve chart-specific default values, third-party repository quality, or every production-grade Kubernetes distribution option in one patch.