Skip to content

docs(helm): correct Vermeer log instructions - #3440

Merged
imbajin merged 1 commit into
apache:masterfrom
hugegraph:codex/fix-vermeer-log-guide
Oct 31, 2026
Merged

imbajin merged 1 commit into
apache:masterfrom
hugegraph:codex/fix-vermeer-log-guide

Conversation

@imbajin

@imbajin imbajin commented Oct 30, 2026 •

Copy link
Copy Markdown
Member

Before

The Helm guide points to a nonexistent Vermeer log file.

After

Document stdout and kubectl logs for the master and worker containers.

Validation and scope

Author-reported validation is recorded in the source PR; independent dots review and exact-head ASF CI are pending. No live deployment or RC acceptance is claimed.

Source review: hugegraph/hugegraph#360. This PR uses the same source branch; see that discussion for detailed validation evidence and limitations.

Visual overview

Vermeer 日志写入 stdout;使用各容器的 kubectl logs。数据目录不是日志落盘位置。 机制示意;不表示独立实测、CI 或同一 RC 已通过。评审核对:2026-10-31。

机制示意

@bitflicker64 bitflicker64 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.

Blocking: no. Score: 9/10. Summary: the stdout and kubectl logs guidance matches the chart, which names the containers master and worker and runs both in one Pod in standalone mode, and Vermeer's InitLogger, which sends logrus output to os.Stdout and never reads log_prefix.

@imbajin
imbajin merged commit e15a058 into apache:master Oct 31, 2026
11 checks passed
@imbajin
imbajin deleted the codex/fix-vermeer-log-guide branch October 31, 2026 17:54
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