From 145e72baa0cf8a0e19b36bc5877121ba4d46c106 Mon Sep 17 00:00:00 2001
From: Caleb Schoepp <caleb.schoepp@fermyon.com>
Date: Fri, 6 Sep 2024 15:16:56 -0600
Subject: [PATCH] Move instruments up a level in factor-variables

Signed-off-by: Caleb Schoepp <caleb.schoepp@fermyon.com>
---
 crates/factor-variables/src/host.rs                     | 2 ++
 crates/factor-variables/src/spin_cli/azure_key_vault.rs | 2 +-
 crates/factor-variables/src/spin_cli/env.rs             | 2 +-
 crates/factor-variables/src/spin_cli/vault.rs           | 2 +-
 4 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/crates/factor-variables/src/host.rs b/crates/factor-variables/src/host.rs
index aa1d70f34f..bc30fe0a79 100644
--- a/crates/factor-variables/src/host.rs
+++ b/crates/factor-variables/src/host.rs
@@ -1,10 +1,12 @@
 use spin_factors::anyhow;
 use spin_world::{async_trait, v1, v2::variables};
+use tracing::{instrument, Level};
 
 use crate::InstanceState;
 
 #[async_trait]
 impl variables::Host for InstanceState {
+    #[instrument(name = "spin_variables.get", skip(self), err(level = Level::INFO), fields(otel.kind = "client"))]
     async fn get(&mut self, key: String) -> Result<String, variables::Error> {
         let key = spin_expressions::Key::new(&key).map_err(expressions_to_variables_err)?;
         self.expression_resolver
diff --git a/crates/factor-variables/src/spin_cli/azure_key_vault.rs b/crates/factor-variables/src/spin_cli/azure_key_vault.rs
index 54fe15bbcc..1ad1df041a 100644
--- a/crates/factor-variables/src/spin_cli/azure_key_vault.rs
+++ b/crates/factor-variables/src/spin_cli/azure_key_vault.rs
@@ -134,7 +134,7 @@ impl AzureKeyVaultProvider {
 
 #[async_trait]
 impl Provider for AzureKeyVaultProvider {
-    #[instrument(name = "spin_variables.get_from_azure_key_vault", skip(self), err(level = Level::INFO), fields(otel.kind = "client"))]
+    #[instrument(name = "spin_variables.get_from_azure_key_vault", level = Level::DEBUG, skip(self), err(level = Level::INFO), fields(otel.kind = "client"))]
     async fn get(&self, key: &Key) -> anyhow::Result<Option<String>> {
         let secret = self
             .secret_client
diff --git a/crates/factor-variables/src/spin_cli/env.rs b/crates/factor-variables/src/spin_cli/env.rs
index 781140a1b8..384e982008 100644
--- a/crates/factor-variables/src/spin_cli/env.rs
+++ b/crates/factor-variables/src/spin_cli/env.rs
@@ -128,7 +128,7 @@ fn load_dotenv(dotenv_path: &Path) -> anyhow::Result<HashMap<String, String>> {
 
 #[async_trait]
 impl Provider for EnvVariablesProvider {
-    #[instrument(name = "spin_variables.get_from_env", skip(self), err(level = Level::INFO))]
+    #[instrument(name = "spin_variables.get_from_env", level = Level::DEBUG, skip(self), err(level = Level::INFO))]
     async fn get(&self, key: &Key) -> anyhow::Result<Option<String>> {
         tokio::task::block_in_place(|| self.get_sync(key))
     }
diff --git a/crates/factor-variables/src/spin_cli/vault.rs b/crates/factor-variables/src/spin_cli/vault.rs
index 9008ce3c5f..0bd7e8ed6c 100644
--- a/crates/factor-variables/src/spin_cli/vault.rs
+++ b/crates/factor-variables/src/spin_cli/vault.rs
@@ -27,7 +27,7 @@ pub struct VaultVariablesProvider {
 
 #[async_trait]
 impl Provider for VaultVariablesProvider {
-    #[instrument(name = "spin_variables.get_from_vault", skip(self), err(level = Level::INFO), fields(otel.kind = "client"))]
+    #[instrument(name = "spin_variables.get_from_vault", level = Level::DEBUG, skip(self), err(level = Level::INFO), fields(otel.kind = "client"))]
     async fn get(&self, key: &Key) -> anyhow::Result<Option<String>> {
         let client = VaultClient::new(
             VaultClientSettingsBuilder::default()