Repository navigation
MOD-19813: Document Memory 0.8.0 with a standalone Identity Service - #4272
Merged
Merged
Conversation
Contributor
Contributor
gudata
force-pushed
the
MOD-19272-ids-subchart-docs
branch
from
October 15, 2026 09:07
b5fb351 to
77ea461
Compare
gudata
force-pushed
the
MOD-19272-ids-subchart-docs
branch
from
October 21, 2026 14:04
77ea461 to
04a0aa1
Compare
gudata
marked this pull request as ready for review
October 21, 2026 15:53
teoparvanov
reviewed
Oct 21, 2026
| | Store Redis | Always | `databases."1".urls` | Session memory JSON, long-term memory hashes, RediSearch indexes, vectors, and TTL-managed data. | | ||
| | Metadata Redis | Always | `metadata.urls`, and optionally `metadata.namespace` (default `iris:memory`) | Store records. | | ||
| | Job Redis | Always | `background_jobs.redis.urls` | Background work, retry state, delayed jobs, and idempotency markers. | | ||
| | Job Redis | Always | `background_jobs.redis.urls` and, when execution idempotency is enabled, `background_jobs.idempotency.urls` | Background work, retry state, delayed jobs, and execution markers. The two settings can use the same Redis. | |
There was a problem hiding this comment.
Could we recommend just noeviction in the Job Redis paragraph below (lines 76–77)? volatile-ttl evicts keys with an expiry under memory pressure, so it is not a non-evicting policy. Using noeviction would also keep this consistent with the Job Redis guidance on the Operations page. Redis eviction policies.
teoparvanov
approved these changes
Oct 21, 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.
Ticket ID
MOD-19813
Description
Update the seven Agent Memory self-managed pages for the published Memory chart
0.8.0and the separately installed Identity Service chart0.10.0. Memory deploys the data plane, workers, and control plane. Operators install Identity Service first, provision the shared credentials, then connect Memory throughdataplane.identityServiceBaseURLandidentityService.credential.The installation procedure provides separate values files, matching tenant settings, secure endpoints, certificate trust, and worker identity. Configuration, authentication, private registry, rotation, Federal Information Processing Standards (FIPS), network policy, and troubleshooting guidance follow the same contract. Published Memory image tags default to
0.8.0; Identity Service uses its own version0.10.0. Direct upgrades from earlier releases with an embedded Identity Service, including0.7.0, remain unsupported.Contract source: Iris pull request 1926 and the published Helm packages. Standalone Identity Service specification.
Summary
flowchart LR Memory["Memory 0.8.0 data plane"] -->|Runtime introspection| Identity["Identity Service 0.10.0"] Identity -->|Grant validation| Control["Memory control plane"]Testing
helm templateandhelm lintwith the documented installation values.bash -nafter placeholder substitution.0.143.1builds successfully after local example generation; all seven rendered pages and 234 section links and anchors pass checks. Frontmatter and aliases are preserved; no new external Markdown links were added./healthfailure against the report's running Memory0.8.0, then verified the corrected/health/readinessexample returns HTTP 200 and{"status":"ready"}.pr-deep-reviewreviewer/challenger passes andponytail-reviewfound no concrete defects.The current chart, example, shell, and rendered-page checks cover documentation head
1edb30458. The supplied deployment report tested earlier revision04a0aa18eon a disposable kind cluster: installation, authentication, promotion, credential rotation, supplied configuration, and local-model evaluation succeeded. Its seven findings are addressed: readiness route, isolated HTTP lab setup, worker idempotency, initial CA Secret configuration, complete local-provider edits, optional test-hook image, and configuration/rotation details. Final reviewer/challenger passes also caught and corrected the need for fresh stores after changing embedding dimensions.The complete revised tutorial was not deployed again; the health correction was checked against the existing running installation. FIPS was rendered, not deployed. Additional package evidence is the successful Memory 0.8.0 release validation, at Iris revision
dfdf966c9d88bb5832f57ddfb02b727f4c1ae488. The shared Cloud OpenAPI health entry is retained because the separate SM Cloud maintenance contract still defines it. The full component-refresh and derived-feed pipeline was not rerun. Browser screenshots were unavailable; inspection used rendered HTML.Evidence
1.0.0.0.8.0plus Identity Service0.10.0.Merge Danger
Door: Two-way; documentation changes can be reverted.
Blast Radius: Deployment. Incorrect instructions can prevent customer installation or authentication; package rendering and connection checks cover the documented wiring.
Main branch synchronization
Pull request 4227 merged as
65dd861b1using a merge commit. The current documentation main (b01b5d9a2) now merges cleanly into this branch, preserving the earlier FIPS reconciliation. All seven reviewed self-managed pages are byte-identical to1edb30458; existing chart, shell, deployment-report, and review evidence remains applicable. The merged site passes the pinned Hugo build, seven-page checks, and 234 section links and anchors. No manual conflict edits or application changes were needed. The FIPS dependency is complete.Commits
8c0a67b— Merge current main after FIPS documentation lands; reviewed pages unchanged.6d6e397a8— Document installation with a standalone Identity Service.04a0aa18e— Align authentication, configuration, and operations with the separate releases.48e129192— Incorporate the FIPS branch and resolve its overlap now.1edb30458— Fix the deployed documentation report findings and require fresh stores for local provider changes.Note
Low Risk
Documentation-only changes with no application code; incorrect install steps could confuse operators but are reversible and validated against published charts.
Overview
Updates the seven self-managed Agent Memory docs for Memory chart
0.8.0and a separately installed Identity Service chart0.10.0. The deployment model no longer treats Identity Service as part of the Memory release; operators installidsfirst, provision shared Secrets (memory-dp-credential,memory-internal-token,ids-control-token), then wire Memory withdataplane.identityServiceBaseURL,identityService.credential, and matchingtenantId/singleTenantID.The Deploy walkthrough is rewritten for two Helm releases, TLS frontends (plus an HTTP-only lab path),
values-ids.yaml, worker idempotency (background_jobs.idempotency), combinedram-catrust, and health checks on/health/readinessinstead of/health. Authentication, configuration, and operations document operator-owned shared credentials, chart-owned introspection fields, stricter dev-onlyauth.method: none, cross-namespace connection, coordinated secret rotation via checksums, independent upgrades, unsupported direct upgrade from bundled0.7.0, and FIPS/network policy split across both charts.Prerequisites and reference add the second chart, mirror/air-gap split, expanded troubleshooting for removed
identityService.*/externalIdentityServicevalues, and API surface notes for liveness/readiness endpoints.Reviewed by Cursor Bugbot for commit 8c0a67b. Bugbot is set up for automated code reviews on this repo. Configure here.