Skip to content

Remove permissions from vector search endpoint recreation test - #6845

Merged
janniklasrose merged 1 commit into
mainfrom
shreyas-goenka/shreyas/remove-vs-recreation-permissions
Sep 26, 2026
Merged

janniklasrose merged 1 commit into
mainfrom
shreyas-goenka/shreyas/remove-vs-recreation-permissions

Conversation

@shreyas-goenka

@shreyas-goenka shreyas-goenka commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Changes

Remove the permissions configuration from the Vector Search endpoint drift/recreated_same_name acceptance test and regenerate its expected output. Keep the checks that remote recreation changes the endpoint UUID and that a subsequent deploy leaves no permanent drift.

Why

Permissions are eventually consistent after an out-of-band endpoint recreation, making the permissions assertions flaky.

Tests

  • Local acceptance test passed with the direct engine for both DMS= and DMS=true.
  • Cloud tests were not run.
  • Go lint passed for all three modules.
  • Full formatting and Python lint were blocked by failure to download the pinned Ruff package from PyPI.

@github-actions github-actions Bot added the DABs DABs related issues label Sep 25, 2026
@eng-dev-ecosystem-bot

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: b7fc329

Run: 36122164695

Env ✅​pass 🙈​skip Time
✅​ aws linux 279 15 5:27
✅​ aws windows 281 13 4:10
✅​ azure linux 278 15 5:37
✅​ azure windows 280 13 4:06
✅​ gcp linux 279 15 5:36
✅​ gcp windows 281 13 4:02
Top 6 slowest tests (at least 2 minutes):
duration env testname
3:59 azure linux TestAccept
3:57 gcp linux TestAccept
3:57 aws linux TestAccept
3:33 aws windows TestAccept
3:30 azure windows TestAccept
3:29 gcp windows TestAccept

@shreyas-goenka
shreyas-goenka marked this pull request as ready for review September 25, 2026 15:48
@shreyas-goenka
shreyas-goenka requested review from a team as code owners September 25, 2026 15:48
@janniklasrose
janniklasrose added this pull request to the merge queue Sep 26, 2026
Merged via the queue into main with commit ecd279b Sep 26, 2026
30 checks passed
@janniklasrose
janniklasrose deleted the shreyas-goenka/shreyas/remove-vs-recreation-permissions branch September 26, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DABs DABs related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants