Skip to content

chore: merge RemoteProver status and tests in backend#1980

Open
SantiagoPittella wants to merge 5 commits intonextfrom
santiagopittella-remote-prover-combined-status
Open

chore: merge RemoteProver status and tests in backend#1980
SantiagoPittella wants to merge 5 commits intonextfrom
santiagopittella-remote-prover-combined-status

Conversation

@SantiagoPittella
Copy link
Copy Markdown
Collaborator

closes #1908

This PR merges the Remote Prover status + test results in a same component. Previously, this were sent individually and the frontend was merging them.

Screenshot 2026-04-21 at 14 39 11

@SantiagoPittella SantiagoPittella force-pushed the santiagopittella-remote-prover-combined-status branch from 35c5902 to 9432470 Compare April 21, 2026 17:41
@bobbinth
Copy link
Copy Markdown
Contributor

Not related to this PR, but I thought about it while looking at the screenshot: could we truncate worker IDs in the remote prover card? Right now, these span 3 lines, which makes it a bit difficult to read. We could still show the full ID on hover or something like that.

@SantiagoPittella
Copy link
Copy Markdown
Collaborator Author

Not related to this PR, but I thought about it while looking at the screenshot: could we truncate worker IDs in the remote prover card? Right now, these span 3 lines, which makes it a bit difficult to read. We could still show the full ID on hover or something like that.

Yeah, we could. We can do something similar to what we do with commitments were we provide the copy button to get the full commitment. I will add it to this PR

@SantiagoPittella
Copy link
Copy Markdown
Collaborator Author

Frontend with the copy button
Screenshot 2026-04-21 at 16 32 57

@bobbinth
Copy link
Copy Markdown
Contributor

Looks great! I'd try to format it so that the status (i.e., "healthy") fits on the same line as the ID.

@SantiagoPittella
Copy link
Copy Markdown
Collaborator Author

Update it to:

image

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.

Monitor: Server-side prover merge

2 participants