Skip to content

Add InferenceService lifecycle state#612

Open
kartikkhurana1898 wants to merge 2 commits into
ome-projects:mainfrom
kartikkhurana1898:add-inferenceservice-lifecycle-state
Open

Add InferenceService lifecycle state#612
kartikkhurana1898 wants to merge 2 commits into
ome-projects:mainfrom
kartikkhurana1898:add-inferenceservice-lifecycle-state

Conversation

@kartikkhurana1898

Copy link
Copy Markdown

Summary

Adds a high-level status.lifecycleState field to InferenceService.

The field is derived from existing Kubernetes/OME status signals:

  • deletion timestamp
  • Ready condition
  • status.modelStatus.transitionStatus
  • status.modelStatus.lastFailureInfo
  • component rollout revisions
  • previous lifecycleState

Motivation

InferenceService already exposes detailed conditions and model transition state, but integrations often need a simple lifecycle summary without reimplementing the condition/status interpretation logic. This adds that summary while keeping the source of truth in existing OME status fields.

State Mapping

  • deleting object -> DELETING
  • hard failure (InvalidSpec, BlockedByFailedLoad, or lastFailureInfo) -> FAILED
  • ready service -> READY
  • progress during initial reconciliation -> CREATING
  • progress after an established state -> UPDATING
  • previously established service with Ready=False and no progress -> FAILED
  • fallback -> CREATING

Testing

  • go test ./pkg/controller/v1beta1/inferenceservice/status ./pkg/apis/ome/v1beta1
  • go test ./pkg/controller/v1beta1/inferenceservice -run '^$'
  • make manifests
  • make generate

@github-actions github-actions Bot added api API/Types changes in pkg/apis helm Helm chart changes crd CRD definition changes inferenceservice InferenceService controller changes controller Controller changes tests Test changes config Configuration changes labels May 14, 2026
ankrovv pushed a commit to ankrovv/ome that referenced this pull request Jun 24, 2026
…-fp8-dynamic InferenceService

Merge in GENAICORE/ome from llama3.3-70bfp8_runtime to main

Squashed commit of the following:

commit d6956eb3884d5cd89feacf3a00f9429d6b892e95
Author: Sachin Goyal <sachin.g.goyal@oracle.com>
Date:   Mon Jun 23 10:28:50 2025 +0530

    fix llama3-3-70b-instruct-fp8-dynamic isvc

commit a7ddb3e19e5a0524ba81cf46757f3ddbc90772d7
Author: SAADALAM <saad.alam@oracle.com>
Date:   Mon Jun 9 19:31:08 2025 +0530

    removed pull-request from releaseBranch

commit bec4a841a9b299d35442aaa25f77506de6c407ef
Author: SAADALAM <saad.alam@oracle.com>
Date:   Mon Jun 9 16:54:08 2025 +0530

    Added the llama 3.3-70b fp8 ondemand runtime
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api API/Types changes in pkg/apis config Configuration changes controller Controller changes crd CRD definition changes helm Helm chart changes inferenceservice InferenceService controller changes tests Test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant