search

mcu-tools / mcuboot

Secure boot for 32-bit Microcontrollers!
Apache License 2.0
1.34k stars 676 forks source link

ci: Update TF-M version to eb8ff0d #2051

Closed davidvincze closed 2 months ago

davidvincze commented 2 months ago

Update TF-M version to keep up with code development. Fixes and new features have been introduced on both sides, using a more recent TF-M version helps to detect regression.

nordicjm commented 2 months ago

@davidvincze any clue as to what's going on with the FIH failures? Seeing them on main CI too and other PRs

utzig commented 2 months ago

@davidvincze any clue as to what's going on with the FIH failures? Seeing them on main CI too and other PRs

I believe this was due to a bug I described on #2053. I updated the hub to point latest to 0.0.2 again so it should be functional now. I am re-running the FIH jobs on #2054 and it looks fine so far.

utzig commented 2 months ago

Please rebase!

With #2053 I updated only the docker image so all the new tools required by newer TFM releases are available. This PR should still fail because just pointing to a newer TFM is not enough, since the scripts in ci/fih_test_docker must also be updated due to different build options, output directories, etc. I got it to the point where it runs gdb but hangs, so I will cherry-pick this commit and send a new PR once I get some time to fix the remaining issues, hopefully tomorrow or Friday.

It there are any fails in other CI jobs, one can rollback by just changing the version in ci/fih-tests_version.sh back to 0.0.2.