Skip to content

Commit 910adb5

Browse files
authored
Merge pull request #2423 from redpanda-data/alena/fix-bedrock-region-on-edit
fix(agents): preserve bedrock region on AI agent edit
2 parents f942e34 + d87ce05 commit 910adb5

1 file changed

Lines changed: 15 additions & 1 deletion

File tree

frontend/src/components/pages/agents/details/ai-agent-configuration-tab.tsx

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,6 +191,10 @@ const extractProviderInfo = (provider: AIAgent_Provider): { apiKeyTemplate: stri
191191
apiKeyTemplate = provider.provider.value.apiKey;
192192
baseUrl = provider.provider.value.baseUrl;
193193
break;
194+
case 'bedrock':
195+
// Bedrock has no apiKey/baseUrl; region is stored via the baseUrl field
196+
baseUrl = provider.provider.value.region || '';
197+
break;
194198
default:
195199
break;
196200
}
@@ -1252,9 +1256,19 @@ export const AIAgentConfigurationTab = () => {
12521256
: displayData.provider?.provider.case || 'openai'
12531257
) as 'openai' | 'anthropic' | 'google' | 'openaiCompatible' | 'bedrock';
12541258

1259+
// When switching to a bedrock provider, update the region from the LLM provider config
1260+
let baseUrl = displayData.baseUrl || '';
1261+
if (formTypeId === 'bedrock' && providersData?.llmProviders) {
1262+
const fullProvider = providersData.llmProviders.find((p) => p.name === value);
1263+
if (fullProvider?.providerConfig?.case === 'bedrockConfig') {
1264+
baseUrl = fullProvider.providerConfig.value?.region || '';
1265+
}
1266+
}
1267+
12551268
updateField({
12561269
llmProvider: value,
1257-
provider: createUpdatedProvider(formTypeId, '', displayData.baseUrl || ''),
1270+
provider: createUpdatedProvider(formTypeId, '', baseUrl),
1271+
baseUrl,
12581272
model: '',
12591273
});
12601274
}}

0 commit comments

Comments
 (0)