-
Notifications
You must be signed in to change notification settings - Fork 376
Conversation
@grahamwhaley Can i get the RequestForComments Label? I can't set labels in this repo. |
@jschintag - sure - you could also put [RFC] in the subject title of the PR - the CI should also pick that up. |
/test |
Codecov Report
@@ Coverage Diff @@
## master #1965 +/- ##
=========================================
Coverage ? 51.61%
=========================================
Files ? 107
Lines ? 14540
Branches ? 0
=========================================
Hits ? 7505
Misses ? 6135
Partials ? 900 |
@devimc Thank you for your review |
5396568
to
2bca4ac
Compare
Here is what i changed:
I also noticed that the qemu-tests for other archs weren`t included in the changes. Fixed it. /test |
CI is failing, @jschintag any thoughts?
|
@devimc I cannot replicate the issue on s390x, i need to further look into it. |
2bca4ac
to
d9fd1d0
Compare
@devimc As far as i can see, the agent receives a grpc Request to add a device with an empty /test |
/test-power |
@grahamwhaley I can't replicate the issue on my local x86 machine.
|
/test |
d9fd1d0
to
bbdcbfb
Compare
/test I rebased the PR to master to solve the Merge conflicts |
CI still failing 😟
|
bbdcbfb
to
cfe8475
Compare
/test-ubuntu |
cfe8475
to
590e9e8
Compare
/test |
/test-ubuntu Let's check if this fixes the netmon error |
4d55774
to
7682a62
Compare
/test |
/test some of the failures seem to be from the CI |
/test-ubuntu |
2d7de41
to
827e41f
Compare
/test |
@jschintag initrd CI is happy, why did you remove that line? |
@devimc I actually don't know why that line was removed. Alice wrote that part, i'll ask her on Monday. I'll also look into the opensuse-15 failure. |
@jschintag suse has been failing, it's not stable right now |
@devimc so, do you think we can merge this PR? |
My mistake. I did something wrong during the rebase probably sorry. Thx @jschintag |
The abstraction of the bridge type to add additional types. Fixes: kata-containers#1153 Signed-off-by: Alice Frosi <afrosi@de.ibm.com> Co-authored-by: Jan Schintag <jan.schintag@de.ibm.com>
In this way it is possible to set bridge variable for each arch when instantiating the hypervisor. Fixes: kata-containers#1153 Signed-off-by: Alice Frosi <afrosi@de.ibm.com> Co-authored-by: Jan Schintag <jan.schintag@de.ibm.com>
On IBM Z, CCW bus type can be additionally used. Fixes: kata-containers#1153 Signed-off-by: Alice Frosi <afrosi@de.ibm.com> Reviewed-by: Jan Schintag <jan.schintag@de.ibm.com>
Create generic function to be reused to the reimplemented methods by various architectures Fixes: kata-containers#1153 Signed-off-by: Alice Frosi <afrosi@de.ibm.com> Reviewed-by: Jan Schintag <jan.schintag@de.ibm.com>
The devno number is use to identify the device on IBM Z arch. Fixes: kata-containers#1153 Signed-off-by: Alice Frosi <afrosi@de.ibm.com> Reviewed-by: Jan Schintag <jan.schintag@de.ibm.com>
Add virtio-blk-ccw support Fixes: kata-containers#1153 Signed-off-by: Alice Frosi <afrosi@de.ibm.com> Reviewed-by: Jan Schintag <jan.schintag@de.ibm.com>
827e41f
to
b075b5c
Compare
/test-ubuntu |
I just rebased the PR to the current master. |
adding dnm label, I'd like to review this again |
@devimc please let us know if we need to split this PR is smallers PR. Without these changes the kata runtime is broken on s390x. |
/test |
@jodh-intel What are your thoughts on this. Is this PR fine as it is, or do we need to split it into smaller PRs? |
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.
Ok - silence is compliance... merging. |
The changes of this PR are
Fixes: #1153
Signed-off-by: Alice Frosi afrosi@de.ibm.com
Reviewed-by: Jan Schintag jan.schintag@de.ibm.com