-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Split out Airbyte Frontend and Cli build. (#11064)
Split out the build so builds are triggered based on files that are changed. This should help with the overwhelming builds we currently have, reduce the Github api load we see, and help save AWS costs. Here we tackle the frontend and cli builds first. Follow up PRs will tackle the connector and platform builds. General build triggering rules: - always trigger if the files directly related to the build are modified. - always trigger on master. - always trigger if .github is modified to test all build changes are compatible. - for the frontend, we also want to trigger if backend changes happen since some set of these changes can affect frontend. - Use https://github.com/dorny/paths-filter that lets us filter on paths and returns true/false if that path has been -modified. This was the best among the alternatives of using the default GHA path syntax (too restrictive), or using the next best alternative library(even uglier/complex if conditionals) I could find with an hour of googling. - Add the relevant conditionals in the frontend and octavia-cli builds. - For the frontend build, it is sufficient to add this conditional in the start step, since the rest of the steps require the start step to run for themselves to succeed.
- Loading branch information
Showing
1 changed file
with
57 additions
and
14 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters