In the current structure it sets slcomp/alter respecting the module parameter, but it never compares the slcomp to the current state. Result is that it force-sets the state regardless of the current config in the database.
The fix would be to compare supp_logcheck[0][0] to YES/NO in the if/else supplemental_logging == True part to set/execute the alter sql. And removing the slcomp stuff everywhere.
In the current structure it sets slcomp/alter respecting the module parameter, but it never compares the slcomp to the current state. Result is that it force-sets the state regardless of the current config in the database.
The fix would be to compare supp_logcheck[0][0] to YES/NO in the if/else supplemental_logging == True part to set/execute the alter sql. And removing the slcomp stuff everywhere.
Sorry for not writing a patch.