Skip to content

Fix artifacts path for builds.#48431

Merged
ashb merged 1 commit intoapache:mainfrom
tirkarthi:add-artifacts
Mar 27, 2025
Merged

Fix artifacts path for builds.#48431
ashb merged 1 commit intoapache:mainfrom
tirkarthi:add-artifacts

Conversation

@tirkarthi
Copy link
Contributor

It seems artifacts value is missing "src" in their path due to which the artifacts like UI build files are not getting included in the wheel builds. "src" is present in excluded list. The UI dist folders were included in beta4 and not in main. With the PR they are included.

Locally using main

unzip -l dist/apache_airflow_core-3.0.0.dev0-py3-none-any.whl | grep -i 'ui/dist'

After PR

      510  2020-02-02 00:00   airflow/api_fastapi/auth/managers/simple/ui/dist/index.html
      131  2020-02-02 00:00   airflow/api_fastapi/auth/managers/simple/ui/dist/.vite/manifest.json
   633445  2020-02-02 00:00   airflow/api_fastapi/auth/managers/simple/ui/dist/assets/index-D-Eul43E.js
      512  2020-02-02 00:00   airflow/ui/dist/index.html
     1201  2020-02-02 00:00   airflow/ui/dist/pin_32.png
      451  2020-02-02 00:00   airflow/ui/dist/.vite/manifest.json
     1472  2020-02-02 00:00   airflow/ui/dist/assets/FailedLogs-u5-_VlqR.js
  4122748  2020-02-02 00:00   airflow/ui/dist/assets/index-BVSsJR78.js

Beta 4 wheel file

unzip -l  ~/Downloads/apache_airflow-3.0.0b4-py3-none-any.whl | grep -i 'ui/dist'
      462  2025-03-20 17:49   airflow/api_fastapi/auth/managers/simple/ui/dist/index.html
      131  2025-03-20 17:49   airflow/api_fastapi/auth/managers/simple/ui/dist/.vite/manifest.json
   568153  2025-03-20 17:49   airflow/api_fastapi/auth/managers/simple/ui/dist/assets/index-DnXVQYs3.js
      516  2025-03-20 17:49   airflow/ui/dist/index.html
     1201  2025-03-20 17:49   airflow/ui/dist/pin_32.png
      451  2025-03-20 17:49   airflow/ui/dist/.vite/manifest.json
     1472  2025-03-20 17:49   airflow/ui/dist/assets/FailedLogs-BhhMEN0w.js
  4111255  2025-03-20 17:49   airflow/ui/dist/assets/index-B9H4E3td.js

@ashb ashb merged commit 15bf565 into apache:main Mar 27, 2025
40 checks passed
@ashb
Copy link
Member

ashb commented Mar 27, 2025

I was possibly a bit quick on merging this, I'm not sure it fixes the problem

pankajkoti pushed a commit to astronomer/airflow that referenced this pull request Mar 28, 2025
shubham-pyc pushed a commit to shubham-pyc/airflow that referenced this pull request Apr 2, 2025
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Apr 4, 2025
simonprydden pushed a commit to simonprydden/airflow that referenced this pull request Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants