oxidecomputer / oxidecomputer/crucible
Make DTrace share common functions
Open
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 260
- Forks
- 34
- Avg merge
- 2d 1h
- Merged PRs (30d)
- 8
Description
Let me see if I can find an easier way.. in a follow up commit :)
Originally posted by @leftwo in https://github.com/oxidecomputer/crucible/pull/1773#discussion_r2389296590
Can we share some common function between all or many of the DTrace scripts, without too much insanity?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the DTrace scripts and the linked pull request discussion to identify which functions are duplicated and whether they can be shared cleanly. Done means common functions are consolidated where practical without making the scripts harder to understand.
Written by the indexing model from the issue text.
Assessment
- Domain
- observability
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100