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
feat(server): add ServiceAccount info to api/v1/userinfo and ui user tab #4944
feat(server): add ServiceAccount info to api/v1/userinfo and ui user tab #4944
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs @jessesuen approval for proto changes.
pkg/apiclient/info/info.proto
Outdated
@@ -26,6 +26,7 @@ message GetUserInfoResponse { | |||
repeated string groups = 3; | |||
string email = 4; | |||
bool emailVerified = 5; | |||
string serviceAccount = 6; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should probably be consistent with PodSpec which would be serviceAccountName:
https://kubernetes.io/docs/tasks/configure-pod-container/configure-service-account/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree with @alexec. Conditional approval provided we rename to serviceAccountName.
I think e2e test failure is unrelated. Re-running. |
Please sync with master to fix conflicts introduced by v2. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs rebase/merge from master.
63f8bbd
to
01e2988
Compare
Signed-off-by: Tianchu Zhao <evantczhao@gmail.com>
…o match up with pod.spec Signed-off-by: Tianchu Zhao <evantczhao@gmail.com>
01e2988
to
7dbc877
Compare
Done |
@simster7 we might want to back-part this one. |
Resolves: #4936
Signed-off-by: Tianchu Zhao evantczhao@gmail.com
Checklist: