Skip to content
This repository was archived by the owner on Oct 25, 2023. It is now read-only.

Do not expose all available metrics by default #32

Closed
Dabz opened this issue May 18, 2020 · 0 comments · Fixed by #35
Closed

Do not expose all available metrics by default #32

Dabz opened this issue May 18, 2020 · 0 comments · Fixed by #35

Comments

@Dabz
Copy link
Owner

Dabz commented May 18, 2020

The number of metrics available in Confluent Cloud Metrics API is increasing every day. As we have more and more data, exposing all of them does not make sense. Instead, we should:

  • Have a default configuration with the most important metrics
  • Have a way to expose new metrics (either through the command line or through a configuration file).
@Dabz Dabz mentioned this issue May 22, 2020
Merged
@Dabz Dabz mentioned this issue Jun 4, 2020
Merged
Dabz added a commit that referenced this issue Jun 4, 2020
@Dabz
This PR introduces a configuration file to the ccloudexporter, this a…
74f3a91 
…lso address many issues:

    #24 - Allow to export data from multiple clusters
    #31 - Remove global grouping per partition and allow grouping per partition for pre-defined topics
    #32 - Do not available all available by default for stability
    #34 - Add a header to identify the ccloudexporter

I will do some rebase to cleanup the git branch before merging.
@Dabz Dabz linked a pull request Jun 5, 2020 that will close this issue
Configuration file #35
Merged
@Dabz Dabz closed this as completed in #35 Jun 5, 2020
Dabz added a commit that referenced this issue Jun 5, 2020
@Dabz
Configuration file (#35)
f3e8daf 
* This PR introduces a configuration file to the ccloudexporter, this also addresses many issues:

    #24 - Allow to export data from multiple clusters
    #31 - Remove global grouping per partition and allow grouping per partition for pre-defined topics
    #32 - Do not available all available by default for stability
    #34 - Add a header to identify the ccloudexporter
@Dabz Dabz mentioned this issue Jul 20, 2020
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Configuration file Dabz/ccloudexporter
1 participant
@Dabz