-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
add new data sources to aws_bedrockagent_data_source #40711
base: main
Are you sure you want to change the base?
add new data sources to aws_bedrockagent_data_source #40711
Conversation
Community NoteVoting for Prioritization
For Submitters
|
Need help in writing acceptance tests since I am not sure what the values of the data configurations should be. It would be nice if someone could assist me on this. |
Would an aws cli example of a "webConfiguration" datasource be helpful to you? I could craft that sometime this week. |
Yes that would certainly help. It would be nice if you could give an example of all configuration too like sales force and share point if possible. |
I've started testing my type=Web. Once finished, I could work on a SalesForce example, but as I will not be willing to share the values of credentialsSecretArn, and I don't have the ability to create a SalesForce account for you, you won't be able to do end-to-end testing, but it might be enough to get the CreateDataSource API call to work, as i suspect the creds are not checked until the sync/injestion job is started. |
yup I totally understand that, I dont think we can implement tests for proprietary solutions (might need insights from maintainers as to how we resolve that) but things that are open we should be able to test |
aws cli example type=Web
|
@ewbankkit I am writing up tests for web configuration but not sure how do we handle salesforce/sharepoint/confluence? |
@hjoshi123 I revised my prior comment with a revised example which provides valid values for a few more of the attributes, so consider adding them to your test case. |
thank you @FireballDWF |
…to f-aws_bedrockagent_data_source-newdatasources
@FireballDWF in the web config in user_agent which is optional but what would be the value for the example that you sent? |
I did not use one, you can supply any string that meets the Length Constraints: "Minimum length of 15. Maximum length of 40." per SDK documentation at https://docs.aws.amazon.com/bedrock/latest/APIReference/API_agent_WebCrawlerConfiguration.html It's really just a passthru to the UserAgent string used by HTTP/S clients |
@FireballDWF I wrote a test based on your input.. let me know if that works.. also I am not sure how I should run it since I saw some of the test involved postgres related commands |
Description
This PR introduces new data source configurations like confluence, sharePoint, salesforce to aws_bedrockagent_data_source resource which was introduced recently Bedrock API. Schema and data models have been updated for the same.
Relations
Closes #40577.
Closes #39770.
References
https://docs.aws.amazon.com/bedrock/latest/APIReference/API_agent_DataSourceConfiguration.html#bedrock-Type-agent_DataSourceConfiguration-webConfiguration
Output from Acceptance Testing