Skip to content

Commit 0a6d4e7

Browse files
dmcilvaneyCopilot
andcommitted
ci(prcheck): split pipeline script into client + thin runner
Refactor the prcheck pipeline script in preparation for a second runner that submits package builds. No behavior change. * Move .github/workflows/scripts/control-tower-prcheck/ to .github/workflows/scripts/control-tower/. * Extract HTTP/auth/polling helpers from run_control_tower_prcheck.py into a shared client.py module (post_scenario, get_job_status, poll_until_terminal, etc.). * Rename the runner to run_prcheck.py and reduce it to a thin CLI on top of client.py. * Update the sources-upload stages template to point at the new script paths. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
1 parent 338885b commit 0a6d4e7

5 files changed

Lines changed: 261 additions & 215 deletions

File tree

.github/workflows/ado/sources-upload.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
# Name: "ControlTower-PRCheck"
2424
# Required variables:
2525
# - ApiAudience : Entra ID audience URI for the Control Tower app
26-
# - ApiBaseUrl : Base URL of the Control Tower service
26+
# - ApiBaseDirectUrl : Direct base URL of the Control Tower APIM endpoint (bypasses Azure Front Door)
2727
# - AzldevCommit : Commit hash for azldev (go install ...@<commit>)
2828

2929
# Trigger controlled by ADO branch policy — not YAML triggers.

.github/workflows/ado/templates/sources-upload-stages.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ stages:
111111
echo "##[endgroup]"
112112
113113
echo "##[group]Python dependencies"
114-
pip install -r .github/workflows/scripts/control-tower-prcheck/requirements.txt
114+
pip install -r .github/workflows/scripts/control-tower/requirements.txt
115115
echo "##[endgroup]"
116116
displayName: "Install dependencies"
117117
env:
@@ -200,7 +200,7 @@ stages:
200200
inlineScript: |
201201
set -euo pipefail
202202
203-
python3 .github/workflows/scripts/control-tower-prcheck/run_control_tower_prcheck.py \
203+
python3 .github/workflows/scripts/control-tower/run_prcheck.py \
204204
--api-audience "$API_AUDIENCE" \
205205
--api-base-url "$API_BASE_URL" \
206206
--build-reason "$BUILD_REASON" \
@@ -209,7 +209,7 @@ stages:
209209
--repo-uri "$UPSTREAM_REPO_URL"
210210
env:
211211
API_AUDIENCE: $(ApiAudience)
212-
API_BASE_URL: $(ApiBaseUrl)
212+
API_BASE_URL: $(ApiBaseDirectUrl)
213213
BUILD_REASON: $(Build.Reason)
214214
COMPONENTS: $(components)
215215
SOURCE_COMMIT: $(sourceCommit)

0 commit comments

Comments
 (0)