You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/04/21 14:03:06 UTC

[GitHub] [airflow] potiuk opened a new pull request, #23150: JUST TESTING DIRECT PR FOR NEW MOSTLY_BREEZE CI

potiuk opened a new pull request, #23150:
URL: https://github.com/apache/airflow/pull/23150

   We have the new Python-based breeze and we want to replace all
   the functionality used from the old Breeze with the new one.
   
   The most important is image management. It is used in many
   places and this PR replaces all the places and removes image
   builidng, pushing, pulling to use the new Breeze everywhere.
   
   That includes:
   
   * Building and pushing CI image on CI
   * Building and pushing PROD image on CI which includes:
     * building Airflow packages
     * building Provider packages
     * waiting for images in parallel (both CI and and PROD)
     * veifying the images (both CI and PROD)
   
   All those commands have been moved to breeze.py and that required
   some modification in the Breeze parameter handling - mainly related
   to adding spaces when help was displayed because long list of
   packages looked very bad in help output.
   
   It's been easier to implement it in one big PR as using
   image building and pulling was deeply embedded in many scripts.
   
   With this change all the scripts in CI use directly breeze
   and combine using command line parameters with evn variables
   directly in the job that execute the breeze commands which
   makes it much easier to understand what is going on and
   repeat it locally using Breeze.
   
   Fixes: #22825
   Fixes: #23077
   Fixes: #23076
   Fixes: #22829
   Fixes: #22828
   Fixes: #22826
   Fixes: #20961
   Fixes: #23102
   Fixes: #21098
   
   <!--
   Thank you for contributing! Please make sure that your code changes
   are covered with tests. And in case of new features or big changes
   remember to adjust the documentation.
   
   Feel free to ping committers for the review!
   
   In case of existing issue, reference it using one of the following:
   
   closes: #ISSUE
   related: #ISSUE
   
   How to write a good git commit message:
   http://chris.beams.io/posts/git-commit/
   -->
   
   ---
   **^ Add meaningful description above**
   
   Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more information.
   In case of fundamental code change, Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals)) is needed.
   In case of a new dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
   In case of backwards incompatible changes please leave a note in a newsfragement file, named `{pr_number}.significant.rst`, in [newsfragments](https://github.com/apache/airflow/tree/main/newsfragments).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] potiuk commented on pull request #23150: JUST TESTING DIRECT PR FOR NEW MOSTLY_BREEZE CI

Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #23150:
URL: https://github.com/apache/airflow/pull/23150#issuecomment-1107409801

   Closing as it did it's job and got green.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] potiuk closed pull request #23150: JUST TESTING DIRECT PR FOR NEW MOSTLY_BREEZE CI

Posted by GitBox <gi...@apache.org>.
potiuk closed pull request #23150: JUST TESTING DIRECT PR FOR NEW MOSTLY_BREEZE CI
URL: https://github.com/apache/airflow/pull/23150


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org