Skip to content

rust: update egui_*#79

Merged
florianl merged 2 commits intomainfrom
egui-update
Mar 18, 2026
Merged

rust: update egui_*#79
florianl merged 2 commits intomainfrom
egui-update

Conversation

@florianl
Copy link
Member

@florianl florianl commented Mar 16, 2026

This is an AI assisted updated of the Rust dependencies around egui.

This update is a replacement for #14, #13, #12, #11 and #9.

This is an AI assisted updated of the Rust dependencies around egui.

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@florianl florianl requested a review from a team as a code owner March 16, 2026 10:28
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@christos68k
Copy link
Member

Which revision of ebpf-profiler did you use to test this? I've tried 63d24dc4dd, 534ce0d3e1f and 8c9672cc2 and getting the following:

time=2026-03-16T14:44:47.741-04:00 level=ERROR msg="Request failed: rpc error: code = Internal desc = failed to decode Protobuf message: Sample.link_index: Profile.samples: ScopeProfiles.profiles: ResourceProfiles.scope_profiles: ExportProfilesServiceRequest.resource_profiles: invalid wire type: LengthDelimited (expected Varint)"

@florianl
Copy link
Member Author

Since e755242 all current versions of ebpf-profiler are expected to result in errors like invalid wire type: LengthDelimited (expected Varint), as ebpf-profiler still uses proto v1.9.0 while this project moved on with e755242 to v1.10.0.

As long as ebpf-profiler does not use OTel collector v0.148.0 (to be released in this week) only current main of OTel collector and OTel collector contrib work with proto v1.10.0.

Copy link
Member

@christos68k christos68k left a comment

Choose a reason for hiding this comment

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

Thanks, works for me!

@florianl florianl merged commit 971d1de into main Mar 18, 2026
4 checks passed
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.

2 participants