Repository navigation
Fix ownership transfers, search timestamps and health responses - #192
Merged
Merged
Conversation
maxlastglance
force-pushed
the
fix/ownership-search-health
branch
2 times, most recently
from
November 7, 2026 10:27
f39854d to
81bd089
Compare
maxlastglance
force-pushed
the
fix/ownership-search-health
branch
from
November 7, 2026 10:35
81bd089 to
e127094
Compare
This was referenced Nov 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Database transfers currently send the wrong SDK field and can leave the new owner without a grant. Search rejects space-separated Metadata timestamps in percolator documents. Public health responses also differ between gateway and direct-UI deployments.
Four separate commits address #179–#182:
Closes #179. Closes #180. Closes #181. Closes #182.
Local validation so far: 342 SDK unit tests, 96 affected Metadata tests, including seven owner-transfer/validation tests and OpenAPI coverage, and a production UI build pass. Search integration tests cover raw nested timestamps against fresh and existing percolator indexes, alongside the existing authorization tests. All PR checks pass on the final head. The isolated user-profile preview reports 109 passed/6 failed, previously 104/11, with no newly failing case. Three remaining raw failures are retired catalog calls and three belong to other open findings. A separate owner-control probe confirms immediate reads and writes after revocation/transfer, denies the former owner administrative transfer, and verifies transfer back. Direct UI/gateway health contracts and the retained metrics endpoint pass. The final date-normalization refinement was rechecked with real nested SQL timestamps against OpenSearch 2.18; all three search routes returned the expected database. Search CI also covers fresh/existing indexes and authorization on OpenSearch 2.10. Temporary preview containers were removed; persistent services remain healthy.
Hold merge for maintainer review of the four commits.