Skip to content

Commit

Permalink
Removed bedrockEndpoint to fix Bedrock access issue
Browse files Browse the repository at this point in the history
  • Loading branch information
spugachev committed Nov 26, 2023
1 parent 7931a31 commit 500575c
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 13 deletions.
1 change: 0 additions & 1 deletion bin/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ export function getConfig(): SystemConfig {
bedrock: {
enabled: true,
region: SupportedRegion.US_EAST_1,
endpointUrl: "https://bedrock-runtime.us-east-1.amazonaws.com",
},
llms: {
// sagemaker: [SupportedSageMakerModels.FalconLite]
Expand Down
12 changes: 0 additions & 12 deletions cli/magic-create.ts
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@ const embeddingModels = [
options.prefix = config.prefix;
options.bedrockEnable = config.bedrock?.enabled;
options.bedrockRegion = config.bedrock?.region;
options.bedrockEndpoint = config.bedrock?.endpointUrl;
options.bedrockRoleArn = config.bedrock?.roleArn;
options.sagemakerModels = config.llms?.sagemaker ?? [];
options.enableSagemakerModels = config.llms?.sagemaker
Expand Down Expand Up @@ -135,16 +134,6 @@ async function processCreateOptions(options: any): Promise<void> {
return !(this as any).state.answers.bedrockEnable;
},
},
{
type: "input",
name: "bedrockEndpoint",
message: "Bedrock endpoint - leave as is for standard endpoint",
initial() {
return `https://bedrock-runtime.${
(this as any).state.answers.bedrockRegion
}.amazonaws.com`;
},
},
{
type: "input",
name: "bedrockRoleArn",
Expand Down Expand Up @@ -316,7 +305,6 @@ async function processCreateOptions(options: any): Promise<void> {
region: answers.bedrockRegion,
roleArn:
answers.bedrockRoleArn === "" ? undefined : answers.bedrockRoleArn,
endpointUrl: answers.bedrockEndpoint,
}
: undefined,
llms: {
Expand Down

0 comments on commit 500575c

Please sign in to comment.