Skip to content

Commit 938eafa

Browse files
thomasywangmeta-codesync[bot]
authored andcommitted
Log to monarch_executions directly
Summary: Each process only logs to monarch_executions once at the beginning of the execution so there is no need to add a scuba client that logs to this table into our tracing subscriber Reviewed By: vidhyav Differential Revision: D87664117
1 parent f570837 commit 938eafa

File tree

1 file changed

+12
-14
lines changed
  • hyperactor_telemetry/src

1 file changed

+12
-14
lines changed

hyperactor_telemetry/src/lib.rs

Lines changed: 12 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -634,20 +634,18 @@ pub fn initialize_logging_with_log_prefix(
634634
tracing::debug!("logging already initialized for this process: {}", err);
635635
}
636636
let exec_id = env::execution_id();
637-
tracing::info!(
638-
target: "execution",
639-
execution_id = exec_id,
640-
environment = %Env::current(),
641-
args = ?std::env::args(),
642-
build_mode = build_info::BuildInfo::get_build_mode(),
643-
compiler = build_info::BuildInfo::get_compiler(),
644-
compiler_version = build_info::BuildInfo::get_compiler_version(),
645-
buck_rule = build_info::BuildInfo::get_rule(),
646-
package_name = build_info::BuildInfo::get_package_name(),
647-
package_release = build_info::BuildInfo::get_package_release(),
648-
upstream_revision = build_info::BuildInfo::get_upstream_revision(),
649-
revision = build_info::BuildInfo::get_revision(),
650-
"logging_initialized"
637+
meta::log_execution_event(
638+
&exec_id,
639+
&Env::current().to_string(),
640+
std::env::args().collect(),
641+
build_info::BuildInfo::get_build_mode(),
642+
build_info::BuildInfo::get_compiler(),
643+
build_info::BuildInfo::get_compiler_version(),
644+
build_info::BuildInfo::get_rule(),
645+
build_info::BuildInfo::get_package_name(),
646+
build_info::BuildInfo::get_package_release(),
647+
build_info::BuildInfo::get_upstream_revision(),
648+
build_info::BuildInfo::get_revision(),
651649
);
652650

653651
if !is_layer_disabled(DISABLE_OTEL_METRICS) {

0 commit comments

Comments
 (0)