This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new b141fbcb08 Small fixups to CI diagrams (#29235)
b141fbcb08 is described below

commit b141fbcb08ff59425769f013173c695298981b9f
Author: Jarek Potiuk <[email protected]>
AuthorDate: Mon Jan 30 14:04:33 2023 +0100

    Small fixups to CI diagrams (#29235)
    
    There are few small fixups in CI diagrams that would reflect better
    the current flow:
    
    * while Build ARM images is not dependency of other jobs, they
      contribute to overall build status
    * Canary run is always run as "merge" run so there is no conditional
      needed there.
---
 CI_DIAGRAMS.md | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/CI_DIAGRAMS.md b/CI_DIAGRAMS.md
index 0b6fd95f7e..4fc7724c15 100644
--- a/CI_DIAGRAMS.md
+++ b/CI_DIAGRAMS.md
@@ -112,9 +112,9 @@ sequenceDiagram
     opt
         Note over Tests: Generate constraints
     end
+    Note over Tests: Build ARM CI images
     Tests -->> Airflow Repo: Status update
     deactivate Airflow Repo
-    Note over Tests: Build ARM CI images
     deactivate Tests
 ```
 
@@ -204,9 +204,9 @@ sequenceDiagram
     opt
         Note over Tests: Generate constraints
     end
+    Note over Tests: Build ARM CI images
     Tests -->> Airflow Repo: Status update
     deactivate Airflow Repo
-    Note over Tests: Build ARM CI images
     deactivate Tests
 ```
 
@@ -287,15 +287,11 @@ sequenceDiagram
         end
     end
     Note over Tests: Generate constraints
-    opt In merge run?
-        Tests ->> Airflow Repo: Push constraints if changed
-    end
-    opt In merge run?
-        Note over Tests: Build CI Images<br>[latest]<br>Use latest constraints
-        Tests ->> GitHub Registry: Push CI Image<br>[latest]
-        Note over Tests: Build PROD Images<br>[latest]<br>Use latest 
constraints
-        Tests ->> GitHub Registry: Push PROD Image<br>[latest]
-    end
+    Tests ->> Airflow Repo: Push constraints if changed
+    Note over Tests: Build CI Images<br>[latest]<br>Use latest constraints
+    Tests ->> GitHub Registry: Push CI Image<br>[latest]
+    Note over Tests: Build PROD Images<br>[latest]<br>Use latest constraints
+    Tests ->> GitHub Registry: Push PROD Image<br>[latest]
     Tests -->> Airflow Repo: Status update
     deactivate Airflow Repo
     deactivate Tests
@@ -381,8 +377,8 @@ sequenceDiagram
     Tests ->> GitHub Registry: Push CI Image cache + latest
     Note over Tests: Build PROD Images<br>[latest]<br>Use latest constraints
     Tests ->> GitHub Registry: Push PROD Image cache + latest
+    Note over Tests: Build ARM CI images
     Tests -->> Airflow Repo: Status update
     deactivate Airflow Repo
-    Note over Tests: Build ARM CI images
     deactivate Tests
 ```

Reply via email to