Skip to content

Commit

Permalink
Merge pull request #1296 from solliancenet/kb-fix-app-config-settings…
Browse files Browse the repository at this point in the history
…-app-insights-python

Fixes app config settings associated with App Insights in Python APIs
  • Loading branch information
ciprianjichici authored Jul 29, 2024
2 parents 73ff3d2 + 435c591 commit 14b980b
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 6 deletions.
10 changes: 8 additions & 2 deletions src/dotnet/Common/Constants/Configuration/KeyVaultSecretNames.cs
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,17 @@ public static partial class KeyVaultSecretNames
public const string FoundationaLLM_APIEndpoints_GatekeeperAPI_APIKey =
"foundationallm-apiendpoints-gatekeeperapi-apikey";

/// <summary>
/// The name of the Azure Key Vault secret holding the connection string for the App Insights service used by the Gatekeeper API.
/// </summary>
public const string FoundationaLLM_APIEndpoints_GatekeeperAPI_AppInsightsConnectionString =
"foundationallm-appinsights-connectionstring";

/// <summary>
/// The name of the Azure Key Vault secret holding the API key for the Gatekeeper Integration API.
/// </summary>
public const string FoundationaLLM_APIEndpoints_GatekeeperIntegrationAPI_APIKey =
"foundationallm-apiendpoints-gatekeeperintergrationapi-apikey";
"foundationallm-apiendpoints-gatekeeperintegrationapi-apikey";

/// <summary>
/// The name of the Azure Key Vault secret holding the connection string for the App Insights service used by the Gatekeeper Integration API.
Expand Down Expand Up @@ -135,7 +141,7 @@ public static partial class KeyVaultSecretNames
/// The name of the Azure Key Vault secret holding the API key for the State API.
/// </summary>
public const string FoundationaLLM_APIEndpoints_StateAPI_APIKey =
"foundationallm-apinedpoints-stateapi-apikey";
"foundationallm-apiendpoints-stateapi-apikey";

/// <summary>
/// The name of the Azure Key Vault secret holding the connection string for the App Insights service used by the State API.
Expand Down
2 changes: 1 addition & 1 deletion src/python/AgentHubAPI/app/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
# Open a connection to the app configuration
config = get_config()
# Start collecting telemetry
Telemetry.configure_monitoring(config, f'FoundationaLLM:APIs:{API_NAME}:AppInsightsConnectionString')
Telemetry.configure_monitoring(config, f'FoundationaLLM:APIEndpoints:{API_NAME}:AppInsightsConnectionString')

app = FastAPI(
title=f'FoundationaLLM {API_NAME}',
Expand Down
2 changes: 1 addition & 1 deletion src/python/DataSourceHubAPI/app/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
# Open a connection to the app configuration
config = get_config()
# Start collecting telemetry
Telemetry.configure_monitoring(config, f'FoundationaLLM:APIs:{API_NAME}:AppInsightsConnectionString')
Telemetry.configure_monitoring(config, f'FoundationaLLM:APIEndpoints:{API_NAME}:AppInsightsConnectionString')

app = FastAPI(
title=f'FoundationaLLM {API_NAME}',
Expand Down
2 changes: 1 addition & 1 deletion src/python/LangChainAPI/app/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
# Open a connection to the app configuration
config = get_config()
# Start collecting telemetry
Telemetry.configure_monitoring(config, f'FoundationaLLM:APIs:{API_NAME}:AppInsightsConnectionString')
Telemetry.configure_monitoring(config, f'FoundationaLLM:APIEndpoints:{API_NAME}:AppInsightsConnectionString')

app = FastAPI(
title=f'FoundationaLLM {API_NAME}',
Expand Down
2 changes: 1 addition & 1 deletion src/python/PromptHubAPI/app/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
# Open a connection to the app configuration
config = get_config()
# Start collecting telemetry
Telemetry.configure_monitoring(config, f'FoundationaLLM:APIs:{API_NAME}:AppInsightsConnectionString')
Telemetry.configure_monitoring(config, f'FoundationaLLM:APIEndpoints:{API_NAME}:AppInsightsConnectionString')

app = FastAPI(
title=f'FoundationaLLM {API_NAME}',
Expand Down

0 comments on commit 14b980b

Please sign in to comment.