jscheffl commented on code in PR #53563:
URL: https://github.com/apache/airflow/pull/53563#discussion_r2316988308


##########
providers/edge3/docs/ui_plugin.rst:
##########
@@ -30,13 +30,13 @@ REST API endpoints
 
 The Edge provider adds the following REST API endpoints to the Airflow API:
 
-- ``/api/v1/edge/health``: Check that the API endpoint is deployed and active
-- ``/api/v1/edge/jobs``: Endpoints to fetch jobs for workers and report state
-- ``/api/v1/edge/logs``: Endpoint to push log chunks from workers to the 
Airflow cluster
-- ``/api/v1/edge/workers``: Endpoints to register and manage workers, report 
heartbeat
+- ``/edge_worker/v1/jobs``: Endpoints to fetch jobs for workers and report 
state
+- ``/edge_worker/v1/logs``: Endpoint to push log chunks from workers to the 
Airflow cluster
+- ``/edge_worker/v1/workers``: Endpoints to register and manage workers, 
report heartbeat
+- ``/edge_worker/v1/health``: Check that the API endpoint is deployed and 
active

Review Comment:
   I am "just" moving the swagger docs path to one level up, such that I can 
mount the path `/ui` besides the current `/v1` - for the clients it should be 
transparent as I mount the API endpoint one level up and create a subfolder in 
the API that compensates.



-- 
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