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

Validate that service accounts have the right shape #4647

Closed
mattmoor opened this issue Jul 8, 2019 · 5 comments · Fixed by #4733 or #4919
Closed

Validate that service accounts have the right shape #4647

mattmoor opened this issue Jul 8, 2019 · 5 comments · Fixed by #4733 or #4919
Assignees
Labels
area/API API objects and controllers kind/bug Categorizes issue or PR as related to a bug.

Comments

@mattmoor
Copy link
Member

mattmoor commented Jul 8, 2019

In what area(s)?

/area API

What version of Knative?

HEAD

Expected Behavior

Knative should reject a serviceAccountName: foo@bar.baz.

Actual Behavior

I believe we will just let this through to K8s and likely fail creating the deployment (haven't tried, just noticed this in code).

@mattmoor mattmoor added the kind/bug Categorizes issue or PR as related to a bug. label Jul 8, 2019
@mattmoor mattmoor added this to the Ice Box milestone Jul 8, 2019
@knative-prow-robot knative-prow-robot added area/API API objects and controllers kind/good-first-issue labels Jul 8, 2019
@shashwathi
Copy link
Contributor

/assign @shashwathi

@steren
Copy link
Contributor

steren commented Jul 19, 2019

Were conformance tests written for this validation?

@shashwathi
Copy link
Contributor

shashwathi commented Jul 19, 2019

@steren : No. Valid service account is not part of runtime contract(I think) so does it need a conformance test?

@steren
Copy link
Contributor

steren commented Jul 19, 2019

It's not runtime contract related, but API spec related, I suppose we also test the API?

@shashwathi
Copy link
Contributor

Yes you are right. It is spec related so I will add conformance test for this check 👍
thank you for explaining your thought behind the ask. I appreciate it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/API API objects and controllers kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
5 participants