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

feat: telemetry #8670

Merged
merged 61 commits into from
Mar 24, 2023
Merged

feat: telemetry #8670

merged 61 commits into from
Mar 24, 2023

Conversation

odysa
Copy link
Contributor

@odysa odysa commented Mar 21, 2023

No description provided.

@odysa
boil telemetry
37871ec
@odysa
sys info and ticker
b6d5252
@odysa
add telemetry starting
0cac0e8
@odysa
move to start_leader_srv
f8b7eac
@odysa
post telemetry report
085d146
@odysa
add telemetry_enabled
396bb50
@odysa
refactor telemetry loop
de56427
@odysa
add telemetry fail test
17b0842
@odysa
add telemetry rpc
991135a
@odysa
move telemetry to common
e3f347b
@odysa
refactor reporting loop
b41da80
@odysa
add telemetry rpc service
93c1c9b
@odysa
Merge branch 'main' into feature/telemetry
eb3ef04
@odysa
refactor telemetry config
be5e248
@odysa
refactor telemetry report
a096021
@odysa
add frontend telemetry
6caee0c
@odysa
refactor report creator
8453623
@odysa
refactor url
50fd0bd
@odysa
Merge branch 'main' into feature/telemetry
f3d2c05
@odysa
etcd config
87ef555
@odysa
incorpa config
93e638c
@odysa
fix lience header
433d122
@odysa
add post telemetry tests
31dc7fe
@odysa
fix dashboard proto
c14902b
@odysa
add to system params
c29d8ba
@odysa
Merge branch 'main' into feature/telemetry
99b9f71
@odysa
merge main
bfc0c2c
@odysa
introduce telemetry mgr
4dbd365
@odysa
refacotor system params
5ee5f8f
@odysa
refactor meta reporting and telemetry info fetching
b7e9fb7
odysa and others added 17 commits March 14, 2023 14:03
@odysa
fix dependecies
8e985a2
@odysa
fix borrow issues
0a986ba
@odysa
Merge branch 'main' into feature/telemetry
4a5b39e
@odysa
Merge branch 'main' into feature/telemetry
2bd09ca
@odysa
add tracking_id to backup
41358ea
@odysa
interval and starting change
d944b6c
@odysa
Merge branch 'main' into feature/telemetry
cfa3324
@odysa
disable in risedev and revert backup
3e54af6
@odysa
disable in integration tests
013b392
@odysa
fix integration test env
5c67e35
@odysa
fix check
dec9296
@odysa
Merge branch 'main' into feature/telemetry
d0f8933
@Li0k @yuhao-su
@chenzl25 @xx01cyx @ZENOTME @mergify @wcy-fdu
feat(storage): basic compactor scheduler policy (risingwavelabs#6986)
ae99e55 
Co-authored-by: Yuhao Su <31772373+yuhao-su@users.noreply.github.com>
Co-authored-by: Dylan <chenzl25@mail2.sysu.edu.cn>
Co-authored-by: Yuanxin Cao <60498509+xx01cyx@users.noreply.github.com>
Co-authored-by: ZENOTME <43447882+ZENOTME@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: congyi wang <58715567+wcy-fdu@users.noreply.github.com>
@y-wei
feat(optimizer): optimize always-false filter for batch (risingwavela…
cc6e687 
…bs#8629)

Signed-off-by: Clearlove <yifei.c.wei@gmail.com>
@odysa
Merge branch 'main' into feature/telemetry
68d0317
@odysa
refine env tests
0fe4322
@odysa
fix meta reporting
64778d7
@mergify
Copy link
Contributor

mergify bot commented Mar 21, 2023

⚠️ The sha of the head commit of this PR conflicts with #7384. Mergify cannot evaluate rules on this PR. ⚠️

@hzxa21
Copy link
Collaborator

hzxa21 commented Mar 21, 2023

cc @lmatz Can we merge this PR onto v0.18.0-rc so that we can have telemetry enabled starting from this release?

@wenym1
Copy link
Contributor

wenym1 commented Mar 21, 2023

The number of line changes is different to in #7384.

Seems that we have included some code in main that is after v0.18.0-rc.

@wenym1 wenym1 requested a review from lmatz March 23, 2023 07:19
@lmatz
Copy link
Contributor

lmatz commented Mar 23, 2023

Could you remove the unused import

warning: unused import: `uuid::Uuid`
  --> src/meta/src/rpc/service/telemetry_service.rs:22:5
   |
22 | use uuid::Uuid;

and we are good to go

I tried to push the changes but found the PR comes from your forked repo

@lmatz lmatz merged commit cbd3403 into risingwavelabs:v0.18.0-rc Mar 24, 2023
@hengm3467 hengm3467 mentioned this pull request Apr 3, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lmatz lmatz lmatz approved these changes

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

Successfully merging this pull request may close these issues.

None yet
6 participants
@odysa @hzxa21 @wenym1 @lmatz @Li0k @y-wei