U-boot variables not substituted in template?

hardboprobot commented 5 months ago

Example:

Test id: 65c3c70343cf22c9fe117f3f
Test name: login (baseline-arm64)
Date: 2024-02-07T18:01:46.973000
Log: https://kciapistagingstorage1.file.core.windows.net/staging/baseline-arm64-65c3c58a43cf22c9fe117f2f/log.txt?sv=2022-11-02&ss=f&srt=sco&sp=r&se=2024-10-17T19:19:12Z&st=2023-10-17T11:19:12Z&spr=https&sig=sLmFlvZHXRrZsSGubsDUIvTiv%2BtzgDq6vALfkrtWnv8%3D
Kernel:
- tree: kernelci
- branch: staging-stable
- commit: 317e5a321247ea2eabbcfea5e7e7600efd2a50c9
- describe: staging-stable-20240207.13
Platform: meson-g12b-a311d-khadas-vim3
Lava job id: 12713755
Result: fail

https://lava.collabora.dev/scheduler/job/12713755

Really weird, the Uboot template variables aren't substituted by values:

kvim3#setenv initrd_size ${filesize}
setenv initrd_size ${filesize}
kvim3#tftp {DTB_ADDR} 12713660/tftp-deploy-z94pv2di/dtb/meson-g12b-a311d-khadas-vim3.dtb
tftp {DTB_ADDR} 12713660/tftp-deploy-z94pv2di/dtb/meson-g12b-a311d-khadas-vim3.dtb
Speed: 1000, full duplex
Using dwmac.ff3f0000 device
TFTP from server 192.168.201.1; our IP address is 192.168.201.44
Filename '12713660/tftp-deploy-z94pv2di/dtb/meson-g12b-a311d-khadas-vim3.dtb'.
Load address: 0x0
Loading: *####
5 MiB/s
done
Bytes transferred = 52193 (cbe1 hex)
kvim3#setenv bootargs 'console=ttyAML0,115200n8 root=/dev/ram0  ip=dhcp'
setenv bootargs 'console=ttyAML0,115200n8 root=/dev/ram0  ip=dhcp'
kvim3#{BOOTX}
{BOOTX}
Unknown command '{BOOTX}' - try 'help'

a-wai commented 5 months ago

This is most probably caused by https://lava.collabora.dev/scheduler/job/12713755/definition#defline6, due to the node being refreshed too late (see https://github.com/kernelci/kernelci-core/pull/2347#discussion_r1478483153)

I'm trying to fix it in https://github.com/kernelci/kernelci-core/pull/2342

a-wai commented 5 months ago

Should be fine on the next run

hardboprobot commented 5 months ago

Awesome, thanks!

kernelci / kernelci-api

U-boot variables not substituted in template? #472