[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v1] automation: provide example for downloading an existing container


  • To: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  • From: Olaf Hering <olaf@xxxxxxxxx>
  • Date: Tue, 16 May 2023 19:52:04 +0000
  • Arc-authentication-results: i=1; strato.com; arc=none; dkim=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; t=1684266747; s=strato-dkim-0002; d=strato.com; h=References:In-Reply-To:Message-ID:Subject:Cc:To:From:Date:Cc:Date: From:Subject:Sender; bh=BDMMOZsldew5E9T5aGGViOq4aXtP5rfj0eDk121sUHo=; b=V26oMDKTx+Ey+q4epnIkaz5JpP0V3iFNciVaMZ/XuGpTEpF4EjcXFAb8ZVaVoXHzo3 1LuMoYfl4WPUN3n+TTx54xOlI0zNzF99STP/HcxlIHsaDKBkQwgN8armZRcymVRSjy+h 5X27jlEgWB2+ZUL+/qnPICdOaWiRrwjoDc66iAryzoOqylSnDw4PlgT9UBHWwuQ0k9td GEPYbyGIgPr7ywOag2014WdU7kjeK0yKPMkmArDXofttC54SdXTycuQurBniU07QFnpV YXjuhrG+bhHpJPplvzyc69oSKOmp2pNOr1p52RWk1kLXswcPILuyrExAZB5QrrRVjsOv +uVg==
  • Arc-seal: i=1; a=rsa-sha256; t=1684266747; cv=none; d=strato.com; s=strato-dkim-0002; b=pn33qygtTjnTEZquBGtiwjIp123CnWh5/RtXZ8i11EgFae4xnIoVmNPm5o/Dv0SYR6 MWQcS66dx4zhA0cOay3LCiVlMYPgeA5cdYj5slw0qpCFnwe5d+WJhIzujGLb3nn6me1x poPPYSsmtHTGuVefW7j2I0Evj/SqKM2gGg9XD0DhpteVTz+DePpI42suPmvLF9r8YxMD JxsbjhPKtGimAgQtL0KFskvCRMcyBv9BnHlMUWrxuHgdwYGg28bugntC5cwgS3PKigvQ q2vVGr/6Ydw1qwyRu7kDmsB3kMvyQ+KCJxWTQOr6KC1bXdECWcRJ897I53ZgbCGFitUY IXiw==
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, Doug Goldstein <cardoe@xxxxxxxxxx>
  • Delivery-date: Tue, 16 May 2023 19:52:41 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Am Tue, 16 May 2023 11:46:00 -0700 (PDT)
schrieb Stefano Stabellini <sstabellini@xxxxxxxxxx>:

> I think you have a point that automation/build/README.md should also
> describe how to do what gitlab-ci does but locally (i.e. call
> automation/scripts/build). It should not only describe
> automation/scripts/containerize.

Meanwhile I have figured this out, additional variables must be set. I
already sent a patch for the example. That way I was able to
understand and reproduce the error seen in the CI build.

> https://gitlab.com/xen-project/xen/-/jobs/4284741849

It turned out this bug in qemu is triggered by debug=y vs. debug=n in
the build environment. I have not checked which commit exactly fixed it
in upstream qemu.git, it should probably be backported. Or qemu should
be moved forward to v8.x at some point. I think I have not seen this
specific failure in my own qemu.git builds.

The reason is: --enable-debug will disable _FORTIFY_SOURCE, so the build
succeeds. Without that flag, configure will enable _FORTIFY_SOURCE.

> Sure I see your point. On the other hand Tumbleweed jobs are the ones
> and only with "allow_failure". So among all the possible choices as
> example, do we really need to pick the one and only that has been
> failing for months? :-)

Yeah, this is exactly the point, to give copy&paste commands so that
contributors can investigate such failures locally.

I did not follow the state of the openSUSE builds in the past months. I
think Tumbleweed did succeed a few weeks ago because the last snapshot
was meanwhile one year old, and all gcc12 bugs are already fixed by now.


Olaf

Attachment: pgp8yCQ5WBx5c.pgp
Description: Digitale Signatur von OpenPGP


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.