Skip to content

CLP-996 Split build from analysis for sonar-xml #588

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
guillaume-dequenne wants to merge 3 commits into
base: master
Choose a base branch
from
Open

CLP-996 Split build from analysis for sonar-xml #588

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
29fefda
CLP-996 Split Maven build from NEXT analysis
guillaume-dequenne Oct 5, 2026
debe6bd
CLP-897 Preserve scan and promotion behavior across split jobs
guillaume-dequenne Oct 9, 2026
de0c9f0
CLP-897 Use the shared build-number claim without reserved env overrides
guillaume-dequenne Oct 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
53 changes: 51 additions & 2 deletions .github/workflows/build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,55 @@ jobs:
artifactory-deployer-role: qa-deployer
use-develocity: true
develocity-url: https://develocity-public.sonar.build/
sonar-platform: none
maven-args: -Pcoverage

- name: Upload plugin build as pipeline artifact
uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
with:
name: build-output-${{ steps.build.outputs.BUILD_NUMBER }}
path: '**/target'
retention-days: 7
overwrite: true

- name: Prepare PVF candidate version
if: ${{ steps.build.outputs.deployed }}
uses: SonarSource/core-languages-tooling-public/prepare-pvf@master
with:
build-version: ${{ steps.build.outputs.project-version }}

analysis:
name: NEXT analysis
needs: build
# Match build-maven's should_scan policy; other refs only build and deploy.
if: >-
github.event_name == 'pull_request' ||
github.ref_name == github.event.repository.default_branch ||
startsWith(github.ref_name, 'branch-') ||
startsWith(github.ref_name, 'feature/long/')
runs-on: github-ubuntu-latest-m
permissions:
id-token: write
contents: read
env:
BUILD_NUMBER: ${{ needs.build.outputs.build-number }}
steps:
- uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
- uses: jdx/mise-action@7a4e45a543138629540c9a1616d08632b893e492 # v5.0.1
with:
version: 2026.10.0
- uses: actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8.0.1
with:
name: build-output-${{ needs.build.outputs.build-number }}
- uses: SonarSource/ci-github-actions/build-maven@v2
with:
deploy: false
skip-build: true
sonar-platform: next
artifactory-reader-role: private-reader
use-develocity: true
develocity-url: https://develocity-public.sonar.build/

build-win:
name: Build Windows
needs:
Expand Down Expand Up @@ -149,9 +191,16 @@ jobs:
promote:
needs:
- build
- analysis
- build-win
- qa
if: ${{ needs.build.outputs.deployed }}
if: >-
!cancelled()
&& needs.build.result == 'success'
&& needs.build-win.result == 'success'
&& needs.qa.result == 'success'
&& (needs.analysis.result == 'success' || needs.analysis.result == 'skipped')
&& (needs.build.outputs.deployed)
runs-on: github-ubuntu-latest-s # Custom GitHub-hosted runner for public repos
name: Promote
permissions:
Expand All @@ -165,7 +214,7 @@ jobs:
promote-pull-request: true

notify-build-failure:
needs: [build, build-win, qa, promote]
needs: [build, analysis, build-win, qa, promote]
if: >-
always()
&& (github.ref_name == github.event.repository.default_branch || startsWith(github.ref, 'refs/heads/branch-') || startsWith(github.ref, 'refs/heads/dogfood-'))
Expand Down
Loading