https://flyte.org logo
#contribute
Title
# contribute
f

Fabio Grätz

12/11/2023, 12:19 PM
@Byron Hsu and co added logic in flyteadmin a while ago to inject a user identifier from the auth token to ExecutionSpec. The user ID is added to the flyteworkflow custom resource but propeller currently doesn’t add it to pods, e.g. as a label. For us it would be very useful to have this info in the pod as a label, e.g. to attribute execution cost to users and in order to link experiments in our experiment tracking server to user accounts. Would it be reasonable in your opinion to add this user identifier to the pod?
k

Ketan (kumare3)

12/11/2023, 4:21 PM
@Fabio Grätz can this dovetail with the threaded auth discussion
f

Fabio Grätz

12/11/2023, 4:27 PM
b

Byron Hsu

12/11/2023, 6:11 PM
I feel the discussion ^^ is much broader involving a new agent. what @Fabio Grätz needs it simply just propogate user info to the pod. +1 for adding that to the pod as long as it doesn’t break existing cases
f

Fabio Grätz

12/14/2023, 4:08 PM
Yes, my question wasn’t about using user credentials within the task but just labelling the task pod with the user id
b

Byron Hsu

12/15/2023, 12:39 AM
imo it doesn’t hurt if we add it
f

Fabio Grätz

12/15/2023, 11:14 AM
In case the others agree, would you be willing to upstream your existing logic @Byron Hsu?
b

Byron Hsu

12/15/2023, 6:06 PM
our codebase is a bit messy now XD it might be easier if you make the change on your side and upstream