Skip to content

Feature/ab#32465 model endpoint configuration management#2335

Merged
JamesPasta merged 2 commits into
devfrom
feature/AB#32465-model-endpoint-configuration-management
Apr 22, 2026
Merged

Feature/ab#32465 model endpoint configuration management#2335
JamesPasta merged 2 commits into
devfrom
feature/AB#32465-model-endpoint-configuration-management

Conversation

@jacobwillsmith
Copy link
Copy Markdown
Contributor

Pull request overview

Updated the OpenAI configuration shape so the runtime reads the standard nested Azure:... keys directly. This keeps appsettings and environment-variable binding aligned with normal .NET configuration behavior and removes the need for explicit compatibility constants.

Changes:

  • Simplified OpenAI config resolution to read Azure:OpenAI:ApiKey and Azure:OpenAI:Endpoint directly
  • Removed the explicit compatibility constants for the raw OpenAI env-var names
  • Preserved the legacy Azure:OpenAI:ApiUrl fallback path
  • Updated the development appsettings override to match the current branch state
  • Kept the rest of the OpenAI runtime behavior unchanged

@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 620
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@jacobwillsmith jacobwillsmith marked this pull request as ready for review April 22, 2026 22:03
@JamesPasta JamesPasta merged commit 8998f1e into dev Apr 22, 2026
25 checks passed
@JamesPasta JamesPasta deleted the feature/AB#32465-model-endpoint-configuration-management branch April 22, 2026 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants