fix: Updated projects-list.json in order to display newly added projects by AdityaPatil22 · Pull Request #6657 · feast-dev/feast

AdityaPatil22 · 2026-07-29T11:37:43Z

What this PR does / why we need it:

projects-list.json was written as a static file once at server startup and never refreshed. When a user ran feast apply to create a new project while the UI was running, the new project never appeared in the UI's project list.

This PR replaces the static file write with a dynamic FastAPI route (GET /projects-list.json) that queries the registry on each request, so newly created projects appear immediately on page refresh.

Which issue(s) this PR fixes:

Fixes #6650

Checks

I've made sure the tests are passing.
My commits are signed off (git commit -s)
My PR title follows conventional commits format

Testing Strategy

Unit tests
Integration tests
Manual tests
Testing is not required for this change

Misc

Existing test_projects_list_* tests updated to hit the dynamic endpoint via TestClient instead of reading the static file from disk. New test test_projects_list_dynamic_refresh verifies that mutating the registry after get_app is reflected in subsequent requests — the core assertion for #6650.

franciscojavierarceo

Moving projects-list.json behind a route avoids freezing the response at startup, but _build_projects_list() still calls list_projects(allow_cache=True). With the default registry cache TTL of 600 seconds, a newly added project can remain absent for up to ten minutes—the exact startup-empty case this is intended to fix. Please consider an uncached read (or explicit refresh) for this low-frequency UI endpoint and add the regression test described in the PR body; the current diff changes only ui_server.py and contains no test update.

AdityaPatil22 · 2026-07-29T15:26:38Z

@franciscojavierarceo - Updated _build_projects_list to use allow_cache=False so new projects are visible immediately. Also added the regression test (test_projects_list_dynamic_refresh) and updated the existing tests to hit the dynamic endpoint via TestClient instead of reading the static file from disk.

codecov-commenter · 2026-07-30T04:26:00Z

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.46%. Comparing base (c8628eb) to head (43193f2).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #6657   +/-   ##
=======================================
  Coverage   46.45%   46.46%           
=======================================
  Files         414      414           
  Lines       50138    50145    +7     
  Branches     7173     7173           
=======================================
+ Hits        23294    23301    +7     
  Misses      25204    25204           
  Partials     1640     1640

Flag	Coverage Δ
go-feature-server	`30.58% <ø> (ø)`
python-unit	`47.77% <100.00%> (+<0.01%)`	⬆️

Files with missing lines	Coverage Δ
sdk/python/feast/ui_server.py	`25.56% <100.00%> (+0.92%)`	⬆️

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c8628eb...43193f2. Read the comment docs.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

ntkathole · 2026-07-30T04:33:20Z

@AdityaPatil22 This turns every browser refresh into a full database round-trip, completely bypassing the caching layer that Feast deliberately built.

The fix should keep allow_cache=True and let the existing TTL handle staleness. Users who need faster UI refresh can already set cache_ttl_seconds: 10 (or whatever suits them) in their config.

I think we can :

Document that users can lower cache_ttl_seconds for faster UI refresh.
Add a Refresh button in the UI that calls store.registry.refresh() (via a new POST /api/registry/refresh endpoint). This lets users explicitly invalidate the cache on demand without adding hidden performance costs on every page load.

AdityaPatil22 · 2026-07-30T10:35:09Z

@ntkathole - I have added a refresh button in the UI which lets the users invalidate the cache on demand. Please let me know if there needs to be any change in the UI for the button.

ntkathole · 2026-07-30T12:30:28Z

 | Production (frequent schema changes) | `thread` | 60 |

+### UI refresh
+


I think better place to add these docs is at docs/reference/alpha-web-ui.md

Moved the docs to docs/reference/alpha-web-ui.md file

ntkathole · 2026-07-30T12:30:55Z


    ui_dir_ref = importlib_resources.files(__spec__.parent) / "ui/build/"  # type: ignore[name-defined, arg-type]
    with importlib_resources.as_file(ui_dir_ref) as ui_dir:
-        projects_dict = _build_projects_list(store, project_id, root_path)


this change is not required and can be kept same as original

Reverted the changes to original state

ntkathole · 2026-07-30T12:33:37Z

+    def get_projects_list():
+        return _build_projects_list(store, project_id, root_path)
+
+    @app.post("/api/registry/refresh")


this endpoint should be on REST API server

Moved the endpoint from the main app to rest_app inside _setup_rest_mode, so it's now served at /api/v1/registry/refresh on the REST API server.

ntkathole · 2026-07-30T12:36:45Z

+            </EuiTitle>
+          </EuiFlexItem>
+          <EuiFlexItem grow={false}>
+            <EuiButtonIcon


This refresh button should be accessible from any page, also you can use a proper labeled button for refresh

Moved the refresh button to header bar for it to be accessible from every page.

Signed-off-by: Aditya Patil <adpatil@redhat.com>

ntkathole · 2026-07-31T03:27:22Z

+                      isLoading={refreshing}
+                      size="s"
+                    >
+                      Refresh Registry


just Refresh would work, "Refresh Registry" - too technical for end user

ntkathole · 2026-07-31T03:29:22Z

+      const parsed = ProjectsListSchema.parse(json);
+      queryClient.setQueryData("feast-projects-list", parsed);
+    } finally {
+      setRefreshing(false);


No user feedback on success or failure, Add EuiToast / EuiGlobalToastList with message like "Refresh successful/failed"

ntkathole · 2026-07-31T03:30:24Z

                  {!data && <EuiFlexItem />}

+                  <EuiFlexItem grow={false}>
+                    <EuiButton


Use EuiButtonEmpty or EuiButtonIcon with a tooltip instead of a filled button
Or use color="text" to make it less prominent than default primary

ntkathole · 2026-07-31T03:31:44Z


 Examples of custom tabs can be found in the `ui/custom-tabs` folder.
+
+## Refreshing the project list


it's not just refreshing project list, it's refreshing complete registry

ntkathole · 2026-07-31T03:33:05Z

@@ -1,6 +1,7 @@
-import React, { useState, useRef, useEffect } from "react";
+import React, { useState, useRef, useEffect, useContext } from "react";


If a user lands on the project selection page and doesn't see their project, the Refresh button in the Layout header isn't visible - they have to first pick a project, then refresh.

AdityaPatil22 requested a review from a team as a code owner July 29, 2026 11:37

franciscojavierarceo reviewed Jul 29, 2026

View reviewed changes

AdityaPatil22 requested a review from franciscojavierarceo July 29, 2026 15:26

ntkathole added the ok-to-test label Jul 30, 2026

ntkathole removed the ok-to-test label Jul 30, 2026

AdityaPatil22 force-pushed the fix/new-project-display-SQL-reg branch from 767f7f8 to aa3e43a Compare July 30, 2026 10:06

ntkathole reviewed Jul 30, 2026

View reviewed changes

Aditya Patil added 4 commits July 30, 2026 22:02

fix: Updated projects-list.json in order to display newly added projects

64a2830

Signed-off-by: Aditya Patil <adpatil@redhat.com>

fix: Bypass registry cache for projects-list endpoint

687b299

Signed-off-by: Aditya Patil <adpatil@redhat.com>

feat: Added registry refresh button in UI

1cf4b90

Signed-off-by: Aditya Patil <adpatil@redhat.com>

fix: Move UI registry refresh button to layout header

43193f2

Signed-off-by: Aditya Patil <adpatil@redhat.com>

AdityaPatil22 force-pushed the fix/new-project-display-SQL-reg branch from 35cdf51 to 43193f2 Compare July 30, 2026 16:32

AdityaPatil22 requested a review from ntkathole July 30, 2026 16:40

ntkathole reviewed Jul 31, 2026

View reviewed changes

		\| Production (frequent schema changes) \| `thread` \| 60 \|

		### UI refresh


		Examples of custom tabs can be found in the `ui/custom-tabs` folder.

		## Refreshing the project list

		@@ -1,6 +1,7 @@
		import React, { useState, useRef, useEffect } from "react";
		import React, { useState, useRef, useEffect, useContext } from "react";

Uh oh!

Conversation

AdityaPatil22 commented Jul 29, 2026

What this PR does / why we need it:

Which issue(s) this PR fixes:

Checks

Testing Strategy

Misc

Uh oh!

franciscojavierarceo left a comment

Choose a reason for hiding this comment

Uh oh!

AdityaPatil22 commented Jul 29, 2026

Uh oh!

codecov-commenter commented Jul 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

ntkathole commented Jul 30, 2026

Uh oh!

AdityaPatil22 commented Jul 30, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

codecov-commenter commented Jul 30, 2026 •

edited

Loading