Skip to content

Conversation

@Vishvajeet590
Copy link
Contributor

@Vishvajeet590 Vishvajeet590 commented Jan 6, 2026

Description

This PR adds support for configurable liveness and readiness probes in Merlin model deployments, allowing users to customize probe parameters through both the UI and Python SDK. This enhancement provides users with fine-grained control over health check behavior for their model deployments

Modifications

Sending the Liveness and Readiness probes config values from the UI.

Backend API changes

Extended ResourceRequest model to support two new optional fields:

  • liveness_probe
  • readiness_probe

Updated deployment templating so provided probes are rendered into Kubernetes resources

Relevant areas:

  • api/service/deployment_service.go
  • api/cluster/resource/templater.go

UI changes

Added UI controls to configure liveness & readiness probe parameters.
Updated deployment details view to display saved probe configs after deployment (like CPU limits).

Python SDK

Added liveness and readiness probe configuration support to the Python SDK, enabling programmatic probe customization

Screenshot of Ui changes

image

@Vishvajeet590 Vishvajeet590 added the enhancement New feature or request label Jan 9, 2026
Copy link
Contributor

@anantadwi13 anantadwi13 left a comment

Choose a reason for hiding this comment

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

lgtm, but i have 1 question about the docker file

@anantadwi13
Copy link
Contributor

thanks for the changes

@anantadwi13
Copy link
Contributor

anyway, could you attach some UI previews of this probe config? thanks

@Vishvajeet590 Vishvajeet590 enabled auto-merge (squash) January 19, 2026 22:41
@Vishvajeet590 Vishvajeet590 merged commit 9be3a76 into main Jan 20, 2026
32 checks passed
@Vishvajeet590 Vishvajeet590 deleted the probe-expose branch January 20, 2026 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants