Skip to content

fix: Build feature-server image from local source instead of PyPI - #321

Merged
ntkathole merged 1 commit into
mainfrom
fix/dockerfile-source-build-main
Aug 14, 2026
Merged

ntkathole merged 1 commit into
mainfrom
fix/dockerfile-source-build-main

Conversation

@ntkathole

Copy link
Copy Markdown
Member

Summary

The Konflux Dockerfile installs feast from PyPI via requirements.txt, which means Python SDK fixes committed to the repo are not included in the built container image until a new PyPI release. This changes the build to install from local source code.

Changes

  • Dockerfile: Install from local source using SETUPTOOLS_SCM_PRETEND_VERSION instead of PyPI
  • Tekton pipeline: Changed path-context to repo root so full source tree is available as build context
  • Removed unnecessary runtime packages from Stage 2 (python3.12-devel, postgresql-devel, gcc, openblas-devel)
  • Copy non-Python data files (static, templates, snowflake scripts) for runtime
  • Create minimal feast UI build directory for runtime compatibility

Test Plan

  • Verify image builds successfully with Konflux pipeline
  • Verify feast commands work correctly in the built image

Made with Cursor

The Konflux Dockerfile installs feast from PyPI via requirements.txt,
which means Python SDK fixes committed to the repo are not included in the
built container image until a new PyPI release. This changes the build to
install from local source code, ensuring all committed fixes are present.

Changes:
- Update Dockerfile to copy and install from local source
- Update Tekton path-context from subdirectory to repo root
- Remove unnecessary runtime packages (python3.12-devel, postgresql-devel, gcc, openblas-devel)
- Copy non-Python data files (static, templates, snowflake scripts)
- Create minimal feast UI build directory for runtime compatibility

Signed-off-by: ntkathole <nikhilkathole2683@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@ntkathole

Copy link
Copy Markdown
Member Author

/build-konflux feature-server

@ntkathole
ntkathole merged commit 3f89cd2 into main Aug 14, 2026
1 check 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.

2 participants