-
Notifications
You must be signed in to change notification settings - Fork 142
fix: update MAPDL image version and remove DPF server activation step in documentation build workflow #4118
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
… in documentation build workflow
|
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
Reviewer's GuideThis PR bumps the MAPDL image version used in the documentation CI workflow and refactors the DPF server activation by removing the standalone activation step, updating environment variables, and enabling inline server startup. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this 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 updates the documentation build workflow to use a newer MAPDL image version and streamlines the DPF server setup process. The changes consolidate DPF server configuration into the main MAPDL startup script instead of using a separate activation step.
- Updates MAPDL image version from v24.2-ubuntu-student to v25.2-ubuntu-cicd
- Removes dedicated DPF server activation step and integrates DPF configuration into the main MAPDL startup process
- Adds DPF port configuration variables to support the integrated setup
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4118 +/- ##
==========================================
+ Coverage 91.30% 91.32% +0.01%
==========================================
Files 189 189
Lines 15649 15650 +1
==========================================
+ Hits 14289 14292 +3
+ Misses 1360 1358 -2 🚀 New features to boost your workflow:
|
|
@pyansys-ci-bot LGTM. |
pyansys-ci-bot
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…tion step in documentation build workflow (#4118)" (#4132) * revert: "fix: update MAPDL image version and remove DPF server activation step in documentation build workflow (#4118)" This reverts commit 502a5e2. * chore: adding changelog file 4132.miscellaneous.md [dependabot-skip] --------- Co-authored-by: pyansys-ci-bot <[email protected]>

Description
Update docker image tag in documentation build workflow.
Issue linked
NA
Checklist
draftif it is not ready to be reviewed yet.feat: adding new MAPDL command)Summary by Sourcery
Update documentation build workflow to use MAPDL v25.2-ubuntu-cicd image and streamline DPF server activation
CI: