Skip to content

fix: Add UI gRPC runtime dependencies#6530

Merged
franciscojavierarceo merged 1 commit into
v0.64-branchfrom
codex/fix-ui-grpc-runtime-deps
Jun 16, 2026
Merged

fix: Add UI gRPC runtime dependencies#6530
franciscojavierarceo merged 1 commit into
v0.64-branchfrom
codex/fix-ui-grpc-runtime-deps

Conversation

@franciscojavierarceo

Copy link
Copy Markdown
Member

What changed

  • Promote grpcio, grpcio-reflection, and grpcio-health-checking into base Python package dependencies.
  • Regenerate the plain Python requirement lockfiles for 3.10, 3.11, and 3.12 so no-extra installs include those packages.
  • Add a small pyproject regression test for the UI runtime dependency set.

Why

feast ui now mounts the REST registry routes. That path imports feast.registry_server, which imports grpc, grpc_health, and grpc_reflection at module import time. In feast==0.64.1.dev6529, those packages are only installed with the grpcio extra, so a plain pip install feast==0.64.1.dev6529 fails to start feast ui with ModuleNotFoundError.

This keeps the default package install aligned with the runtime imports used by the UI.

Validation

  • Clean no-extra install from this branch with python3.12 -m venv /tmp/feast-ui-grpc-deps-check && pip install .
  • Verified imports after the no-extra install:
    • import grpc
    • from grpc_health.v1 import health
    • from grpc_reflection.v1alpha import reflection
    • from feast.registry_server import RegistryServer
    • from feast.ui_server import get_app
  • Directly executed test_ui_runtime_grpc_dependencies_are_base_dependencies
  • git diff --check

Note: local pytest collection in an ad hoc env was blocked by unrelated environment issues (minio missing without test extras, then Python 3.12 distutils missing through pytest-timeout). CI should run with the repo's normal test environment.

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>
@franciscojavierarceo franciscojavierarceo requested a review from a team as a code owner June 16, 2026 14:17
@franciscojavierarceo franciscojavierarceo merged commit b9d5ba1 into v0.64-branch Jun 16, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant