From nobody Fri May 3 22:39:02 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass(p=reject dis=none) header.from=citrix.com ARC-Seal: i=1; a=rsa-sha256; t=1678468119; cv=none; d=zohomail.com; s=zohoarc; b=fuUQQshBA/kdXz41rMFnhdNiCDyeSgFuHHHkoAhkOaHnGCm+OilfSNyUHCKyH2M2s7eIA7CxkIlndusEx/oVfYwACUyVtX//38gFiCJHFVj+Bcq+z9856tBx7q7qxpL2ypnV4BbTC6hj7IJma8z3lzYxwX21Yg0oYr5fBiEUBUg= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1678468119; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To; bh=byYdNWxL4Rm8ODICHHmSDN4LHWePQzEpJeHETWUvybc=; b=mOy6TbrUEMgFKOwzs0UOVG2c+T2zJm27K7gUljBkhMB4D+Pp85Gxn0G4a43Mfle29qm2aegZ3AKtLHpvboTsls1EhFiV5+jP/LHvDARDFYSsphaKlnzo4AmabTkb7GCQhO/Ag3ZvMOs/VY2eN/TaD6dKv78wJFftVR35LgtMKP0= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass header.from= (p=reject dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1678468119334959.3662127378258; Fri, 10 Mar 2023 09:08:39 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.508537.783271 (Exim 4.92) (envelope-from ) id 1pagDp-000234-FG; Fri, 10 Mar 2023 17:08:01 +0000 Received: by outflank-mailman (output) from mailman id 508537.783271; Fri, 10 Mar 2023 17:08:01 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1pagDp-00022x-Bt; Fri, 10 Mar 2023 17:08:01 +0000 Received: by outflank-mailman (input) for mailman id 508537; Fri, 10 Mar 2023 17:08:00 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1pagDo-00022r-K1 for xen-devel@lists.xenproject.org; Fri, 10 Mar 2023 17:08:00 +0000 Received: from esa1.hc3370-68.iphmx.com (esa1.hc3370-68.iphmx.com [216.71.145.142]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id 17887a98-bf66-11ed-87f5-c1b5be75604c; Fri, 10 Mar 2023 18:07:57 +0100 (CET) X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 17887a98-bf66-11ed-87f5-c1b5be75604c DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=citrix.com; s=securemail; t=1678468077; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=qJUHxRCsNx0u9euoS9TNdoV5MXh2jlAdF3MSmsTTQNg=; b=VE5Spo3DgUk58XmcBU9cit8ok45wKZyc4U+CMDn8GHp3hbTF3E4yrEqb 72VPuanGLxStClWLS1ElJKaebpqVXp8m99JZm3IpejN0iNj5w+Em0EAJU fHEIN837x5INa4bvTmE4+mttgQB2GrCUEaBTdeK8+PiINWFgkVME0NDr3 s=; Authentication-Results: esa1.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none X-SBRS: 4.0 X-MesageID: 100764149 X-Ironport-Server: esa1.hc3370-68.iphmx.com X-Remote-IP: 162.221.156.123 X-Policy: $RELAYED IronPort-Data: A9a23:F0KSMa+I2iEAdop7pGysDrUDjH6TJUtcMsCJ2f8bNWPcYEJGY0x3y jFJCmCBM/mKNGrweNslO4y+oEkDvpbQzIIyTgQ6pX88E34SpcT7XtnIdU2Y0wF+jCHgZBk+s 5hBMImowOQcFCK0SsKFa+C5xZVE/fjUAOG6UKicYXoZqTZMEE8JkQhkl/MynrlmiN24BxLlk d7pqojUNUTNNwRcawr40Ire7kI/1BjOkGlA5AdmPqkT5AW2e0Q9V/rzG4ngdxMUfaEMdgKKb 76r5K20+Grf4yAsBruN+losWhRXKlJ6FVHmZkt+A8BOsDAbzsAB+v9T2M4nQVVWk120c+VZk 72hg3ASpTABZcUgkMxFO/VR/roX0aduoNcrKlDn2SCfItGvn9IBDJyCAWlvVbD09NqbDkl3q OwyKDohUCvSnum0nKqDEftHvpwseZyD0IM34hmMzBncBPciB5vCX7/L9ZlT2zJYasJmRKiEI ZBDMHw2MUqGOkcUUrsUIMtWcOOAj3/jczpeuRSNqLA++WT7xw1tyrn9dtHSf7RmQO0Mxh7H+ D6fozWR7hcyHdKumDSK9zWXouaUmQb3fqRRRJ2y36s/6LGU7jNKU0BHPbehmtGph0j7V99BJ kg8/is1sbN05EGtVsP6XRCzvDiDpBF0c9tPEcUq5QeV0K3W7g2FQG8eQVZ8hMcO7ZFsA2Zwj xnQwo2vXGY02FGIdZ6D3o7JvRmyMwQrFj4bbi1YVTNe8fLb+I5m23ojUe1fOKKyi9T0HxT5z DaLsDUyit0vsCIb60mo1QuZ2mzx//AlWiZwv1yKBTz9smuVcab/P+SVBU7nAeGsxWpzZn2Ip zA6lseX94ji5rndxXXWEI3h8FxEjstp0QEwY3Y1QvHNFBz3oRZPmLy8BxkvTHqFyu5eJVfUj Lb74Gu9HqN7MnqwdrNQaImsEcksxqWIPY27CaCPPoQfOsMsK1PvEMRSiam4hT2FraTRuftnZ cfznTiEVx729piLPBLpHrxAgNfHNwg1xH/JRICT8vhU+eP2WZJhcp9caAHmRrlgvMu5TPD9r 4432z2il08OD4UTo0D/reYuELz9BSFmVMGp9ZEJKbXrz8gPMDhJNsI9CIgJI+RN95m5XM+Tp hlRhmcwJILDuED6 IronPort-HdrOrdr: A9a23:d2apPansdgKVacUIJYalLctuxGLpDfLo3DAbv31ZSRFFG/Fw9/ rCoB17726QtN91YhsdcL+7V5VoLUmzyXcX2/hyAV7BZmnbUQKTRekP0WKL+Vbd8kbFh41gPM lbEpSXCLfLfCJHZcSR2njELz73quP3jJxBho3lvghQpRkBUdAF0+/gYDzranGfQmN9dP0EPa vZ3OVrjRy6d08aa8yqb0N1JNQq97Xw5fTbiQdtPW9f1DWz X-IronPort-AV: E=Sophos;i="5.98,250,1673931600"; d="scan'208";a="100764149" From: Anthony PERARD To: CC: Ian Jackson , Roger Pau Monne , Anthony PERARD Subject: [OSSTEST PATCH] PDU/IPMI: Be less aggressive with IPMI commands Date: Fri, 10 Mar 2023 17:07:21 +0000 Message-ID: <20230310170721.14493-1-anthony.perard@citrix.com> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @citrix.com) X-ZM-MESSAGEID: 1678468120519100001 Content-Type: text/plain; charset="utf-8" The machine rimava0 doesn't power off or power on when we keep doing "power status" then "power off" ipmi command without waiting. Currently, osstest does: loop ipmitool power status ipmitool power off sleep 1 This loop goes on forever, the machine never change power state. Adding some wait in between both command works. While here, test for timeout just after a status check instead of sending the `power off` command again which we would never check the result of. Signed-off-by: Anthony PERARD Reviewed-by: Ian Jackson --- Osstest/PDU/ipmi.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Osstest/PDU/ipmi.pm b/Osstest/PDU/ipmi.pm index 21c94d98..ce498cb7 100644 --- a/Osstest/PDU/ipmi.pm +++ b/Osstest/PDU/ipmi.pm @@ -69,10 +69,10 @@ sub pdu_power_state { my $count =3D 60; for (;;) { last if $getstatus->() eq $onoff; + die "did not power $onoff" unless --$count > 0; + sleep(1); =20 system_checked((@cmd, qw(power), $onoff)); - - die "did not power $onoff" unless --$count > 0; sleep(1); } } --=20 Anthony PERARD