Skip to content

feat(gooddata-sdk): [AUTO] Deprecate LLM Endpoint API and add resolveLlmProviders endpoint#1541

Open
yenkins-admin wants to merge 1 commit intomasterfrom
auto/openapi-sync-C005-20260413-r17062
Open

feat(gooddata-sdk): [AUTO] Deprecate LLM Endpoint API and add resolveLlmProviders endpoint#1541
yenkins-admin wants to merge 1 commit intomasterfrom
auto/openapi-sync-C005-20260413-r17062

Conversation

@yenkins-admin
Copy link
Copy Markdown
Contributor

Summary

Added SDK wrapper for the new resolveLlmProviders endpoint. Created CatalogResolvedLlmProvider and CatalogResolvedLlmModel entity model classes in a new file resolved_llm_provider.py, added resolve_llm_providers() service method to CatalogWorkspaceService, exported both new classes from gooddata_sdk/__init__.py, and wrote unit tests verifying the model serialization. The deprecated endpoint/model annotations in the OpenAPI diff do not require SDK changes since the SDK does not wrap the deprecated resolveLlmEndpoints endpoint.

Impact: deprecation | Services: gooddata-afm-client, gooddata-metadata-client

Files changed

  • gooddata-sdk/packages/gooddata-sdk/src/gooddata_sdk/catalog/workspace/entity_model/resolved_llm_provider.py
  • gooddata-sdk/packages/gooddata-sdk/src/gooddata_sdk/catalog/workspace/service.py
  • gooddata-sdk/packages/gooddata-sdk/src/gooddata_sdk/__init__.py
  • gooddata-sdk/packages/gooddata-sdk/tests/catalog/test_resolved_llm_provider.py

Source commits (gdc-nas)

  • a19eb19 Merge pull request #21393 from hkad98/jkd/llm-endpoint-deprecation
OpenAPI diff
     "/api/v1/actions/ai/llmEndpoint/test": { "post": {
+        "deprecated": true,
+        "description": "Will be soon removed and replaced by testLlmProvider."
     "/api/v1/actions/ai/llmEndpoint/{llmEndpointId}/test": { "post": {
+        "deprecated": true,
+        "description": "Will be soon removed and replaced by testLlmProviderById."
     "/api/v1/actions/workspaces/{workspaceId}/ai/resolveLlmEndpoints": { "get": {
+        "deprecated": true,
+        "description": "Will be soon removed and replaced by LlmProvider-based resolution."
+    "/api/v1/actions/workspaces/{workspaceId}/ai/resolveLlmProviders": {
+      "get": { "operationId": "resolveLlmProviders", "summary": "Get Active LLM configuration for this workspace" }
+    },
       "JsonApiLlmEndpointIn": {
+        "deprecated": true, "description": "Will be soon removed and replaced by LlmProvider."
       "JsonApiLlmEndpointOut": { "deprecated": true },
       "JsonApiLlmEndpointPatch": { "deprecated": true },
+      "ResolvedLlm": { "description": "The resolved LLM configuration, or null if none is configured." },
+      "ResolvedLlmProvider": { "allOf": [{ "$ref": "ResolvedLlm" }, { "properties": { "id": {...}, "models": {...}, "title": {...} } }] },
+      "ResolvedLlms": { "properties": { "data": { "oneOf": [{ "$ref": "ResolvedLlmEndpoint" }, { "$ref": "ResolvedLlmProvider" }] } } }

Workflow run


Generated by SDK OpenAPI Sync workflow

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2026

Codecov Report

❌ Patch coverage is 69.69697% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.64%. Comparing base (d7f50b7) to head (72c34f8).

Files with missing lines Patch % Lines
...-sdk/src/gooddata_sdk/catalog/workspace/service.py 25.00% 6 Missing ⚠️
...og/workspace/entity_model/resolved_llm_provider.py 83.33% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1541      +/-   ##
==========================================
- Coverage   78.66%   78.64%   -0.02%     
==========================================
  Files         230      231       +1     
  Lines       15400    15433      +33     
==========================================
+ Hits        12114    12137      +23     
- Misses       3286     3296      +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant