-
Notifications
You must be signed in to change notification settings - Fork 593
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
knative-eventing-injection
can't be used with private registry
#1862
Comments
To be clear, I think this is an issue with Eventing itself, not the test infrastructure since this situation will happen during normal usage not just testing. |
Good catch. This does seem to be a fundamental issue with the approach, and should at least be documented. |
/assign @duglin |
/milestone v0.10.0 |
/assign |
/unassign |
@lionelvillard: GitHub didn't allow me to assign the following users: beemarie. Note that only knative members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@lionelvillard It looks like if I comment, I should be able to be assigned? |
/assign @beemarie |
/milestone v0.12.0 |
@lionelvillard: You must be a member of the knative/knative-milestone-maintainers GitHub team to set the milestone. If you believe you should be able to issue the /milestone command, please contact your and have them propose you as an additional delegate for this responsibility. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Describe the bug
While trying to get the e2e tests to work using a private registry, I found that the tests that add the
knative-eventing-injection
label to a namespace will fail because the pods that get automatically created will fail to be able to pull the needed images. This is because those pods are associated with theeventing-broker-filter
ServiceAccount which was also automatically created and there doesn't appear to be a way to inject a pull secret into that service account.To Reproduce
Run the e2e tests with all images in a private registry.
Knative release version
v0.8.0
/cc @n3wscott
The text was updated successfully, but these errors were encountered: