- Notifications
You must be signed in to change notification settings - Fork2.7k
Fix Maven task rebuild when using jacoco for code coverage#21472
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
v-gayatrij commentedNov 20, 2025
/azp run |
| Azure Pipelines successfully started running 3 pipeline(s). |
v-gayatrij commentedNov 25, 2025
/azp run |
| Azure Pipelines successfully started running 3 pipeline(s). |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
v-gayatrij commentedNov 25, 2025
/azp run |
| Azure Pipelines successfully started running 3 pipeline(s). |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
v-gayatrij commentedNov 28, 2025
/azp run |
| Azure Pipelines successfully started running 3 pipeline(s). |
Uh oh!
There was an error while loading.Please reload this page.
v-gayatrij commentedNov 28, 2025
/azp run |
| Azure Pipelines successfully started running 3 pipeline(s). |
458a2d5 intomasterUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Context
Describe the context or motivation for this PR. Include links to any related Azure DevOps Work Items or GitHub issues.
📌 Associated WI:AB#2327840
Task Name
MavenV4
Description
Remove second maven run(during code coverage publishing step) to avoid running maven lifecycle phases twice.
The second adds verify phase when JaCoCo is selected as the code coverage -
azure-pipelines-tasks/Tasks/MavenV4/maventask.ts
Line 460 in54f548f
This is because JaCoCo plugin binds its check goal to the Maven verify phase, which ensures that all tests have run and their coverage data is available -#6458 (comment).
This PR combines below 2 maven runs adding verify phase to the first run:
run to execute maven goals:
azure-pipelines-tasks/Tasks/MavenV4/maventask.ts
Line 268 in54f548f
maven run before publishing code coverage:
azure-pipelines-tasks/Tasks/MavenV4/maventask.ts
Line 462 in54f548f
Risk Assessment (Low / Medium / High)
Low. Changes covered under feature flag
Change Behind Feature Flag (Yes / No)
Yes
Tech Design / Approach
Documentation Changes Required (Yes/No)
Indicate whether related documentation needs to be updated.
Unit Tests Added or Updated (Yes / No)
Indicate whether unit tests were added or modified to reflect these changes.
Additional Testing Performed
canary test pipeline runs:
FF enabled -Pipelines - Run 20251125.8
FF disabled -Pipelines - Run 20251125.7
Logging Added/Updated (Yes/No)
Telemetry Added/Updated (Yes/No)
Rollback Scenario and Process (Yes/No)
Dependency Impact Assessed and Regression Tested (Yes/No)
Checklist