The GitHub Actions job "Tests" on airflow.git has succeeded. Run started by GitHub user potiuk (triggered by potiuk).
Head commit for run: c350c37afb964bc04d2b2b9f8aece206b0704b9e / Jarek Potiuk <[email protected]> Make passing build args explicit in ci/prod builds When building hte image, breeze converts some simple parameters passed as breeze command (with autocompletion and explanation) into much longer and more complex set of build args that are passed to `docker build` command. The way how passing hte args worked so far is that it was pretty implicit: * **kwargs were used to ingest `click` flags * parameters found as empty/None were filtered out from these * Build*Params dataclass was created out of such kwargs dict * argumenst from dataclass (with some customization) were converted to --build-arg (CAPITALIZED_PROPERTY_NAME) This had a lot of implicitness and it was not easy to understand whether the parameters passed were correct and how they passed through this chain. This change makes all the build arg much more explicit - without kwargs and dictionary. Each CI/PROD build param has now a method where it explicitly converts arguments into build-args - including specifying which of those are optional (where you can actually filter out Empty and None values) and which are required (where an actual value is expected). This PR also cleans up the click flags sequence and their presence as well as the output of help command (they were grouped with more related parameters) Report URL: https://github.com/apache/airflow/actions/runs/6938562596 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
