Closed tylerschlachter closed 1 year ago
Thanks for the PR Tyler. I have some comments, let me know what you think.
Thanks for taking the time to review Ian, I appreciate it.
I pulled a new lmutil output into the tests/data folder and built the new output into the tests as well.
Removed license manager version number hardcoding from read_licenses.parse_server_info.
Wrote a slightly more robust parse_server_info function which allows for different license manager versions. The previous code would cause all read status to be 'DOWN' if the license manager version was not 11.16.2.
Also "(MASTER)" may or may not be present in the lmutil output (it is not present in the environment I am running).
Passes the current unit test.