fix(deploy): update kustomization path, @gauchoracing attribution, add diff link - #100
Merged
Conversation
…dd diff link Mirrors the three fixes made to the vault repo's deploy workflow in Gaucho-Racing/Vault#12: 1. Update kustomization path from kubernetes/manifests/sentinel/kustomization.yaml to kubernetes/gr-foundry/manifests/sentinel/kustomization.yaml Sentinel now lives on gr-foundry after the migration in Gaucho-Racing/infrastructure#129; the deploy job needs to bump the tag in the new path. 2. Attribute the bump commit to @gauchoracing via team@gauchoracing.com (verified on the account) instead of github-actions[bot]. Same PAT the workflow already uses; consistent audit trail. 3. Add a "Click here to view all changes" link at the bottom of the PR body, pointing at GitHub's compare view for OLD_TAG..NEW_TAG in the sentinel repo. One-click access to the unified diff for reviewers.
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
1 participant
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.
Mirrors the three fixes we made to the vault repo's deploy workflow (Gaucho-Racing/Vault#12).
Changes
Kustomization path — from `kubernetes/manifests/sentinel/kustomization.yaml` to `kubernetes/gr-foundry/manifests/sentinel/kustomization.yaml`. Sentinel now lives on gr-foundry after the migration (feat(gr-foundry): migrate sentinel from gr-prod (Vault-backed config) infrastructure#129); without this change, releases would silently no-op on the next tag.
Commit attribution — sets git config to `GR Admin team@gauchoracing.com` (verified on the @gauchoracing account) instead of `github-actions[bot]`. The PAT the workflow uses already belongs to @gauchoracing; matching the commit author to it makes the PR audit trail consistent.
Unified diff link — adds a `Click here to view all changes →` link at the bottom of the auto-generated infra PR body, pointing at GitHub's compare view for OLD_TAG..NEW_TAG in the sentinel repo. Reviewers get one-click access to the full diff instead of clicking through individual commits.
Test plan