On 12/24/25 11:51, Philippe Mathieu-Daudé wrote:
On 22/12/25 13:52, Cédric Le Goater wrote:
On 12/22/25 08:33, Cédric Le Goater wrote:
Commit [1] moved the FW image of the gb200nvl-bmc machine and broke
the associated functional test. Fix that.
[1] https://github.com/legoater/qemu-aspeed-boot/
commit/52451b2472eeb40aa97e131aeea327e9d4a8a78a
Cc: Ed Tanous <[email protected]>
Cc: Patrick Williams <[email protected]>
Signed-off-by: Cédric Le Goater <[email protected]>
---
tests/functional/arm/test_aspeed_gb200nvl_bmc.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/functional/arm/test_aspeed_gb200nvl_bmc.py b/tests/
functional/arm/test_aspeed_gb200nvl_bmc.py
index 8e8e3f05c1b2..e5f2dce0f569 100755
--- a/tests/functional/arm/test_aspeed_gb200nvl_bmc.py
+++ b/tests/functional/arm/test_aspeed_gb200nvl_bmc.py
@@ -11,7 +11,7 @@
class GB200Machine(AspeedTest):
ASSET_GB200_FLASH = Asset(
- 'https://github.com/legoater/qemu-aspeed-boot/raw/refs/heads/
master/images/gb200nvl-obmc/obmc-phosphor-image-gb200nvl-
obmc-20250702182348.static.mtd.xz',
+ 'https://github.com/legoater/qemu-aspeed-boot/raw/refs/heads/
master/images/gb200nvl-bmc/openbmc-20250702182348/obmc-phosphor-image-
gb200nvl-obmc-20250702182348.static.mtd.xz',
'b84819317cb3dc762895ad507705978ef000bfc77c50c33a63bdd37921db0dbc')
def test_arm_aspeed_gb200_openbmc(self):
Applied to aspeed-next.
Thanks, this is part of the 2 build failures blocking current pull
requests (other one being
https://lore.kernel.org/qemu-devel/[email protected]/).
Thanks, I was going to send an aspeed PR yesterday and then, I saw
the tests failing because of the missing v5.0.0 machines.
C.
If we really want to consider our CI as source of truth (and blocking
when not green), then please consider applying it directly on master
as a build fix (maintainers Cc'ed).
