xmos / sln_voice

XCORE-VOICE Solution
Other
10 stars 21 forks source link

fix jenkins label and update xjsl to 0.28 #320

Closed BrennanGit closed 1 year ago

BrennanGit commented 1 year ago

Despite #301 being merged, it seems there were tests that only run on nightlies that saw issues we attempt to remedy here.

Apparently there is an XN file which requires the test to be run on a vrd-type board. Also needed to fix the /projects mount path as we moved off the US agents.

examples:

Also including the xjsl version bump we've seen elsewhere. The most significant change in which is the build discarder.

BrennanGit commented 1 year ago

running once more with full test enabled as thats where we saw the issues (once merged to develop)

BrennanGit commented 1 year ago

ok next issue is the new jenkins agents have 2 boards attached.

dfu-utils needs one selected but we need to translate xtagid into something dfu-utils can understand.

dfu-util: Invalid DFU suffix signature
dfu-util: A valid DFU suffix will be required in a future dfu-util release!!!
dfu-util: More than one DFU capable USB device found! Try `--list' and specify the serial number or disconnect all but one device
xluciano commented 1 year ago

@BrennanGit, I updated this PR with the latest changes from develop. Can you check I solved the merge conflicts correctly?

ed-xmos commented 1 year ago

@BrennanGit any reason we shouldn't merge this?

BrennanGit commented 1 year ago

I wasn't sure the multiple xtags for dfu-utils had been resolved but it does seem to be passing. The PYTHON_VERSION fix is pretty crucial though.

Happy to merge if this next build passes.

ed-xmos commented 1 year ago

BTW, this fail - https://github.com/xmos/sln_voice/actions/runs/6719127858 is fixed in https://github.com/xmos/sln_voice/pull/325 which should be merged shortly

BrennanGit commented 1 year ago

I did a full run and it passed dfu so it's all good. By the time it finished, the base branch had been updated. Last time for sure now.