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

fix(charts): sequencer prometheus rules #1598

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

quasystaty1
Copy link
Contributor

Summary

Fixes prometheus rules and metrics templates for both sequencer and sequencer-relayer charts

Background

ServiceMonitors for alerting and monitoring is missing on sequencer-relayer chart

Changes

  • adds service monitor and prometheus rules templating for sequencer-relayer
  • fixes sequencer app labels

Testing

locally, dusk-10

Metrics

-enables metrics and alerts on sequencer, sequencer-relayer charts

Related Issues

closes #1597

@quasystaty1 quasystaty1 requested a review from a team as a code owner September 30, 2024 15:41
@quasystaty1 quasystaty1 requested a review from aajimal September 30, 2024 15:41
@github-actions github-actions bot added the cd label Sep 30, 2024
@quasystaty1 quasystaty1 added this pull request to the merge queue Sep 30, 2024
Merged via the queue into main with commit a9f5197 Sep 30, 2024
43 checks passed
@quasystaty1 quasystaty1 deleted the quasystaty1/charts/seq-prometheus branch September 30, 2024 16:01
steezeburger added a commit that referenced this pull request Oct 7, 2024
* main: (34 commits)
  feat(proto): add bundle and optimistic block apis (#1519)
  feat(sequencer)!: make empty transactions invalid  (#1609)
  chore(sequencer): simplify boolean expressions in `transaction container` (#1595)
  refactor(cli): merge argument parsing and command execution (#1568)
  feat(charts): astrotrek chart (#1513)
  chore(charts): genesis template to support latest changes (#1594)
  fix(ci): code freeze action fix (#1610)
  chore(sequencer)!: exclusively use Borsh encoding for stored data (ENG-768) (#1492)
  ci: code freeze through github actions (#1588)
  refactor(sequencer): use builder pattern for transaction container tests (#1592)
  feat(conductor)!: implement chain ID checks (#1482)
  chore(ci): upgrade audit-check (#1577)
  feat(sequencer)!: transaction categories on UnsignedTransaction (#1512)
  fix(charts): sequencer prometheus rules   (#1598)
  chore(all): Migrate all instances of `#[allow]` to `#[expect]` (#1561)
  chore(charts,sequencer-faucet): asset precision (#1517)
  chore(docs): endpoints (#1543)
  fix(docker): use target binary build param as name of image entrypoint (#1573)
  fix(ci): ibc bridge test timeout (#1587)
  Feature: Add `graph-node` to charts. (#1489)
  ...
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.

fix(charts): sequencer-relayer serviceMonitor and prometheus
2 participants