-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Added Openshift 3.11 and k8s 1.12.2 with multus providers. #1726
Added Openshift 3.11 and k8s 1.12.2 with multus providers. #1726
Conversation
@SchSeba Can you please add relevant |
0ca54ec
to
5fb3f85
Compare
/hold |
5fb3f85
to
97ad8d9
Compare
/hold cancel |
97ad8d9
to
94a17af
Compare
ci test please |
@SchSeba the openshift multus run complains about a non-existent shasum. The k8s run looks pretty good. |
cluster/os-3.11.0-multus/provider.sh
Outdated
|
||
source cluster/os-3.11.0/provider.sh | ||
|
||
image="os-3.11.0-multus@sha256:b7354263a7828cb13ae103f84c9d7c9d8def8f3edf6014a5e8e330724588afae" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can it be that this should be os-3.11.0-multus@sha256:dfeaa7c1f7c264f953e44c93b50bb76c9bb58b0df8bb94dcaa108db254f87eb3
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know but this is weird can you please try to run docker pull locally and tell me if it works for you? because it works for me.
docker pull kubevirtci/os-3.11.0-multus@sha256:b7354263a7828cb13ae103f84c9d7c9d8def8f3edf6014a5e8e330724588afae
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes works for me. Maybe it was a transient issue? Which shasum is now the correct one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the last one I pushed yesterday.
b7354263a7828cb13ae103f84c9d7c9d8def8f3edf6014a5e8e330724588afae
I will try to run the tests again
ci test please |
@gbenhaim Can I get some help with the openshift 3.11 multus error (fail to pull from docker hub) |
ci test please |
471cbe9
to
919fa62
Compare
@rmohr can you please take a look |
919fa62
to
0cb968f
Compare
0cb968f
to
fc0ed11
Compare
fc0ed11
to
6027689
Compare
ci test please |
ci test please |
@cynepco3hahue I think this is ready to go. |
@SchSeba I see in http://jenkins.ovirt.org/job/kubevirt_kubevirt_standard-check-pr/2859//artifact/check-patch.k8s-multus-1.12.2-release.el7.x86_64/mock_logs/script/stdout_stderr.log
Is that the correct behaviour? |
Looks good to me, if the behaviour mentioned above is correct. |
https://jenkins.ovirt.org/job/kubevirt_kubevirt_standard-check-pr/2859//artifact/check-patch.k8s-multus-1.12.2-release.el7.x86_64/mock_logs/script/stdout_stderr.log failed to finish the run, will re-run CI. |
ci test please |
@cynepco3hahue that's the pci address test. It fails randomly on lanes. We should fix that test asap. |
@rmohr strange first time I saw only part of the log |
What this PR does / why we need it:
Added Openshift 3.11 and k8s 1.12.2 with multus providers.
Release note:
This change is![Reviewable](https://camo.githubusercontent.com/23b05f5fb48215c989e92cc44cf6512512d083132bd3daf689867c8d9d386888/68747470733a2f2f72657669657761626c652e696f2f7265766965775f627574746f6e2e737667)