-
Notifications
You must be signed in to change notification settings - Fork 835
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
fix master e2e tests #1163
fix master e2e tests #1163
Conversation
/test integration |
ok now I'm seeing different problems in the operator logs during the e2e tests:
This seems to be a bug that I've introduced. |
/test integration |
There was a problem with impl entries such as UNKNOWN_IMPLEMENTATION and RANDOM_ABTEST getting treated as prepack. (Need to follow-up on how the operator image is set for local e2e tests. I'm seeing clear discrepancies between image in e2e tests and the code in my workspace. I actually had to push the image before the e2e tests picked up the changes.) Think that is gone now as https://github.com/SeldonIO/seldon-core/blob/gh-pages/jenkins-x/logs/SeldonIO/seldon-core/PR-1163/6.log#L3511 only had one failure and that looks like the intermittent ambassador one. |
/test integration |
/ok-to-test |
@ryandawsonuk: you cannot LGTM your own PR. 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. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ryandawsonuk The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@ryandawsonuk: The following test failed, say
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. I understand the commands that are listed here. |
Merging now. The only 'failure' is that the docker process in the job didn't stop so a test infra problem, not a code issue. |
This is to fix #1164
First need to give operator read access on configmaps as found in #1162