Merge pull request #60 from varkoly/SLE-15-SP4

Check if HANA DB is running on the nodes

package/yast2-sap-ha.changes

@@ -1,3 +1,10 @@
+-------------------------------------------------------------------
+Thu Feb  6 08:58:53 UTC 2025 - Peter Varkoly <[email protected]>
+
+- yast sap_ha should check if HDB is running on primary
+  (bsc#1235773) Build in a check if the DB is running on both nodes.
+- 4.4.11
+
 -------------------------------------------------------------------
 Wed Jan 29 10:54:19 UTC 2025 - Peter Varkoly <[email protected]>
 

package/yast2-sap-ha.spec

@@ -17,7 +17,7 @@
 
 
 Name:           yast2-sap-ha
-Version:        4.4.10
+Version:        4.4.11
 Release:        0
 BuildArch:      noarch
 Source0:        %{name}-%{version}.tar.bz2

src/data/sap_ha/help_prereq_hana_su_co.html

@@ -5,6 +5,7 @@ <h3>SAP HANA System Replication Scale Up: Cost-Optimized</h3>
         There are two machines with SAP HANA installed.
         <br/> Note that the machine you run the wizard on becomes the primary node in the cluster.
     </li>
+    <li>Verify that both databases are up and running.</li>
     <li>
         You have created one of the following:
         <ul type="square">

src/data/sap_ha/help_prereq_hana_su_co_azure.html

@@ -18,6 +18,7 @@ <h3>SAP HANA System Replication Scale Up: Cost-Optimized in Microsoft&reg; Azure
         There are two machines with SAP HANA installed.
         <br/> Note that the machine you run the wizard on becomes the primary node in the cluster.
     </li>
+    <li>Verify that both databases are up and running.</li>
     <li>
         The master node SSH public key is authorized for SSH access to the secondary node. This is needed when SSH password access is disabled, which is the default on Microsoft Azure.
     </li>

src/data/sap_ha/help_prereq_hana_su_mt.html

@@ -5,6 +5,7 @@ <h3>SAP HANA System Replication Scale Up: Multi-Tier</h3>
         There are two machines with SAP HANA installed.
         <br> Note that the machine you run the wizard on becomes the primary node in the cluster.
     </li>
+    <li>Verify that both databases are up and running.</li>
     <li>
         You have created one of the following:
         <ul type="square">

src/data/sap_ha/help_prereq_hana_su_mt_azure.html

@@ -18,6 +18,7 @@ <h3>SAP HANA System Replication Scale Up: Multi-Tier in Microsoft&reg; Azure&reg
         There are two machines with SAP HANA installed.
         <br> Note that the machine you run the wizard on becomes the primary node in the cluster.
     </li>
+    <li>Verify that both databases are up and running.</li>
     <li>
         The master node SSH public key is authorized for SSH access to the secondary node. This is needed when SSH password access is disabled, which is the default on Microsoft Azure.
     </li>

src/data/sap_ha/help_prereq_hana_su_po.html

@@ -5,6 +5,7 @@ <h3>SAP HANA System Replication Scale Up: Performance-Optimized</h3>
         There are two machines with SAP HANA installed.
         <br> Note that the machine you run the wizard on becomes the primary node in the cluster.
     </li>
+    <li>Verify that both databases are up and running.</li>
     <li>
         You have created one of the following:
         <ul type="square">

src/data/sap_ha/help_prereq_hana_su_po_azure.html

@@ -18,6 +18,7 @@ <h3>SAP HANA System Replication Scale Up: Performance-Optimized in Microsoft&reg
         There are two machines with SAP HANA installed.
         <br> Note that the machine you run the wizard on becomes the primary node in the cluster.
     </li>
+    <li>Verify that both databases are up and running.</li>
     <li>
         The master node SSH public key is authorized for SSH access to the secondary node. This is needed when SSH password access is disabled, which is the default on Microsoft Azure.
     </li>

src/lib/sap_ha/configuration/hana.rb

@@ -118,6 +118,7 @@ def configured?
       def validate(verbosity = :verbose)
         SemanticChecks.instance.check(verbosity) do |check|
           check.hana_is_installed(@system_id, @global_config.cluster.all_nodes)
+          check.hana_is_running(@system_id, @instance, @global_config.cluster.all_nodes)
           check.ipv4(@virtual_ip, "Virtual IP")
           check.nonneg_integer(@virtual_ip_mask, "Virtual IP mask")
           check.integer_in_range(@virtual_ip_mask, 1, 32, "CIDR mask has to be between 1 and 32.",

src/lib/sap_ha/semantic_checks.rb

@@ -282,11 +282,34 @@ def hana_is_installed(value, nodes)
 	  end
 	  if status != 0
 	    flag = false
-	    message += "No SAP HANA #{value} is installed on #{node}\n"
+	    message += "<br>No SAP HANA #{value} is installed on #{node}\n"
 	  end
 	end
       end
-      report_error(flag, message, 'SID', value)
+      report_error(flag, message, '', '')
+    end
+
+    # Check if a HANA db is running
+    def hana_is_running(system_id, instance_number, nodes)
+      flag = true
+      message = ''
+      my_ips = SapHA::System::Network.ip_addresses
+      procname = "hdb.sap#{system_id.upcase}_HDB#{instance_number}"
+      if @no_test
+        nodes.each do |node|
+          log.debug("node #{node} #{my_ips}")
+          if my_ips.include?(node)
+            status = exec_status("pidof", procname)
+          else
+            status = exec_status("ssh", "-o", "StrictHostKeyChecking=no", node, "pidof", procname)
+          end
+          if status != 0
+            flag = false
+            message += "<br>No SAP HANA #{system_id} is running on #{node}"
+          end
+        end
+      end
+      report_error(flag, message, '', '')
     end
 
     # Check if string is a block device
@@ -368,11 +391,10 @@ def error_string(field_name, explanation, value = nil)
       explanation = explanation[0..-2] if explanation.end_with? "."
       explanation = explanation
       if field_name.empty?
-        "Invalid input: #{explanation}"
+        "Error detected: #{explanation}"
       elsif value.nil? || (value.is_a?(::String) && value.empty?)
         "Invalid entry for #{field_name}: #{explanation}."
       else
-
         "Invalid entry for #{field_name} (\"#{ERB::Util.html_escape(value)}\"): #{explanation}."
       end
     end