Hi team, I create two issues and prs inside for re...
# contributeb
Byron Hsu
04/03/2023, 5:09 PMHi team, I create two issues and prs inside for review:
1. [Core feature] Support horovod in mpi task cc: @Kevin Su @Dan Rammer (hamersaw)
2. [Core feature] Add user id to security context cc @Yee needs your pointer about the middleware
Byron Hsu
04/03/2023, 7:33 PMThanks @Kevin Su for the review.
Left a question https://github.com/flyteorg/flytekit/pull/1575/files#r1156378225
Byron Hsu
04/03/2023, 8:35 PM@Yee can you review the 2nd one when you get some time? thanks in advance
Byron Hsu
04/04/2023, 5:44 AM@Kevin Su addressed the comment
Byron Hsu
05/02/2023, 5:06 AMhttps://github.com/flyteorg/flyteidl/pull/388
https://github.com/flyteorg/flyteadmin/pull/549/files
Addressed the comment and added e2e test for the user identifier
@Yee
Byron Hsu
05/03/2023, 3:51 PM@Mick Jermsurawong
m
Mick Jermsurawong
05/03/2023, 5:06 PMthanks Byron for the context here! So if i understand this correctly:
From UI, or flytekit you'll be able to add
user_identifierMick Jermsurawong
05/03/2023, 5:07 PM(Would there be a change in console to show the user identity as well?)
b
Byron Hsu
05/03/2023, 5:08 PMNo no
Byron Hsu
05/03/2023, 5:09 PMQuick chat?
Byron Hsu
05/03/2023, 5:09 PMUser_id will be parsed from oauth access token
151 Views