Title: fix: Build embedded UI from local source by franciscojavierarceo · Pull Request #6526 · feast-dev/feast · GitHub
Open Graph Title: fix: Build embedded UI from local source by franciscojavierarceo · Pull Request #6526 · feast-dev/feast
X Title: fix: Build embedded UI from local source by franciscojavierarceo · Pull Request #6526 · feast-dev/feast
Description: What changed Backport #6525 to v0.64-branch. Update make build-ui to build the checked-out ui/ source and copy the generated build into sdk/python/feast/ui/build. Keep build-ui-local as an alias for the same build path. Backport the explicit MCP enablement guard from master so mocked config does not accidentally mount MCP routes in unit tests. Backport the master REST route registration test simplification so the test verifies router inclusion directly instead of depending on mocked RestRegistryServer construction side effects. Backport the master allowlist comment for the OpenLineage placeholder API key so detect-secrets --all-files passes on the release branch. Why feast ui packages the embedded React app from sdk/python/feast/ui/build. The previous release build target pulled @feast-dev/feast-ui --latest from npm, but npm's latest published package is still 0.57.0. That stale UI treats registryPath as a monolithic registry blob URL, so with the newer Python server generating /api/v1, it requests GET /api/v1 and receives a 404. Building the embedded UI from the repo-local ui/ source keeps the frontend bundle aligned with the REST-backed Python UI server for the 0.64 release branch. The test-only backports are included because v0.64-branch unit tests use a MagicMock config. Without requiring mcp.enabled is True, the mock is truthy and the server mounts MCP routes. After that guard, the route-path assertion still depends on mocked server construction, so this PR also matches master by verifying register_all_routes directly. The OpenLineage manifest change matches master; it is a placeholder value, not a real secret. Fixes #6519. Validation make -n build-ui python3 -m py_compile sdk/python/tests/unit/api/test_api_rest_registry_server.py sdk/python/feast/api/registry/rest/rest_registry_server.py I could not run the full UI build locally because yarn is not installed in this shell. I also could not run the targeted Python test locally because uv is not installed in this shell, and I could not run pre-commit locally because pre-commit is not installed in this shell.
Open Graph Description: What changed Backport #6525 to v0.64-branch. Update make build-ui to build the checked-out ui/ source and copy the generated build into sdk/python/feast/ui/build. Keep build-ui-local as an alias f...
X Description: What changed Backport #6525 to v0.64-branch. Update make build-ui to build the checked-out ui/ source and copy the generated build into sdk/python/feast/ui/build. Keep build-ui-local as an alias f...
Opengraph URL: https://github.com/feast-dev/feast/pull/6526
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:056d8f81-1c8a-a361-b00b-ad998252295b |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | E922:B6D9A:367EA0:495FCC:6A525D86 |
| html-safe-nonce | 6295a29a6c64047b37bfb4dab1aa5aed04c0acd3ea131ef44f3f5f565c8ad76f |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFOTIyOkI2RDlBOjM2N0VBMDo0OTVGQ0M6NkE1MjVEODYiLCJ2aXNpdG9yX2lkIjoiNDc5ODgyMjcyNzQ1OTk1NDA1NCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 53ab9ecb04bbc08abb808e5ffd6368a05f96b49ae2f184904b5fb595f91613cc |
| hovercard-subject-tag | pull_request:3870834130 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/feast-dev/feast/pull/6526/files |
| twitter:image | https://avatars.githubusercontent.com/u/4163062?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/4163062?s=400&v=4 |
| og:image:alt | What changed Backport #6525 to v0.64-branch. Update make build-ui to build the checked-out ui/ source and copy the generated build into sdk/python/feast/ui/build. Keep build-ui-local as an alias f... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | b9a586c06a05a7a86fc7e3f4dbd03e42f6869085879aa184aa6369456dbd50fb |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/feast-dev/feast git https://github.com/feast-dev/feast.git |
| octolytics-dimension-user_id | 57027613 |
| octolytics-dimension-user_login | feast-dev |
| octolytics-dimension-repository_id | 161133770 |
| octolytics-dimension-repository_nwo | feast-dev/feast |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 161133770 |
| octolytics-dimension-repository_network_root_nwo | feast-dev/feast |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 7aed05249554b889eb33d002851a973eebcc7e91 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width