Phase 6A: drop PAT shadow on all-citus-unit-tests build-package + migrate update_package_properties to app token#1195
Open
ihalatci wants to merge 1 commit into
Conversation
…rate update_package_properties to app token Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.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.
Phase 6A — drop dead org PAT on
all-citus-unit-testsPart of the migration off the retired org PAT
secrets.GH_TOKENto thecituspackagingappGitHub App token (actions/create-github-app-token@v3, credsvars.GH_APP_ID+secrets.GH_APP_KEY, ownercitusdata). The org PAT is now dead, so the two remaining references on this base branch are fixed together here..github/workflows/build-package.yml— shadow removal onlyRemoved the two top-level env shadows:
Safe: both jobs already mint the app token, export it to
$GITHUB_ENV, and consume the exported value — nothing depends on the top-level shadow. No other change..github/workflows/update_package_properties.yml— full migration (unmigrated anomaly)This workflow was still entirely on the dead PAT. Now fully on the app token:
GH_TOKENenv (keptPRJ_NAME).actions/create-github-app-token@v3(id: app-token) as the first step.token:now uses${{ steps.app-token.outputs.token }}(checkout kept at@v3).GH_TOKEN/GITHUB_TOKENto$GITHUB_ENV(the--gh_token="${GH_TOKEN}"call resolves via this export).v0.8.10->v0.8.36.grep secrets.GH_TOKENnow returns zero matches on both files. Both files YAML-parse clean (PyYAML).Validation
build-package.ymlauto-fired on push: run 28934573255 — success (https://github.com/citusdata/packaging/actions/runs/28934573255). Publish stays gated OFF because branch !=MAIN_BRANCH(all-citus).update_package_properties.ymlisworkflow_dispatch-only, so not dispatched here; its structure is at parity with the already-validatedall-citusversion.⛔ MERGE HELD — DRAFT for review
Do not merge / no auto-merge. Kept as a draft pending review.