Skip to content

fix: Correct feature-server image name in Helm chart#6607

Open
AdityaPatil22 wants to merge 2 commits into
feast-dev:masterfrom
AdityaPatil22:fix/helm-chart-deployment-fix
Open

fix: Correct feature-server image name in Helm chart#6607
AdityaPatil22 wants to merge 2 commits into
feast-dev:masterfrom
AdityaPatil22:fix/helm-chart-deployment-fix

Conversation

@AdityaPatil22

@AdityaPatil22 AdityaPatil22 commented Jul 16, 2026

Copy link
Copy Markdown

What this PR does / why we need it:

The Helm chart's feature-server subchart references quay.io/feastdev/feature-server-java, which does not exist in the container registry. The
correct image is quay.io/feastdev/feature-server (without the -java suffix). This causes ImagePullBackOff errors when deploying Feast via Helm.

This PR fixes the image repository name in infra/charts/feast/charts/feature-server/values.yaml.

Which issue(s) this PR fixes:

Fixes #6409

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

Verified by deploying the chart to a local Kubernetes cluster and confirming pods reach Running state instead of ImagePullBackOff.

@aniketpalu aniketpalu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

image:
# image.repository -- Docker image for Feature Server repository
repository: quay.io/feastdev/feature-server-java
repository: quay.io/feastdev/feature-server

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AdityaPatil22 I think the correct fix is either:

  • Remove/deprecate the feast umbrella chart's feature-server subchart entirely (it's stale since Java was dropped from CI in May 2025)
  • Rewrite the subchart's deployment.yaml to use feast serve like the standalone feast-feature-server chart does

You can see the Helm chart's deployment template at infra/charts/feast/charts/feature-server/templates/deployment.yaml, it has java -jar command, which won't work with python image.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ntkathole,

Thanks for the detailed analysis!

I've updated the subchart's deployment.yaml to use feast serve instead of the legacy java -jar command, aligning it with the standalone feast-feature-server chart.

I also removed the stale Java-specific templates (configmap, secret, and ingress) simplified the values.yaml and README to reflect the current Python-based configuration.

Could you please take a look when you have a chance and let me know if everything looks good, or if there are any additional changes you'd recommend?

Aditya Patil added 2 commits July 18, 2026 17:04
Signed-off-by: Aditya Patil <adpatil@redhat.com>
Signed-off-by: Aditya Patil <adpatil@redhat.com>
@AdityaPatil22
AdityaPatil22 force-pushed the fix/helm-chart-deployment-fix branch from 4a810a6 to e155575 Compare July 18, 2026 11:35
@AdityaPatil22
AdityaPatil22 requested a review from ntkathole July 18, 2026 11:51
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.

Docker images not published for versions 0.62.0 and 0.63.0 - Helm chart deployment fails with ImagePullBackOff

3 participants