Skip to content

Commit dc91d79

Browse files
feat: Marked some logs as DEBUG (#105)
* Marked some logs as DEBUG * Bumped patch version to 0.49.1 * Enriched adapter related usage logs - marked them debug * Bumped minor version to 0.50.0
1 parent a8c2fd2 commit dc91d79

File tree

6 files changed

+40
-14
lines changed

6 files changed

+40
-14
lines changed

src/unstract/sdk/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
__version__ = "0.49.0"
1+
__version__ = "0.50.0"
22

33

44
def get_sdk_version():

src/unstract/sdk/adapter.py

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
from requests.exceptions import ConnectionError, HTTPError
66

77
from unstract.sdk.adapters.utils import AdapterUtils
8-
from unstract.sdk.constants import AdapterKeys, ToolEnv
8+
from unstract.sdk.constants import AdapterKeys, LogLevel, ToolEnv
99
from unstract.sdk.exceptions import SdkError
1010
from unstract.sdk.helper import SdkHelper
1111
from unstract.sdk.platform import PlatformBase
@@ -62,10 +62,15 @@ def _get_adapter_configuration(
6262
response.raise_for_status()
6363
adapter_data: dict[str, Any] = response.json()
6464

65-
# TODO: Print config after redacting sensitive information
65+
# Removing name and type to avoid migration for already indexed records
66+
adapter_name = adapter_data.pop("adapter_name", "")
67+
adapter_type = adapter_data.pop("adapter_type", "")
68+
provider = adapter_data.get("adapter_id", "").split("|")[0]
69+
# TODO: Print metadata after redacting sensitive information
6670
self.tool.stream_log(
67-
"Successfully retrieved config "
68-
f"for adapter instance {adapter_instance_id}"
71+
f"Retrieved config for '{adapter_instance_id}', type: "
72+
f"'{adapter_type}', provider: '{provider}', name: '{adapter_name}'",
73+
level=LogLevel.DEBUG,
6974
)
7075
except ConnectionError:
7176
raise SdkError(
@@ -110,7 +115,8 @@ def get_adapter_config(
110115
platform_port = tool.get_env_or_die(ToolEnv.PLATFORM_PORT)
111116

112117
tool.stream_log(
113-
f"Connecting to DB and getting table metadata for {adapter_instance_id}"
118+
f"Retrieving config from DB for '{adapter_instance_id}'",
119+
level=LogLevel.DEBUG,
114120
)
115121
tool_adapter = ToolAdapter(
116122
tool=tool,

src/unstract/sdk/audit.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,9 @@ def push_usage_data(
102102
level=LogLevel.ERROR,
103103
)
104104
else:
105-
self.stream_log("Successfully pushed usage details")
105+
self.stream_log(
106+
f"Successfully pushed usage details, {data}", level=LogLevel.DEBUG
107+
)
106108

107109
except requests.RequestException as e:
108110
self.stream_log(
@@ -152,7 +154,9 @@ def push_page_usage_data(
152154
level=LogLevel.ERROR,
153155
)
154156
else:
155-
self.stream_log("Successfully pushed page usage details")
157+
self.stream_log(
158+
"Successfully pushed page usage details", level=LogLevel.DEBUG
159+
)
156160

157161
except requests.RequestException as e:
158162
self.stream_log(

src/unstract/sdk/platform.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,7 @@ def __init__(
3030
- PLATFORM_SERVICE_API_KEY environment variable is required.
3131
"""
3232
self.tool = tool
33-
self.base_url = SdkHelper.get_platform_base_url(
34-
platform_host, platform_port
35-
)
33+
self.base_url = SdkHelper.get_platform_base_url(platform_host, platform_port)
3634
self.bearer_token = tool.get_env_or_die(ToolEnv.PLATFORM_API_KEY)
3735

3836

@@ -78,5 +76,4 @@ def get_platform_details(self) -> Optional[dict[str, Any]]:
7876
return None
7977
else:
8078
platform_details: dict[str, Any] = response.json().get("details")
81-
self.tool.stream_log("Successfully retrieved platform settings")
8279
return platform_details
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,22 @@
1+
import logging
12
import uuid
23

4+
from unstract.sdk.constants import LogLevel
5+
6+
logger = logging.getLogger(__name__)
7+
38

49
class CommonUtils:
510
@staticmethod
611
def generate_uuid() -> str:
712
"""Class method to get uuid."""
813
return str(uuid.uuid4())
14+
15+
16+
# Mapping from python log level to Unstract counterpart
17+
PY_TO_UNSTRACT_LOG_LEVEL = {
18+
logging.DEBUG: LogLevel.DEBUG,
19+
logging.INFO: LogLevel.INFO,
20+
logging.WARNING: LogLevel.WARN,
21+
logging.ERROR: LogLevel.ERROR,
22+
}

src/unstract/sdk/utils/usage_handler.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,9 @@ def on_event_end(
9090
):
9191
model_name = self.llm_model.metadata.model_name
9292
# Need to push the data to via platform service
93-
self.stream_log(log=f"Pushing llm usage for model {model_name}")
93+
self.stream_log(
94+
log=f"Pushing llm usage for model {model_name}", level=LogLevel.DEBUG
95+
)
9496
llm_token_counter: TokenCounter = TokenCounter.get_llm_token_counts(payload)
9597
Audit(log_level=self.log_level).push_usage_data(
9698
platform_api_key=self.platform_api_key,
@@ -107,7 +109,10 @@ def on_event_end(
107109
):
108110
model_name = self.embed_model.model_name
109111
# Need to push the data to via platform service
110-
self.stream_log(log=f"Pushing embedding usage for model {model_name}")
112+
self.stream_log(
113+
log=f"Pushing embedding usage for model {model_name}",
114+
level=LogLevel.DEBUG,
115+
)
111116
Audit(log_level=self.log_level).push_usage_data(
112117
platform_api_key=self.platform_api_key,
113118
token_counter=self.token_counter,

0 commit comments

Comments
 (0)