Patch Review
Patch Review
Suggested Reviewers
Different developers have different strengths and areas of focus. Use the following table to decide who might be the best person to review your patch. You can then add them under the Reviewers section on the right hand side in github.
| Area | Reviewer |
|---|---|
| Attributes | clumens |
| Documentation | anyone |
| C refactoring | anyone |
| CTSlab | clumens |
| Command line tools | anyone |
| Fencing | |
| IPC | |
| Other daemons | narwahl |
| Packaging | |
| Python | clumens |
| Regression tests | |
| Scheduler | narwahl |
| Unit tests | anyone |
| XML |
- Last Author
- clumens
- Last Edited
- Jan 30 2025, 2:10 PM
Event Timeline
Comment Actions
There's a LOT that I still don't understand about the scheduler (especially with regard to colocation), but I've probably reviewed or modified more of that code than anyone who's still on the team.
Let's be especially cautious with non-trivial changes for a while.