| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-03-23 | |||
| 13:31:03 | sean-k-mooney | actully i have to join a meeting | |
| 13:31:08 | gibi | sean-k-mooney: I've just re-reviewed before lunch | |
| 13:31:15 | gibi | sean-k-mooney: it looks pretty close | |
| 13:31:39 | gibi | sean-k-mooney: I have one deeper question about the amount of ARQ delete calls nova makes | |
| 13:31:48 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/714439 | |
| 13:31:48 | gibi | the rest is mostly easy fixes | |
| 13:32:00 | sean-k-mooney | ok so i would proably be worth my time looking at it again today | |
| 13:32:57 | gibi | sean-k-mooney: yes. I also pinged dansmith as I think he can re-review now as well | |
| 13:33:12 | dansmith | yep, I'm looking at alex_xu's query on that | |
| 13:36:51 | dansmith | gibi: will start going over that in a little bit | |
| 13:37:03 | gibi | cool | |
| 13:37:32 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/714439 | |
| 13:44:47 | lyarwood | kashyap / stephenfin / melwitt ; https://review.opendev.org/#/c/696834/ - can you hit this again (rbd_swap) today if you have time. FWIW with the increase in required versions I'm not going to be able to test this upstream until I get Fedora 31 images built | |
| 13:45:02 | lyarwood | https://review.opendev.org/#/c/708416/ in this change | |
| 13:45:25 | kashyap | lyarwood: Hi, it's on TODO for today. | |
| 13:49:56 | lyarwood | kashyap: thanks | |
| 14:08:22 | openstackgerrit | Brin Zhang proposed openstack/nova master: Expose instance action event details out of the API https://review.opendev.org/694430 | |
| 14:09:59 | openstackgerrit | Merged openstack/nova master: Bump python-subunit minimum to 1.4.0 https://review.opendev.org/713611 | |
| 14:11:44 | openstackgerrit | jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714454 | |
| 14:21:52 | openstackgerrit | jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459 | |
| 14:25:32 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add instance actions v283 samples test https://review.opendev.org/706251 | |
| 14:26:19 | brinzhang | gibi: Done, thanks | |
| 14:33:42 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add instance actions v283 samples test https://review.opendev.org/706251 | |
| 14:43:35 | kashyap | lyarwood: Should the blockCopy() change then be dependent on the F31 change - to prove testability? | |
| 14:48:18 | lyarwood | kashyap: I've already proven it with F30 and 5.10/4.3 tbh | |
| 14:48:56 | kashyap | lyarwood: Ah, then your comment here confused me :) | |
| 14:50:00 | sean-k-mooney | lyarwood: whats the minium qemu you need | |
| 14:50:04 | sean-k-mooney | will 4.2 work | |
| 14:50:21 | lyarwood | kashyap: so the new version constrats you suggested to avoid some known issues is what's causing us to be unable to test this prior to F31 | |
| 14:50:46 | kashyap | lyarwood: Hmm; /me taps on the table and thinks ... | |
| 14:50:49 | lyarwood | sean-k-mooney: Libvirt 5.10 and QEMU 4.{3,2} I forget which | |
| 14:51:46 | kashyap | Seems like the diskimage-builder patch is at least being reviewed by the Right Person(tm). | |
| 14:52:08 | sean-k-mooney | lyarwood: i think ubuntu 20.04 will have libvirt 6.0 and qemu 4.2 | |
| 14:52:33 | lyarwood | right | |
| 14:53:00 | lyarwood | sean-k-mooney: so support was introduced a while ago, kashyap just suggested we use a later version to include some bugfixes for blockCopy. | |
| 14:53:29 | sean-k-mooney | sure but this should not be a blocker right | |
| 14:53:36 | lyarwood | and using that later version is now causing issues testing this somewhere in the gate as the virt-preview repo has only built these packages for F31 | |
| 14:53:39 | sean-k-mooney | we can test with the older versions? | |
| 14:53:57 | lyarwood | we can't if the constraints in code limit us to the newer versions | |
| 14:54:09 | lyarwood | otherwise we switch to the older blockRebase method | |
| 14:54:11 | sean-k-mooney | well the virt preview repo and fedora is not stble anyway | |
| 14:54:31 | kashyap | Yeah, FWIW, it's also the libvirt upstream suggested based on our use-case | |
| 14:54:48 | kashyap | sean-k-mooney: The reason for going with the current 6.1 and QEMU 4.2 (at least Ubuntu has this): | |
| 14:55:21 | sean-k-mooney | kashyap: i was tying to config the version did you find them | |
| 14:55:42 | hrw | Debian is at 5.0 and 3.1 | |
| 14:56:03 | sean-k-mooney | hrw: ya we are not talking about min versions here | |
| 14:56:04 | kashyap | sean-k-mooney: See my second comment on line 286: https://review.opendev.org/#/c/696834/6/nova/virt/libvirt/driver.py@286 | |
| 14:56:26 | kashyap | sean-k-mooney: The recommended version constants we're now using fix a disk image corruption bug. | |
| 14:56:49 | sean-k-mooney | right so we are defaulting to 5.10 for libvirt | |
| 14:57:02 | sean-k-mooney | and what version for qemu | |
| 14:57:16 | lyarwood | sean-k-mooney: we were | |
| 14:57:35 | sean-k-mooney | oh you bumped it to 6.1 | |
| 14:57:40 | lyarwood | sean-k-mooney: now it's 6.1.0 for Libvirt and QEMU 4.3.0 | |
| 14:57:47 | sean-k-mooney | so i dont think 6.1 will be in 20.04 | |
| 14:57:58 | sean-k-mooney | or 4.3 i think they are using 4.2 | |
| 14:58:11 | sean-k-mooney | so im not sure this will be a good base | |
| 14:58:15 | kashyap | lyarwood: I think you got the QEMU version wrong; now it is 4.2.0 | |
| 14:58:29 | sean-k-mooney | ya i think it should be 4.2 as well | |
| 14:59:52 | lyarwood | ah! | |
| 15:00:08 | sean-k-mooney | can we use 6.0 for libvirt | |
| 15:00:10 | kashyap | :) | |
| 15:00:26 | sean-k-mooney | i also dont think 6.1 will by in ubuntu 20.04 | |
| 15:00:33 | kashyap | sean-k-mooney: Let me check with the libvirt dev... | |
| 15:00:37 | sean-k-mooney | it might be in the cloud archive | |
| 15:01:42 | hrw | ubuntu focal is on 6.0 so it will stay at 6.0 unless uca | |
| 15:02:31 | sean-k-mooney | yep | |
| 15:03:01 | efried_gone | bauzas: +2 | |
| 15:03:25 | bauzas | efried_gone: with love | |
| 15:03:39 | bauzas | go save the Openshift world | |
| 15:12:59 | kashyap | sean-k-mooney: lyarwood: Good news — we can lower the libvirt version from 6.1.0 to 6.0.0, thanks to lyarwood's "provide backing file explicitly" fix: https://opendev.org/openstack/nova/commit/0cfe9c81e3fe4d | |
| 15:12:59 | kashyap | sean-k-mooney: lyarwood: Good news — we can lower the libvirt version from 6.1.0 to 6.0.0, thanks to lyarwood's "provide backing file explicitly" fix: https://opendev.org/openstack/nova/commit/0cfe9c81e3fe4d | |
| 15:13:07 | kashyap | Added a comment in the change to that effect. | |
| 15:13:31 | lyarwood | kashyap: ack thanks | |
| 15:13:34 | sean-k-mooney | cool so we can use libvirt 6.0.0 and qemu 4.2.0 as the minium safely | |
| 15:14:20 | sean-k-mooney | the other way to test this would be via rdo and a centos job | |
| 15:14:26 | sean-k-mooney | centos 8 | |
| 15:14:43 | lyarwood | do we have an AV repo in CentOS 8? | |
| 15:15:01 | sean-k-mooney | we should, im not sure but we definetly should | |
| 15:15:22 | sean-k-mooney | well in centos 8 or in rdo | |
| 15:15:39 | sean-k-mooney | we should not be testing with the defalut qemu and libvirt on centos | |
| 15:15:56 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Use virDomainBlockCopy to swap volumes when using -blockdev https://review.opendev.org/696834 | |
| 15:49:54 | hrw | 97 tempest failures on aarch64 show that it is too early for that CI job | |
| 15:55:51 | artom | hrw, well, it's a start :) It'll probably remain non-voting and/or experimental for a bit, but those 97 failures can be chipped away at one by one | |
| 15:56:24 | sean-k-mooney | hrw: you could add it to the periodic pipe line | |
| 15:57:21 | hrw | ;D | |
| 15:57:24 | sean-k-mooney | hrw: but ya as artom siad the failures can be adressed one by one | |
| 15:57:42 | sean-k-mooney | its non voting anyway so it wont block the build | |
| 15:57:59 | sean-k-mooney | and its in a seperate pipleline alreday so the check pipepline wont wait for it | |
| 15:58:04 | sean-k-mooney | so it wont slow down the gate | |
| 15:58:40 | sean-k-mooney | it proably makes sense to not merge it untill its closer to working but you can always recheck the patch | |
| 15:58:53 | sean-k-mooney | if you do that however i would suggest disableing the other jobs | |
| 15:59:07 | sean-k-mooney | so as to not was gate resouces | |
| 16:01:47 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/714439 | |
| 16:02:04 | hrw | ok, will reedit it then back | |
| 16:03:26 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/714439 | |
| 16:04:25 | hrw | ok, my redeployment finished, can now teach tempest to use it | |
| 16:34:28 | gibi | /away | |
| 16:46:23 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: update quota apis with keystone limits and usage https://review.opendev.org/713499 | |
| 16:55:27 | hrw | Took 15.46 seconds to build instance. | |
| 16:55:30 | hrw | now better | |