search

SUSE / suse-best-practices

SUSE Best Practices documentation series
42 stars 74 forks source link

[doc] Issue in "TEST PROCEDURE:" #446

Closed m0rbo closed 2 months ago

m0rbo commented 2 months ago

10.2.2 Test: Crash the primary node on site B (node 2) # EXAMPLE 31: TEST CRASH_PRIMARY_NODE_SITE_B

TEST PROCEDURE::

https://documentation.suse.com/sbp/sap-15/html/SLES4SAP-hana-sr-guide-PerfOpt-15/index.html#id-1.12.7.5.2.4

I think there is a mistake using the word "secondary" in the "TEST PROCEDURE" sentence here: "TEST PROCEDURE: Crash the secondary node by sending a 'fast-reboot' system request."

While the Description says crash the primary... "DESCRIPTION: Simulate a crash of the site B node running the primary SAP HANA database."

I think the Test Procedure sentence is meaning "crash the former secondary node, which is now primary" ?

So, the test procedure is not actually testing of crashing the "secondary" HANA node?

Even the "Expected" section starts with: The cluster detects the failed primary node (node 2) and declares it UNCLEAN and sets the primary node (node 2) to status "partition with quorum".

So, we're really crashing the primary node.

The TEST PROCEDURE line should be "Crash the primary node..." or "Crash the current primary node(02)..."

chabowski commented 2 months ago

@lpinne @fmherschel would you mind having a look and confirm? Thank you!

lpinne commented 2 months ago

Hi, thanks for pointing to this. Indeed it should be "crash the primary". I will update the doc. Regards, Lars

chabowski commented 2 months ago

Closing issue as fixed.