Page MenuHomeClusterLabs Projects
Create Task
Maniphest T890

Log errors in pcmk_parse_interval_spec() callers
Open, WishlistPublic
Actions

Description

Currently, the function logs errors itself. However it has no context, so the message is unspecific. To help users better, drop that log and have all callers check the return code and log an appropriate error. Also, the function should return pcmk_rc_bad_input rather than EINVAL since it's a user error rather than coding error.

Event Timeline

kgaillot triaged this task as Wishlist priority.Oct 2 2024, 3:37 PM
kgaillot created this task.
kgaillot created this object with edit policy "Restricted Project (Project)".
clumens assigned this task to waltdisgrace.Mar 18 2025, 2:55 PM
clumens added a subscriber: clumens.

Additionally, it would be nice to have a unit test or two for this function. Most of what it does is call another function, several of which are already unit tested, but checking that the inputs/outputs are as expected would be good.