Skip to content

CLP-897 Split build from analysis - #587

Closed
guillaume-dequenne wants to merge 1 commit into
masterfrom
CLP-897
Closed

guillaume-dequenne wants to merge 1 commit into
masterfrom
CLP-897

Conversation

@guillaume-dequenne

Copy link
Copy Markdown
Contributor

Part of CLP-897

Summary

  • Split the Maven build/deploy producer from the NEXT analysis consumer.
  • Upload the exact Maven target outputs as a three-day artifact and restore them for the scan.
  • Reuse the same build number and Maven scan configuration so the analysis remains equivalent.
  • Keep promotion and releasability gated by the scan.

Dependency

Validation

  • Parsed the updated workflow as YAML.
  • Ran actionlint, accounting for existing Sonar-specific workflow extensions.
  • Ran git diff --check.

@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

CLP-897

@sonarqube-next

Copy link
Copy Markdown

@gitar-bot

gitar-bot Bot commented Oct 13, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

🟡 Medium risk · Separates Maven deployment and scanning through artifact transfer and new job gating.

Splits the Maven build/deploy producer from the NEXT analysis consumer, uploading target outputs as a three-day artifact and restoring them for the scan. Reuse of the same build number and Maven configuration keeps analysis equivalent while gating promotion and releasability by the scan. No issues found.

Verification

Functional validation No results

Rules No rules evaluated

Auto-approval Not enabled

Set up

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Counting what did not apply, without listing it.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant