Skip to content

Fix cache key identity for SonarJava - #6267

Merged
guillaume-dequenne merged 1 commit into
masterfrom
CLP-900
Oct 1, 2026
Merged

guillaume-dequenne merged 1 commit into
masterfrom
CLP-900

Conversation

@guillaume-dequenne

@guillaume-dequenne guillaume-dequenne commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Part of CLP-900

Use explicit job and matrix namespaces for the Maven QA caches, and remove the synthetic POM used only to change a cache hash. Disable the overlapping cache in the shared Maven setup action. A restore-only fallback to the existing Build cache seeds each namespace on its first run; new saves remain namespaced.

Validation: PR CI passed. Both Linux ruling QA variants restored the previous Build cache through the fallback and saved distinct new keys. The custom rules job restored an exact hit for its new namespaced key. In the last 30 days, 14 of 134 commits touched a POM; the POM hash only rotates when a POM changes. A repeat run would confirm hits for the newly saved ruling keys.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Fix cache key identity for SonarJava CLP-1119 Fix cache key identity for SonarJava Sep 29, 2026
@hashicorp-vault-sonar-prod

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

Copy link
Copy Markdown
Contributor

CLP-1119

@datadog-sonarsource

This comment has been minimized.

@guillaume-dequenne guillaume-dequenne changed the title CLP-1119 Fix cache key identity for SonarJava Fix cache key identity for SonarJava Sep 29, 2026
@guillaume-dequenne
guillaume-dequenne marked this pull request as ready for review September 30, 2026 13:03

@mary-georgiou mary-georgiou left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gitar-bot

gitar-bot Bot commented Oct 1, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

🟡 Medium risk · CI Maven dependency cache keys now use job-specific namespaces and fallback restores.

Gives Maven QA caches explicit job and matrix namespaces, removing the synthetic POM previously used to change cache hashes. A restore-only fallback to the existing Build cache seeds each namespace on first run while keeping new saves namespaced. No issues found.

Review coverage

🧪 Functional validation 1 of 1 objectives covered

📋 Rules No rules evaluated

🤖 Auto-approval Not enabled · Set up

Implementation Status ✅ 1 of 1 objectives covered
✅ CLP-900 - 1 of 1 objectives covered

This PR covers giving Maven QA caches explicit namespaces in sonar-java.

✅ 1 covered here
  • ✅ Give Maven QA caches explicit namespaces in sonar-java
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

@sonarqube-next

sonarqube-next Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

@guillaume-dequenne
guillaume-dequenne merged commit 6c242ed into master Oct 1, 2026
17 checks passed
@guillaume-dequenne
guillaume-dequenne deleted the CLP-900 branch October 1, 2026 15:51
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.

2 participants