https://flyte.org logo
#announcements
Title
# announcements
p

Prada Souvanlasy

07/18/2022, 8:59 AM
hello! Is there a mean to add custom tags on the Flyte provided metrics mentioned there? The only available tag is currently the workflow name (
wf
), but we would like to add dimensions to the relevant metrics depending of the workflow inputs or the custom labels provided while submitting the workflow. Those labels are only added on the relevant pods as far as I saw but that would be great if those labels could be somehow added to the metrics as well
d

Dan Rammer (hamersaw)

07/18/2022, 12:52 PM
Hi @Prada Souvanlasy! Currently, the prometheus metric labels are configurable in FlytePropeller with this configuration option. However, the only supported options are here. We have these restrictions because prometheus best practices are to keep metric labels to very low cardinality. Because of the way things are internally tracked memory utilization can grow very quickly if we added metric labels for things like workflow inputs, etc. What kind of custom labels are you looking for?
p

Prada Souvanlasy

07/18/2022, 1:01 PM
hi, thanks for your answer! The Prometheus best practices regarding the labels make sense and in our case we would not end with an unreasonable number of labels. A basic use-case example would be executing a given workflow several times on different tenants. The inputs would change ofc, however there is no need to add all inputs as labels. We would have been very fine with simply adding a
tenant_id
label to the metrics, that we could provide at the same time as the workflow inputs while submitting the execution
i.e something that behaves more or less the same as the Kubernetes pods labels/annotations we could add through https://docs.flyte.org/projects/flytekit/en/latest/generated/flytekit.remote.remote.Options.html#flytekit.remote.remote.Options
k

Ketan (kumare3)

07/18/2022, 1:53 PM
@Prada Souvanlasy you can add labels to an execution and those will get added to the pods
That should then be available to the pods
p

Prada Souvanlasy

07/18/2022, 1:55 PM
hi @Ketan (kumare3), I know. We would actually need the same for the metrics
k

Ketan (kumare3)

07/18/2022, 1:55 PM
Custom metrics on the workflow sounds like a dangerous recipe- we recently found some users had bloated cardinality - which becomes expensive and needs more memory
Can you not use project as the tenant I'd?
p

Prada Souvanlasy

07/18/2022, 2:01 PM
no, we can expect to have dozens of tenants. We would like to stick with one project (and its related domains) and publish/run workflows from there. Only the inputs (and labels) would change from a tenant to another. Additionally, the tenant id was a simple example but some other dimensions are needed as well. The concern you both mentioned is understandable, did you consider limiting the number of metrics custom labels at runtime so an execution could fail fast if it exceeds that limit?
k

Ketan (kumare3)

07/18/2022, 2:04 PM
@Prada Souvanlasy you can use one project to publish but execute in a separate project
p

Prada Souvanlasy

07/18/2022, 2:08 PM
I guess those projects would need to be created beforehand, is that right? I don't think we would like to add that additional maintenance on our side each time a tenant needs to be added. Regardless, as I mentioned, that would not cover the actual use-case since the tenant id is not the only dimension we need to add. In our precise case, we got 3 business dimensions we need to track
k

Ketan (kumare3)

07/18/2022, 3:16 PM
Why don't you propose a change, all the code is in flytestdlib
I think we should be careful
29 Views