Skip to content

Commit e08f742

Browse files
copilot comment resolving
1 parent e8bab67 commit e08f742

4 files changed

Lines changed: 12 additions & 19 deletions

File tree

src/backend/.env.sample

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,9 @@ AZURE_AI_AGENT_MODEL_DEPLOYMENT_NAME = ""
2828
APP_ENV = "dev"
2929

3030
# Basic application logging (default: INFO level)
31-
AZURE_BASIC_LOGGING_LEVEL=DEBUG
31+
AZURE_BASIC_LOGGING_LEVEL=INFO
3232
# Azure package logging (default: WARNING level to suppress INFO)
33-
AZURE_PACKAGE_LOGGING_LEVEL=DEBUG
33+
AZURE_PACKAGE_LOGGING_LEVEL=WARNING
3434
# Comma-separated list of specific logger names to configure (default: empty - no custom loggers)
3535
# Example: AZURE_LOGGING_PACKAGES=azure.identity.aio._internal,azure.monitor.opentelemetry.exporter.export._base
3636
AZURE_LOGGING_PACKAGES=

src/backend/common/config/config.py

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@
1515
import os
1616

1717
from azure.identity.aio import ClientSecretCredential
18-
from azure.identity.aio import DefaultAzureCredential as AioDefaultAzureCredential
19-
from azure.identity.aio import ManagedIdentityCredential as AioManagedIdentityCredential
2018

2119
from helper.azure_credential_utils import get_azure_credential
2220

@@ -56,16 +54,14 @@ def __init__(self):
5654

5755
def get_azure_credentials(self):
5856
"""Retrieve Azure credentials, either from environment variables or managed identity."""
59-
if os.getenv("APP_ENV", "prod").lower() == "dev":
60-
if all([self.azure_tenant_id, self.azure_client_id, self.azure_client_secret]):
61-
return ClientSecretCredential(
62-
tenant_id=self.azure_tenant_id,
63-
client_id=self.azure_client_id,
64-
client_secret=self.azure_client_secret,
65-
)
66-
return AioDefaultAzureCredential()
67-
return AioManagedIdentityCredential(client_id=self.azure_client_id)
57+
if all([self.azure_tenant_id, self.azure_client_id, self.azure_client_secret]):
58+
return ClientSecretCredential(
59+
tenant_id=self.azure_tenant_id,
60+
client_id=self.azure_client_id,
61+
client_secret=self.azure_client_secret,
62+
)
63+
return self.__azure_credentials
6864

6965

7066
app_config = Config()
71-
print(f"[DEBUG] AI_PROJECT_ENDPOINT: '{os.getenv('AI_PROJECT_ENDPOINT')}'")
67+
print(f"[DEBUG] AI_PROJECT_ENDPOINT: '{os.getenv('AI_PROJECT_ENDPOINT')}'")

src/backend/sql_agents/convert_script.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,7 @@ async def convert_script(
101101
AuthorRole(response.role),
102102
)
103103
current_migration = None
104+
is_complete = True
104105
break
105106
case AgentType.SYNTAX_CHECKER.value:
106107
result = SyntaxCheckerResponse.model_validate_json(
@@ -271,13 +272,12 @@ async def convert_script(
271272
FileResult.ERROR,
272273
),
273274
)
275+
is_complete = True
274276
break
275277

276278
if comms_manager.group_chat.is_complete:
277279
is_complete = True
278280

279-
break
280-
281281
migrated_query = current_migration
282282

283283
# Handle the case where migration failed and current_migration is None

src/frontend/.dockerignore

Lines changed: 0 additions & 3 deletions
This file was deleted.

0 commit comments

Comments
 (0)