Skip to content

Correct the memory-mappings log table and widen the fix guidance - #536

Merged
g-despot merged 1 commit into
mainfrom
docs/mmap-error-page-accuracy
Sep 9, 2026
Merged

g-despot merged 1 commit into
mainfrom
docs/mmap-error-page-accuracy

Conversation

@g-despot

@g-despot g-despot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #521 for the not-enough-memory-mappings entry, verified against weaviate/weaviate#12738 (including its review-fix pass) and the Helm chart.

  • The log table now lists the eight wired log sites with their exact current messages; the "Where the phrase is" column and the version-pinned claims are replaced by a search-for-the-phrase invariant.
  • The Kubernetes tab leads with the Helm chart's built-in sysctlInitContainer (its 524288 default sits below the recommended value), and a new Docker tab covers Linux hosts, the --sysctl refusal, and Docker Desktop's VM on Windows and macOS.
  • known-issues now recommends the same 4194304 as this page and defers platform steps here.

Review note: the three field-style table rows (apply command, failed to reload local index, error updating tenants) match weaviate/weaviate#12738 after its review-fix commit — if that PR's log shapes change again, re-check those rows.

- Replace the log table with the eight verified log sites and swap the
  version-pinned 'where the phrase is' column for a search-the-phrase
  invariant.
- Lead the Kubernetes tab with the Helm chart's built-in sysctl init
  container; note the DaemonSet must be privileged.
- Add a Docker tab: host sysctl on Linux, the --sysctl refusal, and
  Docker Desktop VM steps for Windows and macOS.
- Align known-issues to the same 4194304 recommendation and defer its
  platform steps to the errors page.

@orca-security-eu orca-security-eu Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 2   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca
🛡️ The following SAST misconfigurations have been detected
NAME FILE
medium Container Command Detected with Extended Privileges ...luster-resources.mdx View in code
medium Container Command Detected with Extended Privileges ...luster-resources.mdx View in code

@g-despot
g-despot merged commit 5994f77 into main Sep 9, 2026
13 checks passed
@g-despot
g-despot deleted the docs/mmap-error-page-accuracy branch September 9, 2026 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant