Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-11-11 Thread via GitHub


bbovenzi commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2468393258

   We did resolve this on slack. Manually building the UI once with `pnpm 
install && pnpm build` created the `/dist` directory necessary for fastapi to 
host the UI's html file.


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-11-08 Thread via GitHub


LefterisXefteris commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2465706343

   this is what i get at the log:
   
   172.18.0.1:57800 - "GET /webapp HTTP/1.1" 307
   172.18.0.1:57800 - "GET /webapp/ HTTP/1.1" 500
   
   Key Error:
   jinja2.exceptions.TemplateNotFound: /index.html
   
   Location:
   File "/opt/airflow/airflow/api_fastapi/core_api/app.py", line 76, in webapp
   return templates.TemplateResponse("/index.html", {"request": request}, 
media_type="text/html")


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-11-08 Thread via GitHub


bbovenzi commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2465700130

   And then looking at your logs in the terminal running breeze? does it 
explain the internal error at all?
   


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-11-08 Thread via GitHub


LefterisXefteris commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2465693089

   When I navigate to /docs, it works perfectly. However, when I try to access 
http://localhost:29091/webapp, I get an Internal Server Error.


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-11-08 Thread via GitHub


bbovenzi commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2465678001

   Wait, have you navigated to `http://localhost:29091/webapp`?


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-11-08 Thread via GitHub


LefterisXefteris commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2465471328

   
   
   When running Airflow in the Breeze development environment, there's an 
inconsistency between the webserver and FastAPI services accessibility:
   
   ### Working
   - Legacy UI (webserver) is accessible at `http://localhost:28080`
   - Successfully authenticating with credentials
   - Webserver service starts correctly with `airflow webserver` command
- API documentation IS accessible at http://localhost:29091/docs`
   
   ### Not Working
   - FastAPI endpoints return 404 responses when accessing through 
`http://localhost:29091`
   - Service starts without errors using `airflow fastapi-api` command
   
   
   ### Environment Details
   - Using Breeze development environment
   - Python virtual environment configured
   - PostgreSQL backend
   - Default authentication configured
   
   ### Questions
   1. Are additional development dependencies required? Specifically:
  - `pip install -e ".[devel-all]"`
  - Specific provider packages
  - Hatch build system dependencies
   
   ### Attempted Steps
   1. Started services via Breeze environment
   2. Started FastAPI service with `airflow fastapi-api`
   
   ### Expected Behavior
   FastAPI endpoints should be accessible at `http://localhost:29091with proper 
authentication.
   
   ### Logs
   404 - not found
   
   


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-11-08 Thread via GitHub


bbovenzi commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2464881780

   When you run `breeze start-airflow` the other services are running but not 
the FastAPI one. Is that right?
   
   If so, can you try to if there are any errors in the terminal? You can also 
just run fastapi by running `breeze` and then inside of breeze running `airflow 
fastapi-api`
   


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-11-07 Thread via GitHub


LefterisXefteris commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2463337647

   Hello @bbovenzi !
   I've set up the new development environment following the instructions in 
`dev/breeze/README.md`. Current status:
   
   ✅ Successfully implemented:
   - Breeze installation and configuration using the new UV-based workflow
   - Created UV virtual environment and synced dependencies from 
`dev/breeze/uv.lock`
   - Legacy UI accessible at :28080
   - Postgres backend running
   
   ❌ Issue:
   Receiving 404 Not Found when attempting to access the FastAPI UI at 
`localhost:29091`. 
   
   Current setup:
   ```bash
   # Environment
   breeze start-airflow --dev-mode AND
   breeze start-airflow --dev-mode --python 3.10 --use-uv --backend postgres
   
   # Running containers show proper port mapping
   $ docker ps
   CONTAINER ID   IMAGE  PORTS  


   d16ac27f695e   ghcr.io/apache/airflow/main/ci/python3.10:latest   
0.0.0.0:29091->9091/tcp


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-11-04 Thread via GitHub


LefterisXefteris commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2455594338

   Hello! I'm working on this issue and I can now dedicate time to complete it. 
I notice you mentioned a modern web application, but when I run the development 
servers (frontend with yarn dev in airflow/www/ and backend with airflow 
webserver), I'm only seeing the old web interface with the navigation bar at 
the top. Could you please guide me on how to access the modern web application 
interface? I want to ensure I'm working on the correct UI version for this 
implementation.


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-11-04 Thread via GitHub


bbovenzi commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2455614603

   Make sure to pull down the latest from main. I just did some big changes to 
the UI on Friday 


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-11-04 Thread via GitHub


bbovenzi commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2455610970

   The new UI code is in `airflow/ui` and you only have to run `breeze 
start-airflow --dev-mode` and the new UI will be at localhost:29091 with hot 
reload already working


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-10-23 Thread via GitHub


LefterisXefteris commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2433379243

   Thank you very much!


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-10-23 Thread via GitHub


bbovenzi commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2433377030

   @LefterisXefteris go for it


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] AIP-38 | Add Pool Summary stats to Dashboard [airflow]

2024-10-23 Thread via GitHub


LefterisXefteris commented on issue #43328:
URL: https://github.com/apache/airflow/issues/43328#issuecomment-2433356810

   Hello @bbovenzi !
   I would like to work on this issue, if that's okay.
   
   Kind regards,
   Lefteris


-- 
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-unsubscr...@airflow.apache.org

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