Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

Cometbft: enable prometheus exporter #149

Merged
merged 12 commits into from
Jan 9, 2024

Conversation

WafflesVonMaple
Copy link
Contributor

No description provided.

@WafflesVonMaple WafflesVonMaple changed the title enable prometheus exporter Cometbft: enable prometheus exporter Jan 3, 2024
@WafflesVonMaple WafflesVonMaple marked this pull request as ready for review January 3, 2024 20:01
@WafflesVonMaple
Copy link
Contributor Author

@joroshiba worth chatting on this one since we are kinda crafting cometbft ourselves here. Worth looking at cometbft/cometbft#899

Copy link
Member

@joroshiba joroshiba left a comment

Choose a reason for hiding this comment

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

mostly new lines to be fixed, couple open questions.

charts/sequencer/templates/_helpers.tpl Outdated Show resolved Hide resolved
charts/sequencer/templates/service.yaml Outdated Show resolved Hide resolved
charts/sequencer/templates/servicemonitor.yaml Outdated Show resolved Hide resolved
charts/sequencer/values.yaml Outdated Show resolved Hide resolved
namespace: {{ .Values.global.namespace }}
spec:
selector:
app: {{ .Values.config.moniker }}-sequencer
Copy link
Member

Choose a reason for hiding this comment

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

Do we need to update this and other areas to use the new helpers name and labels?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we dont have to do it this way at all either, was just a fast way to ensure service monitor selectors picked up the right app name

@WafflesVonMaple WafflesVonMaple merged commit 56d229d into main Jan 9, 2024
1 check passed
@WafflesVonMaple WafflesVonMaple deleted the cometbft_prometheus_enable branch January 9, 2024 22:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants