Micrometer: Router policy should be disabled by default (to be consistent with Camel)

[JiriOndrusek]

Bug description

As you can see in Camel documentation, RoutePolicyFactory is not enabled by default. CQ should be consistent with this behavior and change default value for property camel.metrics.enableRoutePolicy to false (code)

[jamesnetherton]

The docs state that the default is true:

https://camel.apache.org/components/3.20.x/micrometer-component.html#_sb_option_camel_metrics_enable-route-policy

[JiriOndrusek]

Default is true for CSB, unfurtunately for plain Camel default is false. Because users have to set routePolicy, like in the test. (Explanation in the doc is From Java you just add the factory to the CamelContext as shown below: context.addRoutePolicyFactory(new MicrometerRoutePolicyFactory());

Therefore CQ should default to false and ticket should be reported for CSB also.

[zhfeng]

@JiriOndrusek it can be closed? or is there anything about document we need to do?

[JiriOndrusek]

@zhfeng thanks for noticing the issue!

TBH I still think that my previous comment is right. Let me explain:

• Micrometer Router policy in Camel is disabled by default, see documentation, following code has to be used to enable policy: context.addRoutePolicyFactory(new MicrometerRoutePolicyFactory());.
• Micrometer Router policy in CSB is enabled by default, see documentation
• Micrometer Router policy in CQ is currently enabled by default, but I think, that it should be disabled (and therefore disabled with plain Camel)

I'll prepare a PR which changes this behavior as I suggested above. Therefore we should keep this issue opened.