| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-03-23 | |||
| 13:23:49 | sean-k-mooney | ah i know that seamed familar | |
| 13:24:27 | sean-k-mooney | they use primary and secondary instead of subnode terminology | |
| 13:24:37 | sean-k-mooney | that we use for devstack | |
| 13:24:56 | hrw | sean-k-mooney: after playing with devstack in past there is no stick long enough to touch it (forgive poor idiom use) | |
| 13:25:39 | sean-k-mooney | hrw: devstack and kolla-ansible are my two favorite installers | |
| 13:25:45 | hrw | ;) | |
| 13:25:46 | sean-k-mooney | depending on what im doing | |
| 13:25:50 | hrw | yoctozepto: ^^ idea? | |
| 13:26:13 | lyarwood | gibi: sorry was out at lunch, no I hadn't seen any issues since my changes landed :/ | |
| 13:26:17 | hrw | sean-k-mooney: I only use k-a for deployment. at least I know that it works and gives me sane setup | |
| 13:26:40 | hrw | sean-k-mooney: when I used devstack the way of dealing with issues was 'restart and hope' | |
| 13:26:50 | sean-k-mooney | hrw: ya i use k-a for any long lived setup and use devstack for everything else | |
| 13:27:40 | sean-k-mooney | devstack is actully pretty reliable you just need to learn how to use it | |
| 13:27:51 | sean-k-mooney | and how to fix common issues | |
| 13:28:18 | sean-k-mooney | normally when it breaks its because of a depency did a new release | |
| 13:29:07 | sean-k-mooney | once you know what to look for fixing it get much simpler | |
| 13:29:13 | hrw | possible | |
| 13:29:13 | gibi | lyarwood: I hope this is just a one time issue. I've rechecked the patch see if it goes now | |
| 13:29:42 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/714439 | |
| 13:29:48 | hrw | let's see | |
| 13:30:50 | sean-k-mooney | gibi: whats the current state fo the cyborg series | |
| 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 | gibi | the rest is mostly easy fixes | |
| 13:31:48 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/714439 | |
| 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 | |