Skip to content

Conversation

@bn222
Copy link
Contributor

@bn222 bn222 commented Dec 17, 2025

Add lifecycle management APIs

  • LifeCycleService: initialization and configuration
  • DeviceService: Device enumeration and VF management
  • HeartbeatService: Health monitoring and status checks

These APIs provide standardized interfaces for:

  • Platform initialization (Init RPC)
  • Device discovery and configuration (GetDevices, SetNumVfs RPCs)
  • Health monitoring (Ping RPC)

@bn222 bn222 requested a review from a team as a code owner December 17, 2025 14:34
@bn222 bn222 changed the title Add lifecycle management APIs for xPU platforms Add lifecycle management APIs Dec 17, 2025
@bn222 bn222 force-pushed the main branch 2 times, most recently from 5cf8775 to 194b1e6 Compare December 17, 2025 14:46
Make the linter pass

Signed-off-by: Balazs Nemeth <bnemeth@redhat.com>
@bn222 bn222 force-pushed the main branch 6 times, most recently from 7478306 to 3317b36 Compare December 22, 2025 16:30
@sandersms
Copy link
Contributor

The last checkin for the refactor should be removed from this PR and submitted separately as it breaks many things in the API building from the makefile.

@bn222 bn222 force-pushed the main branch 2 times, most recently from e35b1ca to 2f2d486 Compare December 30, 2025 08:17
- LifeCycleService: DPU/IPU initialization and configuration
- DeviceService: device enumeration and VF management
- HeartbeatService: health monitoring and status checks

Provides comprehensive lifecycle management capabilities
extracted from OpenShift DPU Operator for standardization
in the opi-api project.

Signed-off-by: Balazs Nemeth <bnemeth@redhat.com>
@bn222
Copy link
Contributor Author

bn222 commented Dec 30, 2025

@sandersms ready for another round

@bn222
Copy link
Contributor Author

bn222 commented Jan 7, 2026

@sandersms let me know if more changes are needed.

@wizhaoredhat
Copy link

LGTM @sandersms / @pdp2shirts could you help merge it?
We have a lot of code to contribute (1000s of lines of go code from downstream). And we need a way to move fast. WDYT?

Copy link
Contributor

@sandersms sandersms left a comment

Choose a reason for hiding this comment

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

These changes look good now.

@sandersms sandersms merged commit 0b58e48 into opiproject:main Jan 26, 2026
5 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.

3 participants