fix(bedrock): Grant delete vectors permission to knowledge base execution role

[gazoakley]

Fixes #1305

---

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

[krokoko]

Thanks @gazoakley , I think the fix looks good, have you tested it already to confirm ? If yes, we can make the PR ready and ship it.

[krokoko]

hi @gazoakley , just checking if the PR is ready for review ? Thanks !

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.34%. Comparing base (368ec39) to head (9872e2e).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff            @@
##             main    #1306    +/-   ##
========================================
  Coverage   95.34%   95.34%            
========================================
  Files          87       87            
  Lines       26698    26699     +1     
  Branches      785      479   -306     
========================================
+ Hits        25455    25456     +1     
  Misses       1243     1243            

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

This pull request is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. If you wish to exclude this issue from being marked as stale, add the "backlog" label.

[gazoakley]

hi @gazoakley , just checking if the PR is ready for review ? Thanks !

Hi @krokoko - in theory it works, but I haven't been able to get npm link to work in such as a way to test fully. Given the unit test passes I think it's likely to work when built properly. Are you OK to push this anyway?

[krokoko]

hi @gazoakley , just checking if the PR is ready for review ? Thanks !

Hi @krokoko - in theory it works, but I haven't been able to get npm link to work in such as a way to test fully. Given the unit test passes I think it's likely to work when built properly. Are you OK to push this anyway?

Yes I would like to test it to verify before shipping, let me do a test this week

[krokoko]

testing now

[krokoko]

Tested, fix works as expected
Created a KB with the fix above, uploaded files, synced, added another file, synced, removed a file, synced
See attached picture:

@gazoakley thanks for this !