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

[RHCLOUD-37402] Replace Grafana lag metrics with AWS MSK and use timeseries pansel #435

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jessicarod7
Copy link
Member

@jessicarod7 jessicarod7 commented Jan 28, 2025

What?

https://issues.redhat.com/browse/RHCLOUD-37402

  1. Replace outdated metrics in Grafana panels with new metrics pulled from AWS MSK
  2. Replace deprecated Grafana graph (AngularJS) panels with timeseries

Why?

  • The datasource for the existing Kafka metrics is being deprecated, so it must be replaced
  • Plots using the AngularJS "graph" are being deprecated

How?

  1. Update "Consumer lag" panel from kafka_consumergroup_group_lag to aws_kafka_sum_offset_lag_sum metric
  2. Switch "Request rate", "Errors", and "Connector errors" panels from deprecated "Graph" type to "Time series"
    • Update "RDS storage usage panel to use current AppSRE cluster as datasource, and use "Time series" type

Testing

Visualized while making edits in Grafana.

Secure Coding Practices Checklist Link

Secure Coding Checklist

  • Input Validation
  • Output Encoding
  • Authentication and Password Management
  • Session Management
  • Access Control
  • Cryptographic Practices
  • Error Handling and Logging
  • Data Protection
  • Communication Security
  • System Configuration
  • Database Security
  • File Management
  • Memory Management
  • General Coding Practices

@jessicarod7
Copy link
Member Author

not included in this PR would be switching the "Request rate" panel over to the metric api_3scale_gateway_api_status, but I'm not sure if that's appropriate for this service

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant