Skip to content

ci: adding memory limitation to MAPDL command line#3693

Merged
germa89 merged 2 commits into
mainfrom
ci/implement-MAPDL-memory-limitations
Jan 23, 2025
Merged

ci: adding memory limitation to MAPDL command line#3693
germa89 merged 2 commits into
mainfrom
ci/implement-MAPDL-memory-limitations

Conversation

@germa89

@germa89 germa89 commented Jan 23, 2025

Copy link
Copy Markdown
Collaborator

Description

This should help to improve MAPDL container estability.

Issue linked

Close #3378

Checklist

@germa89
germa89 requested a review from a team as a code owner January 23, 2025 12:47
@ansys-reviewer-bot

Copy link
Copy Markdown
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@germa89 germa89 self-assigned this Jan 23, 2025
@github-actions github-actions Bot added the CI/CD Related with CICD, Github Actions, etc label Jan 23, 2025
@codecov

codecov Bot commented Jan 23, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.09%. Comparing base (e6221a6) to head (10a463b).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3693      +/-   ##
==========================================
- Coverage   87.14%   87.09%   -0.06%     
==========================================
  Files         187      187              
  Lines       14697    14697              
==========================================
- Hits        12808    12800       -8     
- Misses       1889     1897       +8     

@germa89

germa89 commented Jan 23, 2025

Copy link
Copy Markdown
Collaborator Author

I'm happy enough with the CICD.

@pyansys-ci-bot LGTM.

@pyansys-ci-bot pyansys-ci-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Approving this PR because germa89 said so in here 😬

LGTM

@germa89
germa89 merged commit ed13b59 into main Jan 23, 2025
@germa89
germa89 deleted the ci/implement-MAPDL-memory-limitations branch January 23, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Related with CICD, Github Actions, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Set MAPDL memory and CPU limits on CICD

2 participants