Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CI improvements #242

Merged
merged 3 commits into from
Aug 19, 2023
Merged

CI improvements #242

merged 3 commits into from
Aug 19, 2023

Commits on Aug 18, 2023

  1. CI: rework into multiple workflows 

    Github Actions does not support adding dependencies between multiple
workflows. For this reason, we previously defined all CI actions in a
single workflow file. As we really don't like this approach, here is
another attempt to improve the situation.

We now have a meta workflow, that will call other workflows based on
the changed files. If any file is being changed that would result in a
changed build container, we call the build_container workflow. The
build_container workflow calls the build_willow workflow once it's done.
If the changed files would not result in a changed build container, we
directly call the build_willow workflow.

This will make the CI more efficient, and faster in case we don't change
any files that would result in a changed build container. It will also
work around the limitation of Github Actions that prevent us from
publishing container images to GHCR for PRs openend from forks. This
will still be a problem when people want to submit changes to the build
container, but we're still investigating options how to fix this
properly.
    @stintel
    stintel committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    f076f40 View commit details
    Browse the repository at this point in the history

  2. patches: drop directory 

    Since cbd03e3 ("utils.sh: use willow branch from our ESP-ADF fork")
we no longer need to apply patches when preparing the build container.
Drop the patches directory as it is no longer used since that commit.
    @stintel
    stintel committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    03ac28e View commit details
    Browse the repository at this point in the history

  3. CI: only build Willow when needed 

    There is no point in running a Willow build when there aren't any
changes to source files, or other files included in the Willow images.
    @stintel
    stintel committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    7365601 View commit details
    Browse the repository at this point in the history