Repository navigation
MOD-20121: Update Playbook operator extraction setup - #4331
Draft
tylerhutcherson wants to merge 2 commits into
Draft
tylerhutcherson wants to merge 2 commits into
tylerhutcherson wants to merge 2 commits into
Conversation
|
|
Contributor
Contributor
🧠 Redis MemoryFound 5 related items from repository history:
Memory updated at 2cf762f |
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.
Ticket and stack
MOD-20121. This draft stacks on operator-docs #4256 at
430c680a53e1a1060e26b8f46abb3ed6c7c6b753; its parent branch and PR are unchanged. Public RAM setup is coordinated in #4330. Backend: redislabsdev/iris#2065, following MOD-20115 and MOD-20116. UI recovery/copy: cloud-ui #3346.What changes
The older operator draft prescribes grant preparation and bundled Identity Service values that the new extraction stack no longer uses. This updates the full fresh-install walkthrough to separate Identity Service releases with matching runtime/validator credentials, static RAM worker read trust, a tenant-confined RAM Control Plane viewer, live source discovery, synchronous ready/off binding creation, explicit enable/recovery validation, and source-independent pause/delete. Ordinary application grants and RAM's own worker remain independent.
The control-plane OpenAPI artifact is copied byte-for-byte from backend Smithy-generated public output; its source hash and revision were updated together. Existing 13-operation/hash checks remain unchanged. The data-plane reference and SDK pins remain unchanged: normalized DP wire hash matches the locked published SDK input.
Independent deep and ponytail review corrected unsupported global IdS
product_adminmapping, health URLs, extraction-overlay retention during upgrades, and the metadata JSON prerequisite. Reviewer/challenger agreement was 100%; all four source-backed fixes were verified. The latest docs-only follow-up refreshes the internal review record with completed local qualification and coordinated UI release requirements; public pages and generated artifacts are unchanged by that follow-up.Qualification and publication hold
Keep this PR DRAFT. Do not merge or publish until the backend stack is merged/released and matching chart/image versions and endpoint requirements are verified. Both foundation PRs have merged; child #2065 remains unmerged and the stack is unreleased. Existing placeholder versions are intentional. Coordinate parent #4256, public RAM #4330 and this PR. Merging docs
mainpublishes live automatically.Locked production/API source
af81d8655a5c2ff48003cc96fb7bce42daf1935fcompleted 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. The restack has no Smithy/API change; the generated CP artifact andsource-lock.jsonremain unchanged at qualified API source af81. Reverify the lock/artifact against the eventual release before publication. Backend qualification record.The existing contributor
license/claremains pending here, on #4330, #4256 and earlier docs drafts. No agreement was signed, asserted or changed. A human must resolve that condition if it persists; the documented status recheck did not clear it.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
ci-auththen passed all four cases in 873.105 seconds, including both projected probe-token rotations with strict unchanged worker UID, wrong-audience rejection and three RAM promotion jobs. Its report hascomplete: true, exit 0 and no missing/unsuccessful cases.@redislabsdev/cloud-ui-irisultra@0.0.19each passed three on-prem browser cases: live-source lifecycle, stale-version recovery and late-source discovery. These are three source/destination pairs in one configured tenant, not multiple on-prem accounts.044628e4, with one account and no eligible sources. Current-candidate cloud A/B/C and lifecycle mutations were unavailable there; no account/infrastructure was provisioned to force them.These results do not establish live-model quality, production licensing, deployed Neptune reachability, published-version support, fleet capacity or spending limits.
Coordinated UI release
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.
The installed
0.0.19bundle preserves the tested happy path but lacks #3346's blocked/off recovery and copy corrections. Separately for on-prem UI consumption, normal post-merge MFE publication must be followed by an explicit update of the actual Irispackage.jsonand lockfile pin, a rebuilt/released UI gateway, and a packaged smoke without a local bundle override. There is no automatic cross-repository pin updater. Passing with a reviewed local bundle is not proof that a published image includes those corrections.Documentation validation
TestControlConfigValidatesOperatorJWTBlockspassed; Helm rendering alone is not proof of Go startup-policy validity.b4e5ad6ff318dcd8e9f60bb75fb2d109058891d0e211e62f9781b24f221cf7f6; final-child byte comparison passed. DP wire SHA256 remains991a428a09c362b4b45e1f90bf32c5d9980c6933d2f31d12b2dd730d5a6f34b3.2cf762f940f6acee5a9cb7b49799253e45f4f341passed remote build/test/staging CI: 99 successful checks, 1 skipped. The fresh SDK/reference/example/type workflow passed all 138 tests in 239.95 seconds and the locked SDK-source check; full-site builds and staging deployments also passed.license/clais the only pending status; no review finding is open.The review-record-only follow-up passed focused content/link/provenance checks and
git diff --check, with independent backend-owner approval. It leaves public pages, generated artifacts, source lock and SDK pins unchanged; earlier page/SDK validation remains applicable. Normal commit hooks were preserved.Current docs branch revision:
2cf762f940f6acee5a9cb7b49799253e45f4f341. Build/test/staging checks are complete at this revision. CLA, draft and coordinated-release holds remain in place.@hristopopovredis