Skip to content

Refactor: Replace AWS CLI usage with Boto3 for Bedrock GenAI demos - #1042

Open
Mohd-Hassan wants to merge 2 commits into
mainfrom
Consumer_Bedrock_Demos
Open

Refactor: Replace AWS CLI usage with Boto3 for Bedrock GenAI demos#1042
Mohd-Hassan wants to merge 2 commits into
mainfrom
Consumer_Bedrock_Demos

Conversation

@Mohd-Hassan

Copy link
Copy Markdown
Collaborator

Improves reliability of Bedrock interactions by using native Boto3 SDK instead of shell-based AWS CLI commands.

@DallasBowden DallasBowden left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Notebooks need to be edited since they no longer being used in a workshop.

Remove any references to getting the AWS keys from our Google Bucket.

Remove the configure_aws cell.

Ask for the access_key, secret_access_key and region the same way we do in other AWS notebooks.

Configure the boto3 call without the AWS access token.

Update the html styles for Dark Mode.

Update the queryband DEMO= values to be the same as the notebook filename.

@DallasBowden DallasBowden left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please go back through these and remove all the references to color:#00233C.
Please remove the button to get the aws keys.
Can we connect to our LiteLLM and pull the necessary aws credentials?

@DallasBowden DallasBowden left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove references to color:#00233C.
Remove the button to view the aws access credentials.
Can we connect his to our LiteLLM for the aws credentials?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants