Skip to content

Enabling pre-submit by addressing the comments received from DevOps team #119

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
sandeeplocharla wants to merge 19 commits into
base: main
Choose a base branch
from
+2,369 −208
Open

Enabling pre-submit by addressing the comments received from DevOps team #119

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
1c4f49a
Addressed the comments received from Devops team
sandeeplocharla Oct 13, 2026
30a0c7a
Check CI scripts out of the PR branch for the presubmit pilot.
sandeeplocharla Oct 13, 2026
961596c
Look up the indexed pull request commit without opening the raw build.
sandeeplocharla Oct 13, 2026
b8bbb2f
Read the indexed pull request commit from the multibranch job.
sandeeplocharla Oct 13, 2026
e6bec6f
Use the GitHub pull request head when Jenkins exposes no indexed SHA.
sandeeplocharla Oct 13, 2026
5913e66
Fix multibranch source identity and document the PR-119 pilot setup.
sandeeplocharla Nov 4, 2026
4c85cd0
Retry failing unit tests twice in the presubmit Maven build.
sandeeplocharla Nov 4, 2026
07a8f12
Include the trusted ONTAP prerequisite checker in the CI sparse check…
sandeeplocharla Nov 4, 2026
53afe82
Show the exception on the failed test's HTML log page and in the mail.
sandeeplocharla Nov 5, 2026
a382d57
Make the ONTAP VM tests tolerate a briefly unreachable virtual router.
sandeeplocharla Nov 5, 2026
0580553
Show presubmit times in IST in the mail and report.
sandeeplocharla Nov 5, 2026
6c05462
Use the shared _start_vm in the ONTAP VM volume tests.
sandeeplocharla Nov 5, 2026
ec0807b
Delete failed VM snapshots after each ONTAP VM volume test.
sandeeplocharla Nov 5, 2026
74d3e97
Load the trusted CI scripts from main instead of the pilot branch.
sandeeplocharla Nov 6, 2026
c6d1dfe
Address review comments on the multibranch presubmit.
sandeeplocharla Nov 12, 2026
2cceb9a
Make the presubmit Check reliable enough to be required.
sandeeplocharla Nov 12, 2026
218232a
Report unit-test coverage of the ONTAP plugin from the main build.
sandeeplocharla Nov 12, 2026
10a31cc
Collect functional-test coverage of the ONTAP plugin from the lab VM.
sandeeplocharla Nov 12, 2026
b5f778b
Publish ONTAP plugin coverage in the Check and as one PR comment.
sandeeplocharla Nov 12, 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
38 changes: 38 additions & 0 deletions plugins/storage/volume/ontap/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -173,4 +173,42 @@
</plugin>
</plugins>
</build>
<profiles>
<profile>
<!-- Unit-test coverage of this plugin, produced by the same build that runs
the tests: mvn -Dontap.coverage=true install. The report is written to
target/site/jacoco/jacoco.xml. The surefire run is bound to the
integration-test phase, so the report is bound to verify. -->
<id>ontap-coverage</id>
<activation>
<property>
<name>ontap.coverage</name>
</property>
</activation>
<build>
<plugins>
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<version>${cs.jacoco-plugin.version}</version>
<executions>
<execution>
<id>ontap-coverage-agent</id>
<goals>
<goal>prepare-agent</goal>
</goals>
</execution>
<execution>
<id>ontap-coverage-report</id>
<phase>verify</phase>
<goals>
<goal>report</goal>
</goals>
</execution>
</executions>
</plugin>
</plugins>
</build>
</profile>
</profiles>
</project>
Loading
Loading