Skip to content

Remove dummy license server info from ANSYS#3377

Merged
boegel merged 1 commit intoeasybuilders:developfrom
Flamefire:20240701143255_new_pr_ansys
Jul 3, 2024
Merged

Remove dummy license server info from ANSYS#3377
boegel merged 1 commit intoeasybuilders:developfrom
Flamefire:20240701143255_new_pr_ansys

Conversation

@Flamefire
Copy link
Copy Markdown
Contributor

@Flamefire Flamefire commented Jul 1, 2024

(created using eb --new-pr)

The value passed via -licserverinfo will be written to the installation directory in shared_files/licensing/ansyslmd.ini

It does not make sense to use a dummy value there and might even hurt when ANSYS tries to connect to it and fails.
If not server AND port are specified don't pass -licserverinfo

@Flamefire Flamefire changed the title Remove dummy license server info Remove dummy license server info from ANSYS Jul 1, 2024
@Flamefire
Copy link
Copy Markdown
Contributor Author

Test report by @Flamefire

Overview of tested easyconfigs (in order)

  • SUCCESS ANSYS-2022R1.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
n1336 - Linux RHEL 8.7 (Ootpa), x86_64, Intel(R) Xeon(R) Platinum 8470 (icelake), Python 3.8.13
See https://gist.github.com/Flamefire/01778692ba263e1b86b7bfb6e3d807e8 for a full test report.

@Flamefire
Copy link
Copy Markdown
Contributor Author

Test report by @Flamefire

Overview of tested easyconfigs (in order)

  • SUCCESS ANSYS-2024R1.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
login4 - Linux RHEL 8.7 (Ootpa), x86_64, Intel(R) Xeon(R) Platinum 8470 (icelake), Python 3.8.13
See https://gist.github.com/Flamefire/935e3399254d17db342d46b907070224 for a full test report.

Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel added the bug fix label Jul 3, 2024
@boegel boegel added this to the release after 4.9.2 milestone Jul 3, 2024
@boegel boegel merged commit 4fa3fd6 into easybuilders:develop Jul 3, 2024
@Flamefire Flamefire deleted the 20240701143255_new_pr_ansys branch July 3, 2024 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants