From 2dea0f961abe7bdff0ff682837b9f35ed73ac1ee Mon Sep 17 00:00:00 2001 From: Subham Soni Date: Mon, 13 Apr 2026 01:20:43 -0700 Subject: [PATCH] Standardize XProf hostname resolution logic PiperOrigin-RevId: 898840873 --- tsl/profiler/protobuf/profiler_options.proto | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tsl/profiler/protobuf/profiler_options.proto b/tsl/profiler/protobuf/profiler_options.proto index 04c46af2d..244d25fd5 100644 --- a/tsl/profiler/protobuf/profiler_options.proto +++ b/tsl/profiler/protobuf/profiler_options.proto @@ -101,8 +101,12 @@ message ProfileOptions { // under the repository path. If not set, the current timestamp will be used. string session_id = 14; - // If set, this hostname will be used to name the profile file. - string override_hostname = 15; + // Deprecated: This field is now treated as a boolean toggle. If set + // (non-empty), the profiler ignores its value and uses the system hostname + // (tsl::port::Hostname()) to name the profile file. Otherwise, it falls back + // to request.host_name(). For remote mode, use advanced_configuration with + // key "override_hostnames" (string) instead. + string override_hostname = 15 [deprecated = true]; } // Options for remote profiler session manager.