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

netwatcher :更改uprobe适配判断、增添redis监控信息 (incomplete) #835

Merged
merged 5 commits into from
Jun 21, 2024
Merged

netwatcher :更改uprobe适配判断、增添redis监控信息 (incomplete) #835

merged 5 commits into from
Jun 21, 2024

Conversation

zhangxianyu777
Copy link
Contributor

88a7a4c8d14f4cd1723495375d48866
目前有一点小问题需要解决 其结构为

 int argc; 
 robj **argv;
 unsigned long argv_len_sum;

目前能正确拉取 argc 与argv_len_sum，但不能拉取到argv中的命令名称
其结构为

typedef struct redisObject {
    unsigned type:4;
    unsigned encoding:4;
    unsigned lru:24; 
    int refcount;
    void *ptr;
} robj;

@zhangxianyu777
Copy link
Contributor Author

已修复、下一步将其整合输出至用户态、并拉取其处理时延
5cbb4442c6a2ee0fff26f9a6c957ea9

chenamy2017
chenamy2017 reviewed Jun 17, 2024
View reviewed changes
eBPF_Supermarket/Network_Subsystem/net_watcher/data/connects.log
Copy link
Contributor

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.

这个是将上次的删掉了 不是这次交的

helight
helight reviewed Jun 21, 2024
View reviewed changes
eBPF_Supermarket/Network_Subsystem/net_watcher/netwatcher.c
@@ -617,6 +627,14 @@ static void print_header(enum MonitorMode mode) {
printf("%-20s %-20s %-20s %-20s %-40s %-20s %-20s \n", "Pid", "Tid",
"Comm", "Size", "Sql", "Duration/μs", "Request");
break;
case MODE_REDIS:
printf("==============================================================="
"====================MYSQL "
Copy link
Member

Choose a reason for hiding this comment

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

这里MODE_REDIS 打印又是MYSQL？是不是错了呀

Copy link
Contributor Author

Choose a reason for hiding this comment

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

因为这个功能在测试阶段，这周将它总的改掉

helight
helight approved these changes Jun 21, 2024
View reviewed changes
Copy link
Member

@helight helight left a comment

Choose a reason for hiding this comment

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

ok

@helight helight merged commit e70dc5e into develop Jun 21, 2024
3 checks passed
@zhangxianyu777 zhangxianyu777 mentioned this pull request Jun 27, 2024
Open
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chenamy2017 chenamy2017 chenamy2017 approved these changes

@helight helight helight approved these changes

@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
@zhangxianyu777 @helight @chenamy2017