fix: point sentinel references at sso.gauchoracing.com - #220
Merged
Conversation
gauchoracing
approved these changes
Aug 5, 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.
sso.gauchoracing.com(dashboard/src/consts/config.tsx)dashboard/src/components/Header.tsx)issclaim inauth(SentinelIssuer) andquery(SENTINEL_ISSUER)build.yml, aggregating the six service image builds into one checkWhy the sentinel change
sentinel-v5.gauchoracing.comno longer resolves —digreturns nothing for it. Login on foundry redirects tohttps://sentinel-v5.gauchoracing.com/oauth/authorize?...and dead-ends there.The live discovery document at
https://sso.gauchoracing.com/.well-known/openid-configurationreports:Sentinel's own source agrees —
core/config/verify.goandoauth/config/verify.goboth defaultIssuertohttps://sso.gauchoracing.com.The two issuer constants matter as much as the redirect. They are the expected
issclaim, not URLs that get fetched, so fixing only the authorize URL would have gotten past login and then rejected every token as issuer-mismatched.auth/config/config.goalready carried the comment "Must byte-match SENTINEL_URL" — andSENTINEL_URLishttps://sso.gauchoracing.comin the manifests, so the invariant was documented but not enforced.Deploy workflow
KUSTOMIZATIONpointed atkubernetes/manifests/mapache/kustomization.yaml. That path was valid at5948094but moved when the infrastructure repo split intogr-prod/gr-foundry, and was never updated — soyqhas been reading a nonexistent file and failing the step on every release since. Now targetskubernetes/gr-foundry/manifests/mapache/kustomization.yaml.Also removed a duplicated path: the variable now holds the repo-relative path, prefixed with
infra/for filesystem access and used bare forgit -C infra. The hardcoded path in the diff step is gone.Verified by running the awk bump against the real gr-foundry kustomization: exactly the six
SERVICESimages change, withforeman(2.0.2) andkerbecs(3.2.0) untouched.build.yml
Mirrors Sentinel's aggregator so branch protection has a single required status instead of twelve matrix checks. Two deliberate differences from Sentinel's version:
mapache-go/mapache-pytrigger onpush: branches: [main]with path filters, so they never run for a feature branch push — waiting on them would hang until timeout and fail every PR.publish.ymlis release-only, same problem.v*rather than**, matching the service workflows. Library tags (mapache-py/*,mapache-go/*) don't trigger an image build, so**would wait on runs that never start.To enable: add Build as a required status check on
main.Note
auth,queryanddashboardall changed, so this needs a full release rather than a dashboard-only one. The dashboard in particular must be rebuilt, not just redeployed — Vite substitutesimport.meta.env.VITE_*at build time and bakes the literal into the bundle.