chore: drop deploy/ directory and deployment-specific docs#194
Merged
Conversation
The repo previously shipped three deployment surfaces under deploy/: - deploy/helm/locus-agent — generic Helm chart for an AgentServer. - deploy/locus-workbench — Always-Free OKE workbench stack (terraform + helm + scripts). - deploy/oci-functions — OCI Functions example app. None of this is part of the locus-sdk wheel (hatch builds from src/ only), so removal does not change what consumers install. The deployment surfaces drifted from the SDK's scope and there is no plan to host the OKE workbench as a public offering. Removed: - deploy/ (whole tree, 36 files). - docs/how-to/deploy-workbench-free-tier.md — orphan how-to that walked through deploy/locus-workbench/. - tests/unit/test_deploy_helm_chart.py — unit-tested the deleted Helm chart's templates and the (still-present) root Dockerfile; Dockerfile coverage is left as integration concern. Edited: - Dockerfile — drop the comment referencing the deleted example. - mkdocs.yml — drop the workbench-free-tier nav entry. - README.md — replace the deploy/helm/locus-agent badge with generic "drop the Dockerfile into your own pipeline" wording; point the deploy guide link at the canonical how-to URL. - docs/how-to/deploy.md — replace the bundled-Helm-chart subsection with chart-free OKE intro; keep raw-YAML reference + Functions / Container Instances / Compute sections (all transferable). Signed-off-by: Federico Kamelhar <federico.kamelhar@oracle.com>
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
Remove the three deployment surfaces previously shipped under
deploy/. None of this is part of thelocus-sdkwheel — hatch builds fromsrc/only — so this is purely a repo-surface change and does not affect whatpip install locus-sdkproduces.Deleted directories:
deploy/helm/locus-agent— generic Helm chart for anAgentServer.deploy/locus-workbench— Always-Free OKE workbench stack (Terraform + Helm + scripts). No plan to host as a public offering.deploy/oci-functions— OCI Functions example app.Orphan files also deleted:
docs/how-to/deploy-workbench-free-tier.md— pure walkthrough ofdeploy/locus-workbench/.tests/unit/test_deploy_helm_chart.py— exercised the deleted Helm chart's template files (and the still-present rootDockerfile, which is fine as an integration concern).Edited (reference cleanup):
Dockerfile— drop the comment line pointing at the deleteddeploy/oci-functions/app.pyexample.mkdocs.yml— drop theDeploy workbench on Always-Free OKEnav entry.README.md— replace thedeploy/helm/locus-agentlink with chart-free wording; point theDeploy guidelink at the canonicalhow-to/deploy/URL.docs/how-to/deploy.md— in the## OKE — Kubernetes for productionsection, replace the bundled-Helm-chart subsection with a chart-free intro. Keep the raw-YAML deployment example and the Functions / Container Instances / Compute / Auth / Sessions sections (all transferable).Test plan
grepfordeploy/(locus-workbench|helm|oci-functions)across the repo returns empty.grepfordeploy-workbench-free-tierreturns empty.