Skip to content

Implemented Status Table#64

Merged
blobcode merged 5 commits into
mainfrom
feature/supervisor-status-tracking
Sep 30, 2025
Merged

Implemented Status Table#64
blobcode merged 5 commits into
mainfrom
feature/supervisor-status-tracking

Conversation

@GaminRick7

Copy link
Copy Markdown
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

@blobcode blobcode left a comment

Copy link
Copy Markdown
Collaborator

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)

Comment thread src/api.go Outdated
Comment thread src/status.go Outdated
Comment thread src/api.go Outdated
Comment thread src/util.go Outdated

@blobcode blobcode left a comment

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
Collaborator

@GaminRick7 status on this?

@GaminRick7

Copy link
Copy Markdown
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

2 participants