-
Notifications
You must be signed in to change notification settings - Fork 123
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
Fixing CICD: Adding missing libgomp1
dependency
#2514
Conversation
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
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.
I believe MAPDL instance is not correctly launched... License server issues?
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2514 +/- ##
==========================================
- Coverage 83.83% 83.81% -0.03%
==========================================
Files 44 44
Lines 8685 8685
==========================================
- Hits 7281 7279 -2
- Misses 1404 1406 +2 |
This reverts commit c4cdd05.
For some reason, MAPDL wasn't starting in ubuntu based containers showing the following error:
It was fixed by installing - name: "Installing missing package"
run: |
sudo apt-get update
sudo apt-get install -y libgomp1 Debugging was made by adding shell commands to the YML file: - name: "Test MAPDL launching"
run: |
/ansys_inc/v222/ansys/bin/ansys222 -grpc Pinging @clatapie for visibility |
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.
Thanks @germa89 for having found the issue.
By the way, it is very strange that suddenly we need another linux lib. Because the container hasn't been modified in ages (almost a year?), and we do not install any dependency in it (minimal setup). The container supposedly includes everything needed to run MAPDL (OS, dependencies, MAPDL, etc). So I cannot really understand what changed for now this library to be required. |
It seems that CICD is failing, not being able to run succesfully even a single test.
Fixed. See #2514 (comment)