Skip to content

Conversation

@NickLarsenNZ
Copy link
Member

@NickLarsenNZ NickLarsenNZ commented Mar 20, 2025

Description

This PR cleans up instrumented functions and tracing events.

Screenshots

Before

stackablectl -l trace operator list # version 24.11.3

image

After

cargo run -p stackablectl -- -l trace operator list

image

@NickLarsenNZ NickLarsenNZ force-pushed the chore/clean-instrumentation branch from 73f1729 to 74f9a4b Compare March 20, 2025 12:47
@NickLarsenNZ NickLarsenNZ force-pushed the chore/clean-instrumentation branch from 8458fe8 to 2c72e64 Compare March 21, 2025 12:52
@NickLarsenNZ NickLarsenNZ marked this pull request as ready for review March 21, 2025 13:10
@NickLarsenNZ NickLarsenNZ requested a review from Techassi March 21, 2025 13:10
@NickLarsenNZ NickLarsenNZ moved this to Development: Waiting for Review in Stackable Engineering Mar 21, 2025
@Techassi Techassi moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Mar 21, 2025
Copy link
Member

@Techassi Techassi left a comment

Choose a reason for hiding this comment

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

Nice work, just some small comments.

@NickLarsenNZ NickLarsenNZ requested a review from Techassi March 27, 2025 08:55
@Techassi Techassi added this pull request to the merge queue Mar 27, 2025
Merged via the queue into main with commit 063f4c3 Mar 27, 2025
26 checks passed
@Techassi Techassi deleted the chore/clean-instrumentation branch March 27, 2025 09:10
@Techassi Techassi moved this from Development: In Review to Development: Done in Stackable Engineering Mar 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants