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

knative-eventing-injection can't be used with private registry #1862

Closed
duglin opened this issue Sep 11, 2019 · 11 comments · Fixed by #2245
Closed

knative-eventing-injection can't be used with private registry #1862

duglin opened this issue Sep 11, 2019 · 11 comments · Fixed by #2245
Assignees
Labels
kind/bug Categorizes issue or PR as related to a bug. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release.
Milestone

Comments

@duglin
Copy link

duglin commented Sep 11, 2019

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 the eventing-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

@duglin duglin added the kind/bug Categorizes issue or PR as related to a bug. label Sep 11, 2019
@duglin
Copy link
Author

duglin commented Sep 11, 2019

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.

@grantr
Copy link
Contributor

grantr commented Sep 11, 2019

Good catch. This does seem to be a fundamental issue with the approach, and should at least be documented.

@akashrv
Copy link
Contributor

akashrv commented Sep 16, 2019

/assign @duglin

@paulrossman paulrossman added the priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. label Sep 16, 2019
@akashrv
Copy link
Contributor

akashrv commented Sep 16, 2019

/milestone v0.10.0

@knative-prow-robot knative-prow-robot added this to the v0.10.0 milestone Sep 16, 2019
@paulrossman paulrossman modified the milestones: v0.10.0, v0.11.0-M1 Oct 30, 2019
@lionelvillard
Copy link
Member

/assign

@lionelvillard
Copy link
Member

/unassign
/assign @beemarie

@knative-prow-robot
Copy link
Contributor

@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.
For more information please see the contributor guide

In response to this:

/unassign
/assign @beemarie

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.

@bvennam
Copy link
Contributor

bvennam commented Nov 8, 2019

@lionelvillard It looks like if I comment, I should be able to be assigned?

@lionelvillard
Copy link
Member

/assign @beemarie

@lionelvillard
Copy link
Member

/milestone v0.12.0

@knative-prow-robot
Copy link
Contributor

@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:

/milestone v0.12.0

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants