Skip to content

Conversation

jog1t
Copy link
Contributor

@jog1t jog1t commented Sep 29, 2025

Closes FRONT-793

TL;DR

Added the ability to display datacenter information for actors in the actors list.

What changed?

  • Added actorDatacenterQueryOptions to the default data provider to fetch datacenter information for actors
  • Added a new showDatacenter filter option in the actor filters context
  • Created a new Datacenter component in the actors list row to display the datacenter information
  • Extended the Actor type to include an optional datacenter field
  • Made the "showDestroyed" filter available for both "engine" and "cloud" app types

How to test?

  1. Open the actors list view
  2. Enable the "Show Actors Datacenter" filter option
  3. Verify that datacenter information appears under the actor ID for actors that have datacenter data
  4. Verify that actors without datacenter data show a "-" indicator
  5. Disable the filter and confirm the datacenter information is hidden

Why make this change?

This change improves observability by allowing users to see which datacenter each actor is running in. This information is particularly valuable in multi-datacenter deployments where understanding the physical location of actors can help with troubleshooting, performance analysis, and capacity planning.

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 30, 2025 7:52am
rivet-inspector Error Error Sep 30, 2025 7:52am
rivet-site Error Error Sep 30, 2025 7:52am

Copy link

claude bot commented Sep 29, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

@jog1t jog1t marked this pull request as ready for review September 29, 2025 20:13
Copy link

linear bot commented Sep 29, 2025

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@3018
npm i https://pkg.pr.new/rivet-dev/engine/@rivetkit/engine-runner-protocol@3018

commit: 16fa0a1

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-793

### TL;DR

Added the ability to display datacenter information for actors in the actors list.

### What changed?

- Added `actorDatacenterQueryOptions` to the default data provider to fetch datacenter information for actors
- Added a new `showDatacenter` filter option in the actor filters context
- Created a new `Datacenter` component in the actors list row to display the datacenter information
- Extended the `Actor` type to include an optional `datacenter` field
- Made the "showDestroyed" filter available for both "engine" and "cloud" app types

### How to test?

1. Open the actors list view
2. Enable the "Show Actors Datacenter" filter option
3. Verify that datacenter information appears under the actor ID for actors that have datacenter data
4. Verify that actors without datacenter data show a "-" indicator
5. Disable the filter and confirm the datacenter information is hidden

### Why make this change?

This change improves observability by allowing users to see which datacenter each actor is running in. This information is particularly valuable in multi-datacenter deployments where understanding the physical location of actors can help with troubleshooting, performance analysis, and capacity planning.
@graphite-app graphite-app bot closed this Oct 1, 2025
@graphite-app graphite-app bot deleted the 09-29-feat_add_datacenter_field_to_actors_list_row 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