Skip to content

fix(data-analyst-conversational-assistant): pin agent dependencies - #2170

Open
nehabthakur wants to merge 1 commit into
awslabs:mainfrom
nehabthakur:fix/data-analyst-pin-dependencies
Open

nehabthakur wants to merge 1 commit into
awslabs:mainfrom
nehabthakur:fix/data-analyst-pin-dependencies

Conversation

@nehabthakur

Copy link
Copy Markdown
Contributor

… prevent runtime startup failure

Unpinned mcp resolved to a release that removed streamablehttp_client, causing an ImportError on container start. Add lower and major-version upper bounds to all agent requirements, and drop the unquoted aws-opentelemetry-distro install from the Dockerfile (the shell treated '>=' as a redirect, bypassing the version constraint).

Amazon Bedrock AgentCore Samples Pull Request

Important

  1. We strictly follow a issue-first approach, please first open an issue relating to this Pull Request.
  2. Once this Pull Request is ready for review please attach review ready label to it. Only PRs with review ready will be reviewed.

Issue number:

Concise description of the PR

Changes to ..., because ...

User experience

Please share what the user experience looks like before and after this change

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • I have reviewed the contributing guidelines
  • Add your name to CONTRIBUTORS.md
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Are you uploading a dataset?
  • Have you documented Introduction, Architecture Diagram, Prerequisites, Usage, Sample Prompts, and Clean Up steps in your example README?
  • I agree to resolve any issues created for this example in the future.
  • I have performed a self-review of this change
  • Changes have been tested
  • Changes are documented

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

… prevent runtime startup failure

Unpinned mcp resolved to a release that removed streamablehttp_client, causing an ImportError on container start. Add lower and major-version upper bounds to all agent requirements, and drop the unquoted aws-opentelemetry-distro install from the Dockerfile (the shell treated '>=' as a redirect, bypassing the version constraint).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant