Skip to content

Conversation

@GaminRick7
Copy link
Collaborator

Created a status table to get supervisor info (will be updated to fetch data from heartbeat):

  • Stores supervisor info in redis hash with status, GPU type, timestamps
  • Returns JSON with supervisor details

Endpoints implemented:

/supervisors/status          # See all supervisors
/supervisors/status/{id}     # Check specific supervisor  
/supervisors                 # All supervisors (with ?active=true filter)

Testing: Added 3 dummy supervisors to the status table for testing purposes:

  • worker_amd_001 - AMD GPU, Active
  • worker_nvidia_002 - NVIDIA GPU, Active
  • worker_tt_003 - TT, Inactive

Test using the following endpoints:

/supervisors/status
/supervisors/status/worker_amd_001
/supervisors/status/worker_nvidia_002
/supervisors/status/worker_cpu_003
/supervisors
/supervisors?active=true

@GaminRick7 GaminRick7 linked an issue Sep 22, 2025 that may be closed by this pull request
Copy link
Collaborator

@blobcode blobcode left a comment

Choose a reason for hiding this comment

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

overall functionality looks good, but need unit tests (via go test)

Copy link
Collaborator

@blobcode blobcode left a comment

Choose a reason for hiding this comment

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

please make changes before merging, otherwise lgtm

@blobcode
Copy link
Collaborator

@GaminRick7 status on this?

@GaminRick7
Copy link
Collaborator Author

Adding the changes now.

@GaminRick7 GaminRick7 requested a review from blobcode September 30, 2025 00:06
@blobcode blobcode merged commit c2383be into main Sep 30, 2025
1 check passed
@blobcode blobcode deleted the feature/supervisor-status-tracking branch September 30, 2025 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Supervisor Observability] Supervisor Status Tracking

3 participants