From patchwork Mon Mar 25 15:55:27 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Anton Mikanovich X-Patchwork-Id: 3439 Return-Path: Received: from shymkent.ilbers.de ([unix socket]) by shymkent (Cyrus 2.5.10-Debian-2.5.10-3+deb9u2) with LMTPA; Mon, 25 Mar 2024 16:56:08 +0100 X-Sieve: CMU Sieve 2.4 Received: from mail-ed1-f56.google.com (mail-ed1-f56.google.com [209.85.208.56]) by shymkent.ilbers.de (8.15.2/8.15.2/Debian-8+deb9u1) with ESMTPS id 42PFu70E012003 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT) for ; Mon, 25 Mar 2024 16:56:07 +0100 Received: by mail-ed1-f56.google.com with SMTP id 4fb4d7f45d1cf-5684c1abc33sf3095248a12.2 for ; Mon, 25 Mar 2024 08:56:07 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1711382162; cv=pass; d=google.com; s=arc-20160816; b=PkNt3UXIQNSJHJyr+LwaDCD6CE4cAODSeu4nNyGmbRPs3tXgdRZUxEwWXoUwKESDQn jC+bXoAp7Ze0jcAn7tDBslya3JbUoh0/45icgcxsNVcNYj3uQtec0WDqkeBvmf6SZJF+ 8IZ1pQVWd/nGJhg57IPklAhCKODIHfNXME0BPHhuChcZgXWABzF7FC/Fr/3Bc7lhbazy tnot3ZrkOojC9zsVv6Ag2aHaQKxQyFYvdlu0+2eW0VvIY0ris+Hei216jtKtha5whnDq uT46SEy+W03cFRnlMS5s/+IQwf13Rj6CPx+nI7F1v1DDe/RFzofRaAJ/kkwLVI9GCQgJ ltGA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-unsubscribe:list-subscribe:list-archive:list-help:list-post :list-id:mailing-list:precedence:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:sender:dkim-signature; bh=GMxMvpCS4p7bMnjyP9vJWufogwToQH3SB7BaYose/+U=; fh=zvOtAEx+5wY/xdGDDHtAC8iDiB9IId9AT0Ydj+/wVPo=; b=lCBP66KZJog76ZXg6cNJgvpJekuMDtjyXfkO7M1vy0ciL+KkR8XdFHW+13jUXlLRTN uAzK/4aiAGGo34Hv2/qYtEOGTULBvduqTbi9VDiWtev96ihJybBaPolZwDwVeWWzoBnl KrrfgBpvwaTwfoENY4LmSTw/6Uxmth51leens/5yiTmGKrBfitKKZSHeC/H75il2Bv04 kDzfgCrGi1AJ01rFh5JXB41dQ0QikAcZuHGDMeOlxT8mE7oD05Tkkh4b9YVDCIML6pbM D4Hp/GoRafNYmKggdTwyg27ANglsQrAYqBladJ7mExCjohvo6MRvi9fDwl5FAkUFzvyL 9HCA==; darn=isar-build.org ARC-Authentication-Results: i=2; gmr-mx.google.com; spf=pass (google.com: domain of amikan@ilbers.de designates 85.214.156.166 as permitted sender) smtp.mailfrom=amikan@ilbers.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlegroups.com; s=20230601; t=1711382162; x=1711986962; darn=isar-build.org; h=list-unsubscribe:list-subscribe:list-archive:list-help:list-post :list-id:mailing-list:precedence:x-original-authentication-results :x-original-sender:mime-version:references:in-reply-to:message-id :date:subject:cc:to:from:sender:from:to:cc:subject:date:message-id :reply-to; bh=GMxMvpCS4p7bMnjyP9vJWufogwToQH3SB7BaYose/+U=; b=bECVWMwcTGVJixzAajq+SC7gB/6U18A65p6rX7O6zbKO9Qd9mFQMfb2vFKE6RKaXxI eUuwhXG2vSHWeTdQrk4qvFRjFRu97U17hZOI3idJa9lmRCfVBhfhMtju00T4m4Q3eyZd aURtGjvuEE5uCSkjvfu9l2IXBiIuo+FInWsDaJWwTuPktl22m0SythjIrW5Z6QkTOGX1 cV1UeMss1ab+Q3LEnksy+jUdbSrSmqgXIEztMO8m23pV0M1m/V14Cpj8P03Q+cB+Z9Qo MxOmmMqDTUOJnGzhynU3FgiWGBXgwRdpdt4sJ98eRQK0ViOW1xxJEBCW/mxfrf8DRtQ3 zdZQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1711382162; x=1711986962; h=list-unsubscribe:list-subscribe:list-archive:list-help:list-post :x-spam-checked-in-group:list-id:mailing-list:precedence :x-original-authentication-results:x-original-sender:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :x-beenthere:x-gm-message-state:sender:from:to:cc:subject:date :message-id:reply-to; bh=GMxMvpCS4p7bMnjyP9vJWufogwToQH3SB7BaYose/+U=; b=a6UKe+BJD6Nu02k3ejp/KYuAFm/4Vkjb7N+LcEeFao6XCy/YeUbses41hnP37OKScN 33TESFPNtMwtE8sKrYGFmM7HmbYcJ5uCFEZT3Tot3mN0jHIhNjWYfZFEzV/7BS7kjUEG Ju1FqBQO7f5bjiajNsX0YVXskqiPrkVoFI6vr245JQ6x3jlf2cmT/WYBua84jUkOH9HM R1QVYrOXffiAMx8xVSk8Ee3oy+nbirKdgj/hxQ1SLWzL4uVegJ078RM5cgZbTsV/Ve/X CbRELJbX/U+2dK6QcjcRnmv1bkrTLi7EaNixBKHWkY4VNKMsvCo7AvZfxzeJ17CrTX5g +paA== Sender: isar-users@googlegroups.com X-Forwarded-Encrypted: i=2; AJvYcCUvEV4v0d/eUNipAxF7guPmVegi81axvWNo6r2k5P4pbneQ+EcUN1fMEUiNSzXv28egPkTgGeR0HvxvjkLwcc9LDinxr9w= X-Gm-Message-State: AOJu0YxXRBQiXDUX6EtASfOLI+LmBgEPrIoSqrQ/exnDO6nFGUuT/eMT MLNnaUBPvnkNftIOBmjnSX3PFO8Y/lUVUbT8yPjxPIGCIX+maDnf X-Google-Smtp-Source: AGHT+IGp6BX6QZZ+GtCnLj3xCH/ZHeseKKgle8FFXsmAq6kDkE/swWqIbvNmvd2LokwkjM9RcmRdyg== X-Received: by 2002:a50:9fe6:0:b0:56b:817a:5bcf with SMTP id c93-20020a509fe6000000b0056b817a5bcfmr4947557edf.5.1711382161687; Mon, 25 Mar 2024 08:56:01 -0700 (PDT) X-BeenThere: isar-users@googlegroups.com Received: by 2002:a05:6402:f23:b0:56c:24d5:d6a4 with SMTP id i35-20020a0564020f2300b0056c24d5d6a4ls110924eda.2.-pod-prod-09-eu; Mon, 25 Mar 2024 08:56:00 -0700 (PDT) X-Received: by 2002:a17:906:f2ca:b0:a4b:56be:1e9c with SMTP id gz10-20020a170906f2ca00b00a4b56be1e9cmr1223258ejb.36.1711382159732; Mon, 25 Mar 2024 08:55:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1711382159; cv=none; d=google.com; s=arc-20160816; b=nb4cx95pBH+0qjnLB77PisGtEvjIn22fg81yahL2CPqn6zIo1WESleRiX9LbjuHvaQ NcMY43G9eq2F3M2F+hEZFiVKq8nqjC/NZlJb6c3Yd1aI7+vp6zoeTpStFp3sVYToOpV7 TfkHpiAEBX18q9h8rDLX14PdNzjG9LsqrnmE59u6FfMZ4byJixROaUR4QAvi5/Q+OHhV 5S7ZQFBZDLdHQKpLm3msWulRDhRslCRuaaFjk5C6zAcHdNX3hN/nYJCRiaCDu1gq9Mas tP9Ma60QAPvzOhu4mWXIuQcEQKnG77k1FdCA8e09V49MfA1h3EXff7W1sDOxVuB9yMtB F1NQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from; bh=z+adShdtx7Zxk6VubvDnwhm71AzxKzor7Om8sdeH6U0=; fh=/h9QQkzJ8EboVkWg45aWwpaUro6WMavIVd2OhN45RtE=; b=XGWTf0/8OGGV7+CNdIW5/ZYPjM+cPCS3kSbShdk12RA6dB/utSTvvje69Dhru/01eu vZakrpq5ZtmbBmB8FhoDfMV2FKlghN7jia3c/wftm/tbjhVzbHSVnlo14NbKDtuzZPWe 2eTH+kud3DbRotfWhAoHR5pOy75asqHfMp6Kgz+pfjhsCglwR+YXV1UN3AHg/jvIJtxn x9yBLMh1ZOtMz7MPpbRyJ0an+P6MjCPRZ4w0obZ20iN3hUYFLQfSYQ3br4gDZhikj5xy 6ML37MSgFB9nAzaGpbzu6W1cHBfI+0QBmy4ATz85C9aV3XHqwMNfw/oqv/wTFkJgVaZs baQg==; dara=google.com ARC-Authentication-Results: i=1; gmr-mx.google.com; spf=pass (google.com: domain of amikan@ilbers.de designates 85.214.156.166 as permitted sender) smtp.mailfrom=amikan@ilbers.de Received: from shymkent.ilbers.de (shymkent.ilbers.de. [85.214.156.166]) by gmr-mx.google.com with ESMTPS id v14-20020a170906380e00b00a46acd22106si229727ejc.0.2024.03.25.08.55.59 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Mon, 25 Mar 2024 08:55:59 -0700 (PDT) Received-SPF: pass (google.com: domain of amikan@ilbers.de designates 85.214.156.166 as permitted sender) client-ip=85.214.156.166; Received: from user-B660.promwad.corp ([159.148.83.123]) (authenticated bits=0) by shymkent.ilbers.de (8.15.2/8.15.2/Debian-8+deb9u1) with ESMTPSA id 42PFtnLJ011703 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 25 Mar 2024 16:55:59 +0100 From: Anton Mikanovich To: isar-users@googlegroups.com Cc: Anton Mikanovich Subject: [PATCH 10/23] CI: Fix errors reporting during startvm Date: Mon, 25 Mar 2024 17:55:27 +0200 Message-Id: <20240325155540.4162990-11-amikan@ilbers.de> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240325155540.4162990-1-amikan@ilbers.de> References: <20240325155540.4162990-1-amikan@ilbers.de> MIME-Version: 1.0 X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_MED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_EF,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_MSPIKE_H2,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on shymkent.ilbers.de X-Original-Sender: amikan@ilbers.de X-Original-Authentication-Results: gmr-mx.google.com; spf=pass (google.com: domain of amikan@ilbers.de designates 85.214.156.166 as permitted sender) smtp.mailfrom=amikan@ilbers.de Precedence: list Mailing-list: list isar-users@googlegroups.com; contact isar-users+owners@googlegroups.com List-ID: X-Spam-Checked-In-Group: isar-users@googlegroups.com X-Google-Group-Id: 914930254986 List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= Test case failure should be triggered by self.fail() API. It helps to prepare much more correct log output. So rebuild error handling of startvm related functions. Signed-off-by: Anton Mikanovich --- testsuite/cibuilder.py | 35 +++++++++++++++++++++++------------ 1 file changed, 23 insertions(+), 12 deletions(-) diff --git a/testsuite/cibuilder.py b/testsuite/cibuilder.py index fa30c2f5..d282f8ee 100755 --- a/testsuite/cibuilder.py +++ b/testsuite/cibuilder.py @@ -475,7 +475,7 @@ BBPATH .= ":${LAYERDIR}"\ p1 = subprocess.Popen('exec ' + ' '.join(cmdline), shell=True, stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.PIPE, universal_newlines=True) - self.log.info("Started VM with pid %s" % (p1.pid)) + self.log.info("Starting VM with pid %s" % (p1.pid)) return p1, cmdline, boot_log @@ -506,8 +506,10 @@ BBPATH .= ":${LAYERDIR}"\ if fd == p1.stderr.fileno(): app_log.error(p1.stderr.readline().rstrip()) - self.log.error("Didn't get login prompt") - return 1 + rc = 1 + if time.time() > timeout: + rc = 2 + return rc def vm_parse_output(self, boot_log, multiconfig, skip_modulecheck): @@ -539,10 +541,8 @@ BBPATH .= ":${LAYERDIR}"\ if (module_output in data or skip_modulecheck): if resize_output and not resize_output in data: rc = 1 - self.log.error("No resize output while expected") else: rc = 2 - self.log.error("No example module output while expected") return rc @@ -553,13 +553,16 @@ BBPATH .= ":${LAYERDIR}"\ def vm_turn_off(self, vm): - pid = self.vm_dict[vm][0] - os.kill(pid, signal.SIGKILL) + try: + pid = self.vm_dict[vm][0] + os.kill(pid, signal.SIGKILL) - del(self.vm_dict[vm]) - self.vm_dump_dict(vm) + del(self.vm_dict[vm]) + self.vm_dump_dict(vm) - self.log.info("Stopped VM with pid %s" % (pid)) + self.log.info("Stopped VM with pid %s" % (pid)) + except ProcessLookupError: + self.log.error("Can't stop VM %s" % vm) def vm_start(self, arch='amd64', distro='buster', @@ -611,7 +614,10 @@ BBPATH .= ":${LAYERDIR}"\ rc = self.vm_wait_boot(p1, timeout) if rc != 0: self.vm_turn_off(vm) - self.fail('Failed to boot qemu machine') + if rc == 2: + self.fail("Didn't get login prompt") + else: + self.fail('Failed to boot qemu machine') if cmd is not None or script is not None: self.ssh_user='ci' @@ -638,7 +644,12 @@ BBPATH .= ":${LAYERDIR}"\ if rc != 0: if not keep: self.vm_turn_off(vm) - self.fail('Failed to parse output') + if rc == 1: + self.fail("No resize output while expected") + elif rc == 2: + self.fail("No example module output while expected") + else: + self.fail('Failed to parse output') if not keep: self.vm_turn_off(vm)