From mboxrd@z Thu Jan 1 00:00:00 1970 X-GM-THRID: 7013677320161460224 X-Received: by 2002:a1c:1f8a:: with SMTP id f132mr14214039wmf.186.1633344346339; Mon, 04 Oct 2021 03:45:46 -0700 (PDT) X-BeenThere: isar-users@googlegroups.com Received: by 2002:a05:600c:4f8e:: with SMTP id n14ls9471061wmq.1.gmail; Mon, 04 Oct 2021 03:45:45 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx+ljVakrXncZ8nYGeJDtHm3URCmKoUQ3yzApcJsqjVMglXVYMke2Tob3Vx0DLv/ziZ7MiC X-Received: by 2002:a7b:c1ce:: with SMTP id a14mr4107862wmj.183.1633344345382; Mon, 04 Oct 2021 03:45:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1633344345; cv=none; d=google.com; s=arc-20160816; b=GR0qqg0OtopUgViIsvdiq6c1ZvjFtVWeghDJNaB8/jmZmbuE7fXMv2yU27tA4l2y9Q 2jTjnVZpccp8+pX7RO9O2dLO957128olgChdE79xshwNikHLNolCRP3KjVmv7kBzXHey IdFM8tIuKPNDLJd/ydA6n38RxB7+DnM+XLaGriC8cR84ajokD5QJcUF/k6NECykqN41M b7uTRxia0xMSai0rKu/vJpecjpXOSOMkFZVWG9kLX41D5xKO8AaVpIspcX8zp09rIiFr Q6uoAsIpmQZZzNOxZle0gOxTo7j+tk4KD+fr9wxFcK4ZjWTKyqHfdg6sm5Gv4JLysAP0 PyyA== 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:subject:cc:to:from:date; bh=MaYpKd2lEv1PPWTgGmqqalAcghJILPImhnf/FqLuIII=; b=B7j3et/UMjdCv3CCinOhYvYy/pp6PPH3N/pruwt1hwcRhBtoUkwV/oBnZ5AuhCoyZQ qIZwk+dhnqK7DCQIgRjXqZ6J+r1yM3NP5d1UwmpCSnca5wP0CDgU5gZIw4k67V48Igfk /1EY9iUisdi0avpQgXQBFE2fRdRZpitxukbD5z9IUTPOnVk0nJG5XU9L7ZZ/TzswWntK eByblJBSxIwkLWvGCwInLtw37l72cBtI9RAslQrsyxYwalrSfdbelhoV9Y3eTvo7kK9S 6732+cxvYXXQyM3tkgfyt5pLFTd2GGtvTR+B8v3334K6pLheAfzZuVUFYjU8J7yIXIvc ASIQ== ARC-Authentication-Results: i=1; gmr-mx.google.com; spf=pass (google.com: domain of henning.schild@siemens.com designates 192.35.17.14 as permitted sender) smtp.mailfrom=henning.schild@siemens.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=siemens.com Return-Path: Received: from david.siemens.de (david.siemens.de. [192.35.17.14]) by gmr-mx.google.com with ESMTPS id s194si358462wme.0.2021.10.04.03.45.45 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Mon, 04 Oct 2021 03:45:45 -0700 (PDT) Received-SPF: pass (google.com: domain of henning.schild@siemens.com designates 192.35.17.14 as permitted sender) client-ip=192.35.17.14; Authentication-Results: gmr-mx.google.com; spf=pass (google.com: domain of henning.schild@siemens.com designates 192.35.17.14 as permitted sender) smtp.mailfrom=henning.schild@siemens.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=siemens.com Received: from mail1.sbs.de (mail1.sbs.de [192.129.41.35]) by david.siemens.de (8.15.2/8.15.2) with ESMTPS id 194AjiaI001675 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 4 Oct 2021 12:45:44 +0200 Received: from md1za8fc.ad001.siemens.net ([139.25.0.59]) by mail1.sbs.de (8.15.2/8.15.2) with ESMTP id 194AjivG018260; Mon, 4 Oct 2021 12:45:44 +0200 Date: Mon, 4 Oct 2021 12:45:44 +0200 From: Henning Schild To: Srinuvasan A Cc: isar-users Subject: Re: Builds get fails when we append SWUPDATE yml file Message-ID: <20211004124544.02bc943b@md1za8fc.ad001.siemens.net> In-Reply-To: <57ec0754-a775-48bf-8ac1-39b1f21289afn@googlegroups.com> References: <5ed36530-f7d3-4672-96ce-479f68d00b96n@googlegroups.com> <262e3f23-82c7-e079-1452-b83bd3e9ee8e@siemens.com> <57ec0754-a775-48bf-8ac1-39b1f21289afn@googlegroups.com> X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-TUID: hGPqf4X0RkJQ Hi Srinuvasan, my suggestion would in fact be the same. The layer that brings the wic plugin should extend the list of variables. I removed them because they are in fact "bad style". For the upstream plugins i decided to follow the symlinks to find the actual filenames, something the efibootguard plugin might also want to consider. The variables are in fact nasty hacks for the isar testsuite to start qemu and should have never been used in wic plugins (imho). So you have the following options: 1. extend wicvars in your layer 2. extend wicvars in the layer with the custom plugin 3. upstream that plugin into isar Order is ascending in quality/effort/sustainability of such custom plugins. Henning Am Thu, 30 Sep 2021 21:49:57 -0700 (PDT) schrieb Srinuvasan A : > Hi Florian Bezdeka, > > Thanks for your suggestion to fix it > locally, Let's wait for henning reply on this. > > Thanks, > Srinuvasan.A > > On Thursday, September 30, 2021 at 7:12:06 PM UTC+5:30 Florian > Bezdeka wrote: > > > On 30.09.21 12:50, Srinuvasan A wrote: > > > Hi All and Henning, > > > > if you need feedback from Henning you should add him at least to CC > > or even better TO in this case. Otherwise he just might not read > > this mail. > > > > > > Our downstream project isar-siemens SLLL and isar-cip-core get > > > build failure when we include this commit 6fa0197 > > > < > > https://github.com/ilbers/isar/commit/6fa0197ac36a86b2ae5c27f5b7804dd9a46d3c0b > > > > >. > > > The build get fails when we append SWUPDATE yml file. > > > this will add the KERNEL_IMAGE and INITRD_IMAGE variable, this > > > will defined in sw-description.tmpl file > > > > > > TEMPLATE_VARS += "PN ROOTFS_PARTITION_NAME KERNEL_IMAGE > > > INITRD_IMAGE" SWU_ADDITIONAL_FILES += "${INITRD_IMAGE} > > > ${KERNEL_IMAGE} ${ROOTFS_PARTITION_NAME}" > > > > Henning removed KERNEL_IMAGE and INITRD_IMAGE from the list of > > variables available in wic files, that might be a breaking change, > > but it seems ISAR itself (the ISAR layer) never used it, so it's > > kind of OK but remains a breaking change. > > > > You should be able to add something like "WICVARS += KERNEL_IMAGE > > INITRD_IMAGE" to your layers (as temporary workaround) > > > > > > > > when we add kas/opt/ebg-swu.yml file this will take the > > > " > > https://gitlab.com/cip-project/cip-core/isar-cip-core/-/blob/next/wic/simatic-ipc227e-efibootguard.wks > > > > > < > > https://gitlab.com/cip-project/cip-core/isar-cip-core/-/blob/next/wic/simatic-ipc227e-efibootguard.wks > > > > >" > > > file. > > > this will configure the wic pulgin as "part --source > > > efibootguard-boot", in this script it is expecting those > > > KERNEL_IMAGE and INITRD_IMAGE or otherwise it throws the error. > > > > > > Failure logs below: > > > > > > `DEBUG: Executing python function do_wic_image > > > DEBUG: Executing shell function generate_wic_image > > > INFO: Creating image(s)... > > > > > > WARNING: KERNEL_IMAGE not set. Use default: > > > WARNING: INITRD_IMAGE not set > > > > > > ERROR: file vmlinuz not found in directory > > > /build/tmp/deploy/images/simatic-ipc227e > > > WARNING: exit code 1 from a shell command. > > > ERROR: Error executing a python function in exec_python_func() > > > autogenerated: > > > > > > The stack trace of python calls that resulted in this > > > exception/failure > > was: > > > File: 'exec_python_func() autogenerated', lineno: 2, function: > > > 0001: > > > *** 0002:do_wic_image(d) > > > 0003: > > > File: '/work/isar/meta/classes/wic-img.bbclass', lineno: 141, > > > function: do_wic_image > > > 0137:} > > > 0138: > > > 0139:do_wic_image[file-checksums] += "${WKS_FILE_CHECKSUM}" > > > 0140:python do_wic_image() { > > > *** 0141: bb.build.exec_func("generate_wic_image", d) > > > 0142: bb.build.exec_func("check_for_wic_warnings", d) > > > 0143:} > > > 0144:addtask wic_image before do_image after do_image_tools > > > 0145: > > > File: '/work/isar/bitbake/lib/bb/build.py', lineno: 251, > > > function: > > exec_func > > > 0247: with bb.utils.fileslocked(lockfiles): > > > 0248: if ispython: > > > 0249: exec_func_python(func, d, runfile, cwd=adir) > > > 0250: else: > > > *** 0251: exec_func_shell(func, d, runfile, cwd=adir) > > > 0252: > > > 0253: try: > > > 0254: curcwd = os.getcwd() > > > 0255: except: > > > File: '/work/isar/bitbake/lib/bb/build.py', lineno: 452, function: > > > exec_func_shell > > > 0448: with open(fifopath, 'r+b', buffering=0) as fifo: > > > 0449: try: > > > 0450: bb.debug(2, "Executing shell function %s" % func) > > > 0451: with open(os.devnull, 'r+') as stdin, logfile: > > > *** 0452: bb.process.run(cmd, shell=False, stdin=stdin, > > > log=logfile, extrafiles=[(fifo,readfifo)]) > > > 0453: finally: > > > 0454: os.unlink(fifopath) > > > 0455: > > > 0456: bb.debug(2, "Shell function %s finished" % func) > > > File: '/work/isar/bitbake/lib/bb/process.py', lineno: 182, > > > function: run 0178: if not stderr is None: > > > 0179: stderr = stderr.decode("utf-8") > > > 0180: > > > 0181: if pipe.returncode != 0: > > > *** 0182: raise ExecutionError(cmd, pipe.returncode, stdout, > > > stderr) 0183: return stdout, stderr > > > Exception: bb.process.ExecutionError: Execution of > > > > > '/build/tmp/work/ebsy-buster-amd64/ebsy-image-reference-simatic-ipc227e-wic-swu-img/1.0-r0/temp/run.generate_wic_image.157804' > > > > > failed with exit code 1: > > > INFO: Creating image(s)... > > > > > > WARNING: KERNEL_IMAGE not set. Use default: > > > WARNING: INITRD_IMAGE not set > > > > > > ERROR: file vmlinuz not found in directory > > > /build/tmp/deploy/images/simatic-ipc227e > > > WARNING: exit code 1 from a shell command. > > > > > > DEBUG: Python function do_wic_image finished > > > ` > > > > > > Please revert back this commit. > > > > As the affected commit is not in master yet we have all > > possibilities. Reverting is one of them, but we could also consider > > adding that to the release notes for the next release. That > > decision is up to the maintainers I guess. > > > > > > > > Thanks, > > > Srinuvasan.A > > > > > > -- > > > You received this message because you are subscribed to the Google > > > Groups "isar-users" group. > > > To unsubscribe from this group and stop receiving emails from it, > > > send an email to isar-users+...@googlegroups.com > > > . > > > To view this discussion on the web visit > > > > > https://groups.google.com/d/msgid/isar-users/5ed36530-f7d3-4672-96ce-479f68d00b96n%40googlegroups.com > > > > > < > > https://groups.google.com/d/msgid/isar-users/5ed36530-f7d3-4672-96ce-479f68d00b96n%40googlegroups.com?utm_medium=email&utm_source=footer > > > > >. > > > > > > -- > > Siemens AG, T RDA IOT > > Corporate Competence Center Embedded Linux > > >