CI: Zephyr, test against known working version and also the very latest on main #623
Conversation
The CI does not need to strictly adhere to the coding rules. Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
|
@arnopo I have marked this as RFC until your PR goes in. I will then rebase and repush it and remove the RFC. |
Use last version of checkout action. Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
To have a coherent directory hierarchy, move the open-amp git from root to the "./open-amp" directory. Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
Build system reference applications instead of open-amp deprecated applications. Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout open-amp | ||
| uses: actions/checkout@v4 |
There was a problem hiding this comment.
to add
with:
path: open-amp
| path: libmetal | ||
| - name: build for Zephyr (Known Good) | ||
| id: build_Zephyr | ||
| uses: ./.github/actions/build_ci |
There was a problem hiding this comment.
uses: ./open-amp/.github/actions/build_ci
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout open-amp | ||
| uses: actions/checkout@v4 |
.github/workflows/heathcheck.yml
Outdated
| path: libmetal | ||
| - name: Zephyr Latest | ||
| id: build_Zephyr_latest | ||
| uses: ./.github/actions/build_ci |
There was a problem hiding this comment.
uses: ./open-amp/.github/actions/build_ci
wmamills
force-pushed
the
wip-zephyr-ci
branch
from
7018caa
to
13e6d4c
Compare
Reduce the console output for the Zephyr sdk install. This now matches the intent of libmetal but fixes the wget option. Signed-off-by: Bill Mills <bill.mills@linaro.org>
Allow Zephyr testing to either use locked known good values or the very latest versions. This brings open-amp to parity with libmetal. Open-amp did not have the SDK URL discovery code so that was added here as part of this. The known good versions are best for PR checking as if the build fails it is almost always the PR itself that broke it. The latest version is good for periodically checking compatibility with the very latest Zephyr changes. For now we run both on pushes and PRs. We also run main against the latest zephyr check weekly as a look ahead. Signed-off-by: Bill Mills <bill.mills@linaro.org>
wmamills
force-pushed
the
wip-zephyr-ci
branch
from
13e6d4c
to
7a41f27
Compare
|
@arnopo I rebased this on your CI update. As yours is not in main yet you will see your commits in this PR but they should disappear after you merge your PR. (Let me know if there is any issue with that.) I took the "draft" mark off this now. It should be ready to go as soon as you accept your PR. I went ahead and clones openamp-system-reference for the zephyr jobs to keep things consistent. The zephyr build step does not use the openamp-system-reference directory right now but it may in the future. |
This PR brings open-amp up to parity with libmetal with regard to Zephyr CI testing
open-amp did not have the latest SDK discovery code nor the sdk install reduction of verbosity that were already in libmetal so this PR is a bit bigger than the one for libmetal but the end result is the same.