Skip to content

rust blobs: fix differences#434

Merged
florianl merged 1 commit intomainfrom
fix-rust-blobs
Apr 8, 2025
Merged

rust blobs: fix differences#434
florianl merged 1 commit intomainfrom
fix-rust-blobs

Conversation

@florianl
Copy link
Copy Markdown
Member

@florianl florianl commented Apr 8, 2025

#411 got created before Rust binary blobs were checked for differences but got merged after this check got in place. As #411 introduces some changes to the Rust code, the Rust binary blobs change and therefore the check for differences fails. Fix this by compiling the Rust binary blobs.

This PR fixes the failing CI job on main: https://github.com/open-telemetry/opentelemetry-ebpf-profiler/actions/runs/14328632510

#411 got created before Rust binary blobs were checked for differences but got merged after this check got in place. As #411 introduces some changes to the Rust code, the Rust binary blobs change and therefore the check for differences fails.
Fix this by compiling the Rust binary blobs.

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@florianl florianl requested review from a team as code owners April 8, 2025 09:03
@florianl florianl enabled auto-merge (squash) April 8, 2025 16:25
@florianl florianl merged commit ead430a into main Apr 8, 2025
26 checks passed
@florianl florianl deleted the fix-rust-blobs branch April 8, 2025 17:44
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.

3 participants