Add region and datacenter to add_mapped_common_server_data

Summary: pzmarzly noticed this in D50738259: `ServerData` enum has members for region and DC, but `add_mapped_common_server_data` doesn't fill them, even though it has access to them via fbwhoami Reviewed By: vjt Differential Revision: D50790587 fbshipit-source-id: 738487a092192f568027354b49680c3eb6bac73f
facebookexperimental · Nov 1, 2023 · 3458ecf · 3458ecf
1 parent 2837a3c
commit 3458ecf
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/shed/scuba_sample/src/builder.rs b/shed/scuba_sample/src/builder.rs
@@ -264,9 +264,9 @@ pub enum ServerData {
     Hostname,
     /// Hostname scheme of the server
     HostnameScheme,
-    /// Datacenter Region
+    /// Datacenter Region (e.g. sweden)
     Region,
-    /// Datacenter prefix
+    /// Datacenter (e.g. lla1)
     Datacenter,
     /// Tier of the service
     Tier,