-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
sap_swpm: Add option to enable observer mode in sap_swpm role #709
Comments
@berndfinger This suggestion looks feasible albeit being very much edge case, intended only for testing and engineering purpose. |
Yes. Primary reason would be troubleshooting rather than having to re-run the SWPM manually to find out why it's not happy. Secondary reason is segregation of duties as not all customers are able/willing to provide SAP Technical people appropriate access due to policy or regulatory reasons. |
@rob0d As the primary origin author, I am unsure if we can implement this with consistent success. In previous iterations of this Ansible Role, The Ansible Role at the moment has to use background job / nohup, and a process watch using In either case, to allow Observer Mode would break because the sapinst process would be left running indefinitely and therefore the Ansible Task would run indefinitely. The only option would be to change the process watch, to a file watch. When sapinst exits it will dump a HOWEVER - I would not confidently state that this file's existance is proof of return code 0 exit. .... more thinking is needed about this before we alter. It's a nice feature to allow, but not at the expensive of a trusted execution. |
The @rob0d request is possible to do with this changes in the line 24 of pre_install:
Plus I didn't try it, but if this causes an execution that needs an interaction then ansible will run indefinitely like @sean-freeman pointed out. |
@rob0d If I understand correctly, your request is about adding the ability to watch the installation process with a GUI. Did you already have a look at
The file is owned by |
@sean-freeman @rob0d I just tested running sap_swpm for product ID
I was using the code which replaces |
Keep in mind that historically, when using SAP SWPM Unattended Mode if you do not specify In summary.... I am NOT ok with the following:
I am OK with the following:
And I insist we test properly to ensure reliability:
With the last comment being:
|
Gents, Thanks for looking at this. I did further tests and can confirm the following:
|
Hi @sean-freeman @Wabri @berndfinger. I have tested this at least 20 times with swpm 1 and 2 (v17.1, 17.4 and 18.0) on PAS, AAS, WebDisp and SMD agents installations. |
Hi,
I would like to suggest to add an option to enable observer mode in SWPM.
Observer mode allows user to watch the installation progress without the ability to change anything. User can, however, access all the log files via browser.
The reasons why to allow this:
I would suggest to create a new variable: sap_swpm_swpm_observer_mode which will default to false.
If set to true the swpm command line parameter SAPINST_START_GUISERVER=false needs to be removed.
This is currently set in "SAP SWPM Pre Install - Set sapinst command" on lines 17-25 in file pre_install.yml.
I haven't tested this when running via Ansible playbook, but I see no reason why it shouldn't work.
For this to be more flexible an additional parameter may be introduced (e.g. sap_swpm_swpm_remote_access_user) which will set another command line parameter SAPINST_REMOTE_ACCESS_USER={{ sap_swpm_swpm_remote_access_user }} which will allow a non-root user login to the web interface.
SAP documentation reference: https://help.sap.com/docs/SLTOOLSET/7f0f5f0bdc954c1f80af6159f5e9fc41/3e7738abf8674ed7a6448d8c04fa5de7.html
The text was updated successfully, but these errors were encountered: