-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Comments
/assign @shashwathi |
Were conformance tests written for this validation? |
@steren : No. Valid service account is not part of runtime contract(I think) so does it need a conformance test? |
It's not runtime contract related, but API spec related, I suppose we also test the API? |
Yes you are right. It is spec related so I will add conformance test for this check 👍 |
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).
The text was updated successfully, but these errors were encountered: