Skip to content

NMS-19927: Metadata-Driven Kafka Metric Topic Routing - #8823

Open
christianpape wants to merge 2 commits into
foundation-2026from
jira/NMS-19927
Open

NMS-19927: Metadata-Driven Kafka Metric Topic Routing#8823
christianpape wants to merge 2 commits into
foundation-2026from
jira/NMS-19927

Conversation

@christianpape

Copy link
Copy Markdown
Contributor

@cgorantla cgorantla left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. Some comments and also this needs to be re-targeted on foundation-2026

description = "Show the Kafka topic that metrics for the given node, interface or service are routed to.")
@Service
@SuppressWarnings("java:S106")
public class MetricRoutingTest implements Action {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check if this still works if metric routing is disabled. Users might want to run this command without enabling routing.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works when metric routing is disabled. In this case, a message appears stating that it is disabled and that the default theme is being used.

String metricTopic = null;
boolean disableMetricsSplitting = false;
String metricFilter = null;
boolean metricRoutingEnabled = false;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One thing to note with activator, this might not reload bundle when config gets updated. So if config gets updated after feature resolution, bundle/feature might need to be restarted

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a note to the docs.

@christianpape
christianpape changed the base branch from develop to foundation-2026 August 31, 2026 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants