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.

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

add qps stats in report #273

Merged
merged 2 commits into from
Nov 2, 2023
Merged

add qps stats in report #273

merged 2 commits into from
Nov 2, 2023

Conversation

ii2day
Copy link
Collaborator

@ii2day ii2day commented Oct 27, 2023

feature #269
fix #296
fix #297
1.添加任务执行 agent 中 qps 的统计,包含当前本任务,任务开始前日志打印 qps 统计,任务结束再 次获取 qps 统计,并写入报告
2.修改报告中 cpu memory 结构体,方便引用。
3.修改 netdns request中 数据类型为 int 与 http 保持一致
4.方便查看 e2e 结果,将e2e 中 task 报告打印格式为 yaml
5.删除重复用例

@codecov
Copy link

codecov bot commented Oct 27, 2023

Codecov Report

Merging #273 (c3209d9) into main (59b6879) will increase coverage by 0.47%.
Report is 2 commits behind head on main.
The diff coverage is 59.09%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #273      +/-   ##
==========================================
+ Coverage   39.52%   40.00%   +0.47%     
==========================================
  Files           8        8              
  Lines         544      555      +11     
==========================================
+ Hits          215      222       +7     
- Misses        323      326       +3     
- Partials        6        7       +1     
Flag Coverage Δ
unittests 40.00% <59.09%> (+0.47%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
pkg/loadRequest/loadDns/dns_requester.go 85.89% <59.09%> (-1.69%) ⬇️

@ii2day ii2day force-pushed the pr/ii2day/stats_qps branch 4 times, most recently from d58b83a to ce86142 Compare October 30, 2023 07:58
@ii2day
Copy link
Collaborator Author

ii2day commented Nov 1, 2023

image

@ii2day ii2day changed the title add other task qps stats in report add qps stats in report Nov 1, 2023
@ii2day ii2day force-pushed the pr/ii2day/stats_qps branch 3 times, most recently from d6d3d66 to 5c8845a Compare November 2, 2023 03:35
@ii2day ii2day force-pushed the pr/ii2day/stats_qps branch 5 times, most recently from 3559073 to 2028126 Compare November 2, 2023 08:12
Signed-off-by: ii2day <ji.li@daocloud.io>
Signed-off-by: ii2day <ji.li@daocloud.io>
@weizhoublue weizhoublue merged commit 8f17d25 into main Nov 2, 2023
24 of 25 checks passed
@weizhoublue weizhoublue deleted the pr/ii2day/stats_qps branch November 2, 2023 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

software bug. resource stats data race software bug dns client date race
2 participants