-
Notifications
You must be signed in to change notification settings - Fork 90
feat: add AWS Data Processing MCP server proxy #932
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
"SELECTED_SERVERS": "*" | ||
}, | ||
"disabled": false, | ||
"timeout": 60000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's increase from 60sec to 180sec
"command": "aws-data-processing-mcp-proxy", | ||
"args": [], | ||
"env": { | ||
"AWS_PROFILE_NAME": "remote_mcp_profile", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's this profile and how do we configure it?
|
||
# Install AWS Data Processing MCP proxy wheel file | ||
echo "Installing AWS Data Processing MCP proxy..." | ||
pip install /etc/sagemaker-ui/sagemaker-mcp/amzn_aws_data_processing_remote_mcp_local_proxy-0.1.0-py3-none-any.whl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: Finalize the version
"args": [], | ||
"env": { | ||
"AWS_PROFILE_NAME": "remote_mcp_profile", | ||
"SELECTED_SERVERS": "*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: Finalize on servers/capabilities based on environment/persona
"SELECTED_SERVERS": "*" | ||
}, | ||
"disabled": false, | ||
"timeout": 60000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For each config, increase timeout from 60 to 180sec
Description
Adds a local MCP server that enables proxying to the AWS Data Processing MCP server. This version allows for special handling of things like FAS credentials, etc. to enable the following tools:
Type of Change
Release Information
Does this change need to be included in patch version releases? By default, any pull requests will only be added to the next SMD image minor version release once they are merged in template folder. Only critical bug fix or security update should be applied to new patch versions of existed image minor versions.
If yes, please explain why:
N/A
How Has This Been Tested?
Made changes to prod SMUS, ran startup script manually.
Checklist:
Test Screenshots (if applicable):
Related Issues
[Link any related issues here]
Additional Notes
[Any additional information that might be helpful for reviewers]