Skip to content

docs(etw): add SessionName parameter to service-etw configuration table#2666

Draft
yyuuttaaoo with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-review-comment-2580
Draft

docs(etw): add SessionName parameter to service-etw configuration table#2666
yyuuttaaoo with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-review-comment-2580

Conversation

Copilot AI commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

SessionName was added to EtwInput in this PR but omitted from the plugin documentation, leaving users unable to discover or correctly configure the feature.

Changes

  • docs/cn/plugins/input/extended/service-etw.md: Added SessionName row to the configuration parameters table, documenting its type (string), default value (empty string = system-assigned name), and the stop-and-recreate behavior triggered when a session with the same name already exists at startup.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI changed the title [WIP] Fix code based on review comment docs(etw): add SessionName parameter to service-etw configuration table Jul 21, 2026
Copilot AI requested a review from yyuuttaaoo July 21, 2026 08:35
@Takuka0311

Takuka0311 commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

百炼自动化审查:建议保持开启。

本 PR 为已合并的功能 PR #2580 补充缺失的 SessionName 参数中文文档。变更仅涉及单行文档更新,内容与代码实现一致,属于有效的文档完善工作。当前处于 Draft 状态,需维护者审查后合并。.

最佳落地路径:

保持开启。建议作者或维护者将本 PR 标记为 Ready for Review,核对文档格式与 #2580 实现细节一致后合并,以完善 service-etw 插件用户文档。.

已核对内容:

百炼审查备注:模型 qwen3.6-max-preview;对照提交 768cf9d283dc

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.

4 participants