community.sap_operations icon indicating copy to clipboard operation
community.sap_operations copied to clipboard

rewrite sap_control role to use sapcontrol module out of sap_libs

Open rainerleber opened this issue 3 years ago • 2 comments

rewrite sap_control role to use sapcontrol module out of sap_libs

rainerleber avatar May 17 '22 15:05 rainerleber

Hello, Rainer This is another topic, where I do not understand the intention. What reasons do you see to use sapcontrol module out of sap_libs if sapcontrol can be called as command with ansible. And it will be supported, because this is simple command call. I would be very grateful if you could explain this design decision.

kksat avatar May 17 '22 15:05 kksat

It's also an other purpose. Sapcontrol calls with command module will return strings and it's hard to parse the return value. There is already logic wrapped around sapcontrol: https://github.com/sap-linuxlab/community.sap_operations/tree/main/roles/sap_control

To reduce the logic and extend it. It could be an effort to integrate it.

rainerleber avatar May 17 '22 15:05 rainerleber