anyone opposing to change Noop('Edge would cause a cycle: {} -> {}.', src, dst) to Noop(‘Cycle detected! Edge would ……’)? an integration test is looking for “Cycle detected” string, and it is used in v1 engine.