MLE-24892 Backing off parallel config for now #1857
Conversation
rjrudin
requested review from
BillFarber,
anu3990 and
stevebio
as code owners
|
Copyright Validation Results ⏭️ Skipped (Excluded) Files
✅ All files have valid copyright headers! |
There was a problem hiding this comment.
Pull Request Overview
This PR removes parallel execution configuration from the Jenkins pipeline and reverts to sequential stage execution. The change is being made to address Jenkins issues while maintaining the dynamic stage generation functionality.
Key changes:
- Moved agent configuration from individual stages to pipeline level
- Removed parallel execution wrapper for regression test stages
- Consolidated cleanup operations into a post-always block
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Jenkinsfile
Outdated
| post { | ||
| always { | ||
| junit '**/build/**/TEST*.xml' | ||
| updateWorkspacePermissions() | ||
| tearDownDocker() | ||
| } | ||
| } |
There was a problem hiding this comment.
The post block is placed at the wrong indentation level. It appears to be inside the 'regressions' stage's script block, but post blocks should be direct children of stage blocks, not nested within script blocks. This should be moved to be a sibling of the 'when' and 'steps' blocks of the 'regressions' stage.
Having issues on Jenkins, just want to get dynamic stages in place for now.
rjrudin
force-pushed
the
feature/parallel-stages
branch
from
bc1db6d to
bbf3b48
Compare
Having issues on Jenkins, just want to get dynamic stages in place for now.