Correct swapped input/output and add error handling

[Virsacer]

Hi,
maxInputRate and maxOutputRate can be different so they should be used accordingly.
Also added some checks to prevent errors.

[codeautopilot]

PR summary

This Pull Request addresses two main issues in the InterfaceSubsystem.pm file of the CheckNwcHealth plugin:

1. Corrects the usage of maxInputRate and maxOutputRate to ensure they are applied correctly for input and output rates respectively.
2. Adds error handling to prevent potential errors when these rates are not defined.

The changes ensure that the correct maximum rates are used for calculations and that the code gracefully handles cases where these rates might be undefined, improving the robustness and accuracy of the plugin.

Suggestion

Consider adding unit tests to verify the new error handling logic and ensure that the correct rates are used in various scenarios. This will help in maintaining the integrity of the changes and prevent future regressions.

Disclaimer: This comment was entirely generated using AI. Be aware that the information provided may be incorrect.

Current plan usage: 75.86%

Have feedback or need help?
Discord
Documentation
[email protected]