Skip to content
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

Improve the cluster resource template as per the suse best practises guide #61

Open
Simranpal opened this issue Sep 14, 2020 · 4 comments
Assignees

Comments

@Simranpal
Copy link
Contributor

Peter Schinagl identified some improvements to cluster_resources.j2. and differences to BestPractice SAP_NW740_SLE15 :

  1. "property cib-bootstrap-options" is only available for AWS
    and is different to BestPractice
    rsc_defaults rsc-options: is only available for AWS
    op_defaults op-options: is only available for AWS

  2. Structure of the cluster_resources could be grouped to made it better
    readable e.g. have all IP settings together as a block, similar for the other parts

  3. rsc_socat should be replaced by the azure-lb resource agent

  4. primitive rsc_sap_{{ sid }}_ASCS{{ ascs_instance }} SAPInstance
    op monitor interval=120 vs op monitor interval=11 on BestPractice;
    same for the ERS primitive

  5. ERS group has meta resource stickiness vs none on best BestPractice

https://documentation.suse.com/sbp/all/single-html/SAP_NW740_SLE15_SetupGuide/

@Simranpal
Copy link
Contributor Author

Some points I did not cover :
2. Changing the structure/grouping of cluster resources.
We are grouping it by cloud provider and it seems better approach and will reduce the extra if/else conditions needed otherwise

3.rsc_socat should be replaced by the azure-lb resource agent.
We are already using the azure-lb with socat. I think we don't need to replace it with nc again.

@arbulu89
Copy link
Collaborator

@Simranpal @petersatsuse
-The point 1 is done this by on purpose. Our guide was for AWS. Azure best practices guide doesn't say in any point to use this values: https://docs.microsoft.com/en-us/azure/virtual-machines/workloads/sap/high-availability-guide-suse

We cannot really know what to put there for each of the providers. We just followed the guides

@Simranpal
Copy link
Contributor Author

@diegoakechi cc
Some of the suggestions were made by @petersatsuse as above. However, we set the properties based on the guides and I am not sure if these values need to be set for all providers or for which ones to set.

@petersatsuse
Copy link

I did start a overview table for the various scenarios - i thought i had sent it around.
You are right, some of the suggestions need to be set according to the specific documents for CSP.
I will sent it again...try to have a look, its a start, but i think we need to maintain such a kind of document for the various scenarios

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants