Skip to content

Conversation

jog1t
Copy link
Contributor

@jog1t jog1t commented Sep 29, 2025

Closes FRONT-792

TL;DR

Added a Datacenter column to the Runners Table to display runner datacenter information.

What changed?

  • Added a new "Datacenter" column header to the Runners Table
  • Added a corresponding table cell in the RowSkeleton component to maintain proper layout when loading
  • Added a table cell to display the runner's datacenter value in the Row component

How to test?

  1. Navigate to the Runners Table view
  2. Verify that the "Datacenter" column appears between the "Name" and "Slots" columns
  3. Confirm that each runner's datacenter information is correctly displayed in the new column
  4. Check that the loading skeleton properly includes the new column

Why make this change?

This change improves visibility of runner datacenter information, making it easier for users to identify where runners are located without having to drill down into individual runner details. This information is valuable for managing distributed runner deployments across multiple datacenters.

Copy link

vercel bot commented Sep 29, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
rivet-cloud Ready Ready Preview Comment Sep 29, 2025 8:04pm
rivet-inspector Error Error Sep 29, 2025 8:04pm
rivet-site Error Error Sep 29, 2025 8:04pm

This was referenced Sep 29, 2025
Copy link

claude bot commented Sep 29, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

Copy link
Contributor Author

jog1t commented Sep 29, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


How to use the Graphite Merge Queue

Add the label merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link

pkg-pr-new bot commented Sep 29, 2025

Open in StackBlitz

npm i https://pkg.pr.new/rivet-dev/engine/@rivetkit/engine-runner@3016
npm i https://pkg.pr.new/rivet-dev/engine/@rivetkit/engine-runner-protocol@3016

commit: 68298c3

Copy link

linear bot commented Sep 29, 2025

@jog1t jog1t marked this pull request as ready for review September 29, 2025 19:53
Copy link
Contributor

graphite-app bot commented Sep 29, 2025

Graphite Automations

"Test" took an action on this PR • (09/29/25)

1 assignee was added to this PR based on Kacper Wojciechowski's automation.

Copy link
Contributor

graphite-app bot commented Oct 1, 2025

Merge activity

  • Oct 1, 4:49 AM UTC: NathanFlurry added this pull request to the Graphite merge queue.
  • Oct 1, 4:50 AM UTC: CI is running for this pull request on a draft pull request (#3026) due to your merge queue CI optimization settings.
  • Oct 1, 4:51 AM UTC: Merged by the Graphite merge queue via draft PR: #3026.

graphite-app bot pushed a commit that referenced this pull request Oct 1, 2025
Closes FRONT-792

### TL;DR

Added a Datacenter column to the Runners Table to display runner datacenter information.

### What changed?

- Added a new "Datacenter" column header to the Runners Table
- Added a corresponding table cell in the RowSkeleton component to maintain proper layout when loading
- Added a table cell to display the runner's datacenter value in the Row component

### How to test?

1. Navigate to the Runners Table view
2. Verify that the "Datacenter" column appears between the "Name" and "Slots" columns
3. Confirm that each runner's datacenter information is correctly displayed in the new column
4. Check that the loading skeleton properly includes the new column

### Why make this change?

This change improves visibility of runner datacenter information, making it easier for users to identify where runners are located without having to drill down into individual runner details. This information is valuable for managing distributed runner deployments across multiple datacenters.
@graphite-app graphite-app bot closed this Oct 1, 2025
@graphite-app graphite-app bot deleted the 09-29-feat_add_datacenter_field_to_runners_table branch October 1, 2025 04:51
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.

2 participants