Skip to content

Improve workflow robustness and error handling - #240

Merged
eduard-tita merged 1 commit into
mainfrom
minor-build-improvement
Sep 15, 2026
Merged

eduard-tita merged 1 commit into
mainfrom
minor-build-improvement

Conversation

@eduard-tita

Copy link
Copy Markdown
Collaborator

Description

This pull request updates the CI/CD pipeline to improve artifact handling, error reporting, and test result collection in both GitHub Actions and Jenkins workflows. The main changes ensure that artifacts are always uploaded, test results are reliably collected, and workflow failures are properly surfaced.

CI workflow improvements:

  • Added if: always() to artifact upload steps in .github/workflows/ci-build.yml and .github/workflows/ci-release.yml to ensure artifacts are uploaded even if previous steps fail.

Test result collection:

  • Moved test result collection logic from the Jenkinsfile into runBuildWorkflow.groovy, ensuring test results are always collected based on whether integration tests are run.

Error handling and reporting:

  • Updated both runBuildWorkflow.groovy and runReleaseWorkflow.groovy to check the workflow run conclusion and fail the pipeline if the workflow did not succeed, providing clearer feedback on failures.

Build process adjustments:

  • Modified the release workflow in .github/workflows/ci-release.yml to skip both unit and integration tests during publishing, speeding up the release process.

Artifact contents:

  • Removed test result XML files from the list of uploaded artifacts in the release workflow, as test results are now handled separately.

Links

JIRA: N/A
Build: https://jenkins.ci.sonatype.dev/job/integrations/job/scan-gradle-plugin/job/feature-snapshots/job/minor-build-improvement/

@eduard-tita
eduard-tita requested a review from a team as a code owner September 15, 2026 17:37
@eduard-tita
eduard-tita merged commit 078c2d4 into main Sep 15, 2026
6 checks passed
@eduard-tita
eduard-tita deleted the minor-build-improvement branch September 15, 2026 17:44
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