Repository navigation
MOD-20121: Align Playbook extraction authentication setup - #4330
Draft
tylerhutcherson wants to merge 1 commit into
Draft
tylerhutcherson wants to merge 1 commit into
tylerhutcherson wants to merge 1 commit into
Conversation
|
|
Contributor
Contributor
🧠 Redis MemoryFound 5 related items from repository history (5 new this commit):
Memory updated at 9644dff |
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.
MOD-20121
Ticket. Backend: redislabsdev/iris#2065, stacked on #2064 and #2061. Operator walkthrough: docs #4331, stacked on #4256. UI recovery/copy: cloud-ui #3346.
The optional Playbook extraction instructions still prescribe Identity Service subject grants and lookup permissions. The new Control Plane-owned binding lifecycle uses RAM's existing static worker JWT trust for source reads and a separate tenant-confined Control Plane viewer for source discovery and ownership checks. This replaces that obsolete recipe with the exact RAM Helm values, explains operator consent and static-trust revocation, and preserves ordinary agent-key authentication, RAM's own worker identity, and separately administered service-subject integrations.
The 23 public Agent Memory paths were inspected for this change. This is the only extraction-specific mismatch found; the backend stack does not change RAM's public API, so the RAM OpenAPI snapshots stay unchanged.
Coordinated publication required
Keep this PR DRAFT. Do not merge or publish until the backend stack is merged/released and applicable RAM/Playbook charts and images are verified. Both foundation PRs have merged; child #2065 remains unmerged and the stack is unreleased. Merging docs
mainpublishes live automatically. Coordinate #4256, #4331 and this PR; preserve the CLA gate.The existing Memory installation walkthrough's pinned versions do not establish support for this setup. No minimum versions or release URLs are invented here. Verify release applicability before publication and add released-version guidance then. The backend change provides no existing-installation migration.
Cloud UI #3346 must remain unmerged until the candidate backend is deployed in the Cloud environments where extraction is exposed; its master merge starts QA/production workflows. After backend rollout, run a Cloud preview smoke with an eligible RAM source to verify binding creation and activation before UI release. The older app-qa baseline with no eligible sources does not satisfy this gate.
Separately for on-prem UI consumption, normal post-merge MFE publication must be followed by an explicit update of the actual Iris
package.jsonand lockfile pin, a rebuilt/released UI gateway, and a packaged smoke without a local bundle override. No automatic cross-repository pin updater exists. The currently installed0.0.19bundle does not contain #3346's blocked/off recovery and copy corrections.Current backend restack
Both foundation PRs have merged: MOD-20115 #2061 as
876853ddb2c111150c3f4040343597bb50bc7612and MOD-20116 #2064 asf526654fd9a6e4c32ba0e5548f1b23f8af4ec5f3. The child #2065 is nowe6555c27109800bb7c0cfc25657e8c0e08b56e2d, rebased onto mainbc758425655d4ba952e907c7284081f4992bc1a8after #2091 merged. The test-only repair adapts incoming daily startup checks to metadata-local binding cleanup; all ten feature commits are unchanged by the final rebase. Current backend CI is running; Bugbot skipped the unchanged reviewed patch, human approval remains and no review threads are unresolved. Keep merge/publication on hold until current CI, merge-queue and release gates are satisfied. The previous D8 snapshot passed CI (106 successful checks, 24 skipped) and Bugbot, but that is historical evidence. The canceleda542f186run is not a complete passing result. Current backend review record.The previously qualified backend snapshot is
7c73e2a1952904ff5dce9b5ed8d63aff78989077, on MOD-20116d4921756c745bd6e6d7ee990e73830215dc746e5and MOD-20115be04ba2127fc6bc51b2bc5f005cd20e71b532bee. CI for that snapshot is complete. Child #2065 reports 97 successful checks, 20 skipped, no pending or failed checks, Bugbot SUCCESS and no unresolved review threads. All five deployed profiles passed on the child and MOD-20116. Child Branch Checks, MOD-20116 Branch Checks and MOD-20115 CI completed successfully at these exact heads. Earlieraf81d8655/2ed416db1local qualification and CI remain historical evidence. Previous snapshot review record.Historical backend local qualification
Production/API source
af81d8655a5c2ff48003cc96fb7bce42daf1935fpassed 99 reported checks and Bugbot before expanded local qualification. Earlier qualified child2ed416db157ebd12c3f5081ca9eaaf26c8375f0dcompleted 101 reported successes, 20 skips, no pending or failed checks, Bugbot SUCCESS and no unresolved review threads. This is historical CI evidence for that earlier head. Branch Checks run 39903123145 completed successfully at that exact SHA. Backend qualification record.ci-auth: all four cases passed in 873.105 seconds, including both projected probe-token rotations with strict unchanged worker UID, audience rejection, and three RAM promotion jobs. The report is complete with exit 0 and no missing/unsuccessful cases.b38b4e18and installed0.0.19UI bundles each passed the three on-prem lifecycle/stale-version/late-source browser cases. App-qa was read-only baseline evidence at backend044628e4, with one account and no eligible sources; it did not qualify current-candidate cloud A/B/C or mutations.These local results do not establish live-model quality, production licensing, deployed Neptune reachability, published-version support, fleet capacity or spending limits.
Documentation validation
720f1063826dc920102dcd288890725adfa2559b. Assertions verify static worker read trust, tenant-confined CP viewer, existing agent-key introspection, RAM worker write trust and matching admin tenant. This is the original render evidence, separate from the later backend runtime qualification.git diff --checkpassed. Independent backend-owner review approved the complete diff and operator clarification. No page changes are added by this evidence-only PR-body refresh.Docs revision and remaining checks
Branch revision:
9644dffdbc190bb003e615233c8c69e5adcfd610. Remote build/staging CI passed at that revision: 101 successful checks, 1 skipped; no open code-review findings.license/claremains pending after the documented bot recheck. No agreement was signed, asserted or changed. The same contributor condition is pending on existing docs drafts; human contributor action is still required if it persists. Draft and coordinated-publication holds remain.@hristopopovredis