-
Notifications
You must be signed in to change notification settings - Fork 4
feat: agent core runtime + runtime endpoint #5
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
Conversation
|
Hi there, I am commenting because your project has not yet been published and you have opened a pr to the If you are ready to publish, please issue the following comment on this PR and I will initiate the publication review
If you require more information please reach out to your AWS contact assisting you with this project. |
|
/do-e2e-tests |
|
End to end test has been scheduled |
|
E2E tests in progress |
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.
E2E test has completed with errors. If you are an external contributor, please contact the project maintainers for more information.
|
/do-e2e-tests |
|
End to end test has been scheduled |
|
E2E tests in progress |
1 similar comment
|
E2E tests in progress |
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.
E2E tests completed successfully
|
/confirm-publication |
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.
Review has been requested from required contributors, once they have approved I will remove my request for changes. When this PR is merged your project will be published
Co-authored-by: Scott Schreckengaust <scottschreckengaust@users.noreply.github.com>
scottschreckengaust
left a comment
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.
LGTM
dineshSajwan
left a comment
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.
Looks good, approving it but have some suggestion for the documentation.
| - **IAM Role Management**: Automatic creation of IAM roles with appropriate permissions | ||
| - **Environment Variables**: Pass configuration to your runtime container | ||
| - **JWT Authorization**: Optional JWT authorizer configuration for secure access | ||
| - **Endpoint Management**: Create and manage runtime endpoints for client access |
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.
It will be good if we can add more details about endpoint management. How the default endpoint is created and how you can leverage endpoint to point to specific version. You can refer here - https://github.com/aws/aws-cdk/blob/150b222669d997c7bc10270a27bacf498b2c282a/packages/%40aws-cdk/aws-bedrock-agentcore-alpha/README.md#managing-endpoints-and-versions
| } | ||
| ``` | ||
|
|
||
| ### With JWT Authorization |
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.
We should mention that if no runtime_authorizer_configuration is there then by default underlying service use I AM. Also are we not supporting oAuth for now ? - https://github.com/krokoko/agent-core-cdk-constructs/blob/5e4b3cdcb5a181d1710795d9314263aab42fc267/src/cdk-lib/bedrock-agentcore/runtime/runtime.ts#L753
32eb076
|
/do-e2e-tests |
|
End to end test has been scheduled |
|
E2E tests in progress |
1 similar comment
|
E2E tests in progress |
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.
E2E tests completed successfully
scottschreckengaust
left a comment
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.
LGTM
No description provided.