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

   ## Summary
   - The e2e test `conftest.py` hardcoded `ghcr.io/apache/airflow/main/` as the 
only prefix
     to skip `docker compose pull`. On release branches (e.g. `v3-2-test`), the 
image name
     is `ghcr.io/apache/airflow/v3-2-test/prod/python3.10`, which didn't match, 
so
     `pull=True` was set and `docker compose pull` tried to fetch the image 
from GHCR —
     where it doesn't exist.
   - The image is already loaded locally from CI artifacts via the 
`prepare_breeze_and_image`
     step, so pulling is unnecessary for any `ghcr.io/apache/airflow/` image. 
This matches
     how the UI e2e tests work — they use `docker compose up -d` directly 
without pulling.
   - Broadened the prefix check from `ghcr.io/apache/airflow/main/` to
     `ghcr.io/apache/airflow/` so all branches (main, v3-2-test, future release 
branches)
     skip the pull and use the locally loaded image.
   
   
   Failing test in v3-2-test 
https://github.com/apache/airflow/actions/runs/23756910688/job/69224619173?pr=64469
   
   - [X] Yes (please specify the tool below)
   Claude
   <!--
   Generated-by: [Tool Name] following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   -->
   
   ---
   
   * Read the **[Pull Request 
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)**
 for more information. Note: commit author/co-author name and email in commits 
become permanently public when merged.
   * For fundamental code changes, an Airflow Improvement Proposal 
([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals))
 is needed.
   * When adding dependency, check compliance with the [ASF 3rd Party License 
Policy](https://www.apache.org/legal/resolved.html#category-x).
   * For significant user-facing changes create newsfragment: 
`{pr_number}.significant.rst`, in 
[airflow-core/newsfragments](https://github.com/apache/airflow/tree/main/airflow-core/newsfragments).
 You can add this file in a follow-up commit after the PR is created so you 
know the PR number.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to