sap_facts - stopping or restarting sapstartsrv

[domses]

Hi Folks,

i am using the sap_facts module for multiple projects and had recently experienced serveral stopped sapstartsrv service after exeuting it on a big landscape. Looking at the code i saw a function check_sapstartsrv where there is some kind of check & repair of the sapstartsrv service done.

This is not what i expect of a facts collection plugin - at least it should not be done as default behaviour as this could cause problems in many areas (automation, monitoring and so on) and seems to be too aggresive on business critical systems. The check itself is OK, the ansible should just abort then so there we can do RCA.