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

plan to add workflow labels to the metric? #9849

Open
linkingli opened this issue Oct 17, 2022 · 4 comments
Open

plan to add workflow labels to the metric? #9849

linkingli opened this issue Oct 17, 2022 · 4 comments
Labels

Comments

@linkingli
Copy link

Summary

add workflow labels to the metric?

Use Cases

When I have two workflowTemplates (A,B), I want to know the overall situation of workflow of A and B respectively. The current metric does not support me to do this. Since there is no label for workflow.
NOW:
argo_workflows_count{container="workflow-controller", endpoint="metrics",
instance="127.0.0.1:9090", job="workflow-controller-metrics", namespace="argo",
pod="workflow-controller-7d69c4b564-t6xn8",
service="workflow-controller-metrics", status="Error"}
BETTER:
argo_workflows_count{container="workflow-controller", endpoint="metrics",
instance="127.0.0.1:9090", job="workflow-controller-metrics", namespace="argo",
pod="workflow-controller-7d69c4b564-t6xn8",
service="workflow-controller-metrics", status="Error",labelName="LabelValue"}


Message from the maintainers:

Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

@LilTwo
Copy link
Contributor

LilTwo commented Aug 8, 2023

Hi @terrytangyuan , is this a considerable enhancement?
If so, I can do some studies about it, currently there is no way to know pending workflow count since even with custom metrics, that is very inconvenient if you run many kinds of workflows in your cluster

@terrytangyuan
Copy link
Member

Yeah feel free to submit a proposal

@moonyoungCHAE
Copy link
Contributor

I found it can be work on custom metrics. And I'm worried about that if workflow name is added on default metric labels, it can be create huge amount of metrics.

@LilTwo
Copy link
Contributor

LilTwo commented Aug 26, 2023

Hi @terrytangyuan
opened a PR, please take a look if you have time, thanks!

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 a pull request may close this issue.

5 participants