Model Imogen as a shared Olares service#1
Merged
Conversation
The Olares package now follows the shared-app pattern used by official API and ComfyUI examples: one admin-owned backend chart is marked shared, while each member gets a small user-space proxy entrance for the normal Olares endpoint. The manifest exposes a hidden sharedEntrance for app-to-app calls and keeps the user-facing API route for browser or user-scoped access. Constraint: Olares shared applications require the cluster backend and user access surface to be modeled separately. Constraint: Helm is not installed in this environment, so validation used Ruby YAML parsing and kubectl client dry-runs instead of helm template/package. Rejected: Keep the backend in the root chart | that would continue installing the GPU API per user instead of once as a shared service. Confidence: high Scope-risk: narrow Directive: Keep imagegenapiserver as the only chart that deploys the GPU backend; user installs should only proxy to imagegenapiserver-shared. Tested: Ruby YAML parsing for chart values and admin/member OlaresManifest render paths; kubectl create --dry-run=client for rendered user proxy and shared backend resources; git diff --check. Not-tested: helm lint/template/package because helm is unavailable locally; live Olares install.
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.
Summary
Verification
Not tested