-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Description
Expected Behavior
Feast UI should work when it is served behind a proxy, on a subpath e.g. /feast-ui
Current Behavior
Parts of the feast UI works behind a subpath, but not entirely (nothing is displayed, just the feast logo with a "404" text - refer to screenshot). No requests in the network tab of the web browser are hitting 404.
Steps to reproduce
Serve feast UI as you would e.g. feature_store.serve_ui(), optionally passing in the root_path parameter (it does not help).
Set up an nginx pod with the following configuration (i.e. the nginx pod should have /etc/nginx/conf.d/default.conf with the following contents - dummy_project is the project name, and http://feast-ui-service:8080 is where the feast UI can be accessed from your nginx pod / container):
server {
listen 80 default_server;
location = /feast-ui/ {
rewrite (.*) /feast-ui/p/dummy_project permanent;
}
location /feast-ui/ {
proxy_pass http://feast-ui-service:8080/;
}
location / {
proxy_pass http://feast-ui-service:8080/;
}
}
This configuration works on localhost when nginx can listen on the root path /. However, note that the URL after all the redirects is wrong (it does not have the prefix).
- The first block is required to force a redirect to the
/p/{project_name}. Without this, the page will display 404 as above. - The second block is required to strip away
/feast-uiso the UI app does not receive that path that it is not aware of - The third block is a trick to make this setup work in a local environment, because the app itself will redirect the user back to
/p/dummy_project(without the prefix), which we then proxy into the feast UI app. However, in an actual environment, this setup does not work, because when the url does not contain the/feast-uiprefix, the ingress will not route it to the nginx pod, so the nginx pod cannot proxy the connection to the right place.
Ideally, if the feast ui app is capable of being served on a subpath, only the second location block should be required in the nginx configuration. The first and third location blocks are workarounds.
Specifications
- Version: 0.29.0
Possible Solution
The app should redirect to relative and not absolute paths
