Skip to content

docs: document memory profiling startup env#2589

Merged
v0y4g3r merged 3 commits into
mainfrom
docs/mention-prof-env
Jun 29, 2026
Merged

docs: document memory profiling startup env#2589
v0y4g3r merged 3 commits into
mainfrom
docs/mention-prof-env

Conversation

@v0y4g3r

@v0y4g3r v0y4g3r commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Document that Memory Profile targets must start with jemalloc profiling enabled.
  • Add command-line and Helm values examples for setting the profiling environment variable.
  • Update current and versioned English and Chinese Continuous Profiling docs.

Test Plan

  • Not run; PR creation requested from existing branch.

Add Continuous Profiling guidance that memory profile targets need jemalloc profiling enabled at process startup.

Include command-line and Helm values examples for setting the required environment variable.

Files: docs/enterprise/console-ui/continuous-profiling.md, versioned_docs/version-1.1/enterprise/console-ui/continuous-profiling.md, i18n/zh/docusaurus-plugin-content-docs/current/enterprise/console-ui/continuous-profiling.md, i18n/zh/docusaurus-plugin-content-docs/version-1.1/enterprise/console-ui/continuous-profiling.md
Signed-off-by: Lei, HUANG <ratuthomm@gmail.com>
@v0y4g3r v0y4g3r requested a review from fengjiachun as a code owner June 29, 2026 03:01
Copilot AI review requested due to automatic review settings June 29, 2026 03:01
@v0y4g3r v0y4g3r requested a review from killme2008 as a code owner June 29, 2026 03:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Continuous Profiling documentation to clarify that memory profiling requires jemalloc profiling to be enabled at process startup, and provides examples for setting the required environment variable for CLI and Helm deployments across current and versioned (v1.1) English/Chinese docs.

Changes:

  • Add a note that Memory Profile targets must start with jemalloc profiling enabled.
  • Add CLI example (MALLOC_CONF=prof:true) and Helm values example (base.podTemplate.main.env) for enabling profiling.
  • Apply the same documentation updates to current + v1.1, EN + ZH copies.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
docs/enterprise/console-ui/continuous-profiling.md Adds jemalloc/MALLOC_CONF startup requirement and CLI/Helm examples for Memory Profiles (current EN).
versioned_docs/version-1.1/enterprise/console-ui/continuous-profiling.md Same as above for v1.1 EN docs.
i18n/zh/docusaurus-plugin-content-docs/current/enterprise/console-ui/continuous-profiling.md Same as above for current ZH docs.
i18n/zh/docusaurus-plugin-content-docs/version-1.1/enterprise/console-ui/continuous-profiling.md Same as above for v1.1 ZH docs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/enterprise/console-ui/continuous-profiling.md Outdated
Comment thread versioned_docs/version-1.1/enterprise/console-ui/continuous-profiling.md Outdated
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 29, 2026

Copy link
Copy Markdown

Deploying greptime-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: a9898d3
Status: ✅  Deploy successful!
Preview URL: https://a552793d.greptime-docs.pages.dev
Branch Preview URL: https://docs-mention-prof-env.greptime-docs.pages.dev

View logs

Replace the external GreptimeDB memory profiling URL in Continuous Profiling docs with the local performance tuning memory profiling link.

Signed-off-by: Lei, HUANG <ratuthomm@gmail.com>
Update Continuous Profiling docs to require dashboard v0.2.0-alpha.8.

Signed-off-by: Lei, HUANG <ratuthomm@gmail.com>
@v0y4g3r v0y4g3r merged commit 7ca9adb into main Jun 29, 2026
5 checks passed
@v0y4g3r v0y4g3r deleted the docs/mention-prof-env branch June 29, 2026 06:59
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.

3 participants