Repository navigation
SC-60373 Pilot WIF authentication in build-poetry - #118
aurelien-poscia-sonarsource wants to merge 1 commit into
Conversation
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
CI failed: Both CI job logs (build-poetry and build workflows) are truncated before any failing step appears; all visible setup, WIF/auth-related, and cache steps succeed, so the actual failure is unidentified and its relation to the PR is unclear.OverviewTwo unique error templates were analyzed from 2 logs (jobs 120499156583 and 120499154126). Neither log contains an identifiable failing step, exit code, or error message; both excerpts end during setup/cleanup, so no change-related failure can be confirmed. FailuresUnidentified failure in truncated job logs (confidence: low)
Summary
Code Review ✅ Approved🔴 High risk · PR-triggered scans now use a WIF-enabled action branch; misconfigured federation could authorize scans under an unintended identity. Switches Verification Rules 3/3 met PRs 1 inspectedFunctional validation No results Rules 3 passed
Broader impact PRs 1 inspectedCross-PR 1 PR inspected 0 open impacts
Auto-approval Not enabledFor more verification details, comment Tip Comment OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
Summary
SC-60373, part of SC-50795.
Pilot of SonarSource/ci-github-actions#363:
build-poetryauthenticates SonarQube Cloud scans (sqc-eu,sqc-us) with the GitHub OIDC token (WIF) instead of thesonarcloudandsonarqube-usVault tokens.Both
build-poetry@masterusages point tofeature/ap/SC-60373-wif-sonarqube-cloud-scans:build.yml:Buildjob (single-platform scan,sonar-platform: sqc-eu, already set onmaster).unified-dogfooding.yml:run-shadow-scans: true(EU and US auth steps). A temporarypull_request:trigger is added so it runs on this PR.Nothing is left on the default
nextplatform for single-platform scans:build-poetryis only used in these two places, and the shadow scan step does not usesonar-platform.Do not merge. Close this PR once the pilot is done.
What to check
SonarSource_sonar-dummy-python-osson both EU and US, with trusted scanning enabled.run-irisand theNextscan still read their tokens from Vault.Test plan
Buildworkflow passesUnified Dogfooding scanspasses on this PR (EU and US)🤖 Generated with Claude Code