sap_swpm: Improve SUM handling behaviour for slower systems

[rob0d]

Updated task names to make it clearer that it will look for SPAU or EXIT step.
Changed the time we are waiting for SPAU or EXIT phases to happen to 240*5 minutes = 20 hours

Improve SUM task names for better readability as discussed in #1139

[berndfinger]

LGTM

[marcelmamula]

@rob0d Where does this PR fix issue 1139?

This PR only updates task names and hardcodes longer timeout. It does not change any logic for SPAU steps.

[rob0d]

@rob0d Where does this PR fix issue 1139?

This PR only updates task names and hardcodes longer timeout. It does not change any logic for SPAU steps.

There is no issue with any logic for SPAU or any other steps.

[marcelmamula]

@rob0d Then it would mean "Fixes issue #1139" is incorrect, because it did not fix anything.

Correct sentence should be: "Improve SUM task names for better readability as discussed in #1139".

This is not meant to be pedantic remark, but instead attempt to ensure that other issue is not closed because this PR seemingly fixed it.

[marcelmamula]

LGTM

[marcelmamula]

@berndfinger @rob0d I have removed LGTM reviews because there was commit after reviews were done.

Plus this name is not very good sap_swpm_swpm_env_vars.

[rob0d]

@berndfinger @rob0d I have removed LGTM reviews because there was commit after reviews were done.

Plus this name is not very good sap_swpm_swpm_env_vars.

Oops. That was a mistake. It should have gone to a new PR. Fixing it.
What name would you like.

[marcelmamula]

@rob0d Name would be simple sap_swpm_env_var_dict or sap_swpm_env_vars_dict, but in new PR as you said.

[rob0d]

OK. I've now removed the last commit.

[marcelmamula]

LGTM

[sean-freeman]

@rob0d As you say, need an Issue/PR to discuss child Shell Env Var injection use cases and subsequent Ansible code changes 🙂 No worries on the accidental commit, thanks for the quick resolution