Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Mem_watcher:增加项目的测试程序 #855

Merged
merged 2 commits into from
Jul 10, 2024
Merged

Mem_watcher:增加项目的测试程序 #855

merged 2 commits into from
Jul 10, 2024

Conversation

Super-Lzzx
Copy link
Contributor

92d97cd945c575e3af7ccb20445673b ad9245f74b21749ab4b5236fd8c327d

helight
helight reviewed Jul 5, 2024
View reviewed changes
eBPF_Supermarket/Memory_Subsystem/docs/mem_watcher_vision.md Outdated
@@ -17,7 +17,7 @@
- 执行`make start_service`指令，配置下载docker镜像并启动grafana和prometheus服务
- 执行如下指令开始采集数据以及相关处理：
```c
./data-visual collect /home/ubuntu/lmp/eBPF_Supermarket/Memory_Subsystem/mem_watcher -p
./data-visual collect /home/ubuntu/lmp/eBPF_Supermarket/Memory_Subsystem/mem_watcher/mem_watcher -p
Copy link
Member

Choose a reason for hiding this comment

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

这里使用相对路径比较好一点

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的

Copy link
Member

Choose a reason for hiding this comment

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

这里修改一下，我再合并代码哈

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好！

chenamy2017
chenamy2017 previously approved these changes Jul 8, 2024
View reviewed changes
@helight helight merged commit d8d193c into linuxkerneltravel:develop Jul 10, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@helight helight helight approved these changes

@chenamy2017 chenamy2017 chenamy2017 left review comments

@LinkinPF LinkinPF Awaiting requested review from LinkinPF LinkinPF is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Super-Lzzx @helight @chenamy2017