Repository navigation
feat(foundry): route /api/org/* to sentinel core - #229
Merged
Merged
Conversation
Sentinel adds an org chart under /org — positions, the composed chart, and the admin CRUD. Without a gateway route those paths fall through to the web upstream, so the SPA answers API calls with index.html and a 200 rather than 404ing. Mirrors the route already added to the compose config in the Sentinel repo; the prod and dev route sets are identical again. kerbecs runs with providers.static.watch enabled, so this applies on the ConfigMap update without restarting the pod. The route is inert until sentinel-core ships the endpoints, so it is safe to land ahead of the version bump.
gauchoracing
approved these changes
Oct 6, 2026
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.
Sentinel adds an org chart under
/org—GET /org/positions, the composedGET /org/chart, and the admin CRUD. Without a gateway route those paths fall through to the web upstream, so the SPA answers API calls withindex.htmland a 200 instead of a 404.core-orgroute for/api/org/*to the kerbecs ConfigMap, placed aftercore-analyticsto match the compose config in the Sentinel repoPairs with Gaucho-Racing/Sentinel#147.
Rollout
providers.static.watch: true(and is pinned to 3.2.0, past the 3.1.0 that added the watcher), so this applies on the ConfigMap update — no pod restart neededsentinel-coreships the endpoints, so it is safe to land ahead of the version bump. Landing it after would mean/api/org/*briefly returns the SPA shell instead of dataTesting
kubectl kustomize kubernetes/gr-foundry/manifests/sentinelrenders cleanly; the rendered ConfigMap carries 18 routes includingcore-org, withwatch: trueintactkerbecs.yamland diffed it against the Sentinel repo's compose config: route names and order now match exactly, and thecore-orgblock is identicalcoreupstream exists and that no earlier route prefix shadows/api/org/*