potiuk commented on a change in pull request #8393: Bring back CI optimisations
URL: https://github.com/apache/airflow/pull/8393#discussion_r410036296
 
 

 ##########
 File path: .github/workflows/ci.yml
 ##########
 @@ -80,134 +80,152 @@ jobs:
     name: Build docs
     runs-on: ubuntu-latest
     env:
-      TRAVIS_JOB_NAME: "Build documentation"
       PYTHON_VERSION: 3.6
+      CI_JOB_TYPE: "Documentation"
     steps:
       - uses: actions/checkout@master
-      - name: "Build documentation"
+      - name: "Build CI image"
+        run: ./scripts/ci/ci_prepare_image_on_ci.sh
+      - name: "Build docs"
         run: ./scripts/ci/ci_docs.sh
 
   tests-p36-postgres-integrations:
-    name: "Tests [Postgres9.6][Py3.6][integrations]"
+    name: "[Pg9.6][Py3.6][integrations]"
     runs-on: ubuntu-latest
     needs: [statics, statics-tests]
     env:
-      TRAVIS_JOB_NAME: "Tests [Postgres9.6][Py3.6][integrations]"
       BACKEND: postgres
       PYTHON_VERSION: 3.6
       POSTGRES_VERSION: 9.6
       ENABLED_INTEGRATIONS: "cassandra kerberos mongo openldap rabbitmq redis"
       RUN_INTEGRATION_TESTS: all
+      CI_JOB_TYPE: "Tests"
     steps:
       - uses: actions/checkout@master
-      - name: "Tests [Postgres9.6][Py3.6][integrations]"
+      - name: "Build CI image"
+        run: ./scripts/ci/ci_prepare_image_on_ci.sh
+      - name: "Tests"
         run: ./scripts/ci/ci_run_airflow_testing.sh
 
   tests-p36-postgres-providers:
 
 Review comment:
   Actually I guess we have far less limits now for GitHub actions so rather 
than splitting the tests we can combine the tests back.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to