Comment 12 for bug 2031482

> if not node.is_commissioning():
>
> Which supposedly indicates that the node is a controller in case it is true. It is not intuitive that not > is_commissioning equals that the node is a controller so it is possible that this not always works and
> thus sometimes the code path for controller is being taken for non-controller nodes.

This logic is also executed for hardware-sync. Do you have any nodes deployed with hardware-sync enabled?