Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MGDSTRM-10036] Add bf2_org_kafkaInstanceProfileType to Kafka instance metrics #832

Merged
merged 4 commits into from
Nov 15, 2022

Conversation

tinaselenge
Copy link
Contributor

@tinaselenge tinaselenge commented Nov 10, 2022

https://issues.redhat.com/browse/MGDSTRM-10036

ZookeeperContainersDown and KafkaBrokerContainersDown alerts rely on this label which is set on 'kafka_brokers' metric emitted from Kafka. However, if kafka containers are not running, this metric disappear. In this situation, the alerts never trigger which is counter to their purpose of alerting when containers are not running.

Instead of 'kafka_brokers', the alerts can use Kafka instance metrics emitted from kas-fleetshard so that it doesn't rely on Kafka itself.
Screenshot 2022-11-11 at 18 38 01

… metrics

ZookeeperContainersDown and KafkaBrokerContainersDown alerts rely on this label which is set on 'kafka_brokers' metric emitted from Kafka. However, if kafka containers are not running, this metric dissappear. In this situation, the alerts never trigger which is counter to their purpose of alerting when containers are not running.

Instead of 'kafka_brokers', the alerts can use kafka instance metrics emitted from fleetshard so that it doesn't rely on kafka itself.
@github-actions github-actions bot added the operator changes related to operator label Nov 10, 2022
@tinaselenge tinaselenge changed the title MGDSTRM-10036: Add bf2_org_kafkaInstanceProfileType to Kafka instance metrics [MGDSTRM-10036] Add bf2_org_kafkaInstanceProfileType to Kafka instance metrics Nov 10, 2022
@k-wall k-wall modified the milestone: 0.31.0 Nov 15, 2022
Shorten the label for consistency
@biswassri biswassri added this to the 0.31.0 milestone Nov 15, 2022
@sonarcloud
Copy link

sonarcloud bot commented Nov 15, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@tinaselenge tinaselenge merged commit a9bfbf9 into bf2fc6cc711aee1a0c2a:main Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
operator changes related to operator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants