Skip to content

Conversation

@rjrudin
Copy link
Contributor

@rjrudin rjrudin commented Jan 29, 2026

I seem to recall having issues with this when I tried to do it in the Java Client for dynamic stages, so just executing the tests sequentially.

I seem to recall having issues with this when I tried to do it in the Java Client for dynamic stages, so just executing the tests sequentially.
Copilot AI review requested due to automatic review settings January 29, 2026 16:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes parallel execution support from the Jenkins pipeline's regression testing stage, switching to sequential execution of tests across different MarkLogic image tags. The change addresses issues encountered with parallel dynamic stages.

Changes:

  • Removed parallel execution framework for regression tests
  • Simplified the regression stage to execute tests sequentially
  • Added agent configuration at the stage level instead of using dynamic node allocation
Comments suppressed due to low confidence (1)

Jenkinsfile:1

  • The closing braces structure appears incorrect. Lines 194-196 show three consecutive closing braces that don't match the opening structure of the regressions stage. After removing the parallel execution code, there should only be one closing brace for the steps block, one for the script block, and one for the stage.
@Library('shared-libraries') _

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link

Copyright Validation Results
Total: 1 | Passed: 0 | Failed: 0 | Skipped: 1 | at: 2026-01-29 16:12:33 UTC | commit: 797723a

⏭️ Skipped (Excluded) Files

  • Jenkinsfile

✅ All files have valid copyright headers!

@rjrudin rjrudin merged commit a1ea34a into develop Jan 29, 2026
4 checks passed
@rjrudin rjrudin deleted the feature/jenkins branch January 29, 2026 16:42
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.

3 participants