| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-01 | |||
| 16:36:55 | openstack | Launchpad bug 1711267 in Cinder "Boot from volume in cinder" [Undecided,New] | |
| 16:36:58 | mriedem | has some broken links | |
| 16:51:41 | stephenfin | mriedem: Comments left | |
| 16:59:09 | mriedem | stephenfin: replied; i'm not sure i follow your confusion though | |
| 17:00:22 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console https://review.openstack.org/538003 | |
| 17:01:25 | hrw | mriedem, stephenfin: rewroted. now it adds usb host controller if there is none and adds usb keyboard if there is no keyboard. and does it on !(x86(-64), ppc64, s390x) architectures | |
| 17:02:49 | jaypipes | efried: IBM PowerKVM CI failure. I'll wait until that is resolved. | |
| 17:03:25 | efried | jaypipes: PowerKVM? Non-voting, right? I've been steadfastly ignoring that guy. | |
| 17:03:40 | jaypipes | efried: I was joking with you. | |
| 17:03:50 | efried | jaypipes: PowerVM failures are due to the fact that esberglu is rebuilding some CI systems, I think. | |
| 17:03:53 | jaypipes | efried: clearly, you do not share my brand of humour. | |
| 17:04:25 | hrw | jaypipes: https://review.openstack.org/538003 ^^ ;D | |
| 17:04:33 | efried | jaypipes: Sorry, the gate has me grumpy about CI holding up patches. | |
| 17:05:03 | efried | jaypipes: Just for that, I'm going to put you down as the approver for this blueprint I'm writing up. | |
| 17:05:22 | mmedvede | PowerKVM is voting, but it is not blocking, i.e. it can not prevent a patch with +2+w from merging | |
| 17:05:43 | esberglu | efried: Huh? I'm not doing anything with prod CI, just staging | |
| 17:05:55 | esberglu | Oh this is PowerKVM we're talking | |
| 17:06:32 | efried | esberglu: Okay, so I should be able to recheck powervm failures? | |
| 17:06:55 | efried | esberglu: Seeing OOT failures pretty frequently. | |
| 17:07:04 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: trivial: Resolve Python 3 issues https://review.openstack.org/539907 | |
| 17:07:23 | jaypipes | hrw: I thought mriedem was a no go on that getting in Queens? | |
| 17:07:34 | jaypipes | efried: :P | |
| 17:08:23 | hrw | jaypipes: how that holds you from reviewing? | |
| 17:08:46 | esberglu | efried: Yeah recheck. Only 1 issue hitting OOT, was gonna have someone from REST take a look this afternoon | |
| 17:08:55 | efried | ight | |
| 17:09:25 | hrw | jaypipes: for me it may land in Rocky cycle as well as in Queens. I just have spare time now as what I wanted to have in nova/Queens got merged. | |
| 17:09:44 | mriedem | the usbhost controller for aarch64 is a bug | |
| 17:09:53 | mriedem | working around a limitation in libvirt for non-x86 | |
| 17:09:54 | mriedem | so that's fine | |
| 17:09:54 | hrw | jaypipes: just prefer to have it reviewed when I still remember what is it all about | |
| 17:09:56 | efried | stephenfin: I think if we get up to PS4, you have to remove "trivial". | |
| 17:10:11 | stephenfin | Hahaha | |
| 17:10:12 | hrw | mriedem: is not a bug. but also not a feature | |
| 17:10:18 | stephenfin | efried: Touche :) | |
| 17:10:26 | esberglu | efried: And I would say pretty frequently is a stretch :) | |
| 17:10:44 | esberglu | OOT has failed like 6 times in the last 2 days (and at least 1 was a bad patch) | |
| 17:10:47 | efried | esberglu: Okay; first three I looked at just now. | |
| 17:10:53 | hrw | libvirt devs are very curious about changing defaults so I made patch for nova to do what needs to be done | |
| 17:11:02 | jaypipes | hrw: https://wattsupwiththat.files.wordpress.com/2015/09/not_a_bug_but_a_feature.jpg | |
| 17:11:13 | stephenfin | mriedem: So QEMU broke something and then libvirt managed to fix it? | |
| 17:11:46 | stephenfin | Meaning libvirt 3.10+ and any version of QEMU (including 2.10?) would work? | |
| 17:11:54 | hrw | https://www.redhat.com/archives/libvir-list/2018-February/msg00043.html is thread on libvirt ML if someone want | |
| 17:12:15 | stephenfin | But not libvirt < 3.10 and QEMU >= 2.10? | |
| 17:13:19 | hrw | stephenfin: nevermind which version they change situation I may still end with nova on aarch64 with older libvirt/qemu combo so https://review.openstack.org/538003 is a way | |
| 17:13:58 | stephenfin | hrw: Um, come again? | |
| 17:14:22 | stephenfin | hrw: I was referring to https://review.openstack.org/#/c/532689/, btw | |
| 17:14:28 | hrw | stephenfin: ah | |
| 17:14:48 | hrw | sorry, too late for me probably | |
| 17:15:01 | hrw | multiattach is qemu 2.10+ yes | |
| 17:15:09 | hrw | or sth | |
| 17:24:20 | cfriesen | mriedem: release note has been added as per your request for https://review.openstack.org/#/c/520187/ | |
| 17:25:02 | mriedem | cfriesen: ok but i'm not looking at that until after queens | |
| 17:25:48 | mriedem | stephenfin: you can do multiattach if (1) qemu<2.10 or (2) libvirt>=3.10 (regardless of qemu version) | |
| 17:26:00 | mriedem | libvirt 3.10 does a thing to make it work with qemu 2.10+ | |
| 17:26:04 | mriedem | for shared disks | |
| 17:26:52 | mriedem | stephenfin: https://bugzilla.redhat.com/show_bug.cgi?id=1378242 | |
| 17:26:54 | openstack | bugzilla.redhat.com bug 1378242 in libvirt "QEMU image file locking (libvirt)" [Unspecified,Verified] - Assigned to pkrempa | |
| 17:27:08 | stephenfin | mriedem: Right, figured out the source of my confusion. It was this https://review.openstack.org/#/c/532214/ | |
| 17:27:22 | mriedem | totally unrealted | |
| 17:27:27 | mriedem | ^ is because the job runs without pike uca | |
| 17:27:47 | stephenfin | Yup, I know that now. The fact that it talked about libvirt and QEMU versions but with opposite expectations threw me | |
| 17:28:35 | jaypipes | hrw: reviewed. | |
| 17:28:40 | melwitt | mriedem: wanted to bring this to your attention, I think we're not using the newer cinder api when creating instances with multi-cell https://review.openstack.org/#/c/539755/ | |
| 17:29:03 | melwitt | so another set of eyes would be helpful there | |
| 17:29:06 | stephenfin | mriedem: One last question (mirrored from the review): do we want to suggest the behavior on libvirt < 3.10? Is the behaviour of QEMU < 2.10 considered broken or something? | |
| 17:30:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Migrate "launch instance" user guide docs https://review.openstack.org/540048 | |
| 17:30:53 | hrw | jaypipes: thx. updating code and replying | |
| 17:30:57 | mriedem | stephenfin: i'm having a hard time parsing "suggest the behavior" | |
| 17:31:14 | mriedem | stephenfin: the libvirt driver will report the capability as not being supported if those version constraints aren't satisfied | |
| 17:31:40 | mriedem | stephenfin: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L599 | |
| 17:32:19 | stephenfin | I don't even know what I was trying to say there | |
| 17:32:43 | mriedem | melwitt: yes agree, i had noticed that while adding the multiattach support to the api and looking at the version checks; that version check was added in pike before dan's multi-cell in api stuff landed; i just forgot to push a patch for it | |
| 17:32:54 | mriedem | stephenfin: just tell me you love me | |
| 17:33:16 | jaypipes | hrw: cool. my comments make sense ok? | |
| 17:33:21 | hrw | yes | |
| 17:33:23 | stephenfin | A +2 is the most you're getting at this time of evening, Mr Riedemann | |
| 17:33:25 | jaypipes | coolio. | |
| 17:33:36 | stephenfin | That's done now. Sorry about the confusion | |
| 17:33:46 | jaypipes | stephenfin: he prefers Ms. Riedemann. | |
| 17:34:21 | melwitt | lyarwood: I have a feeling that https://bugs.launchpad.net/neutron/+bug/1744361 might be because we don't wait for the unplug event before we plug. my plan for today is to put up a DNM patch in a project where the job is failing with depends-on the logging and see if I can get it to fail in the same way. then I'll test a patch that waits for unplug events | |
| 17:34:22 | openstack | Launchpad bug 1744361 in neutron "test_server_security_groups failed to reboot with "Domain not found: no domain with matching uuid" because of missing vif event from linuxbridge agent" [Undecided,Confirmed] | |
| 17:34:28 | mriedem | melwitt: +2 | |
| 17:34:39 | melwitt | mriedem: ah, cool. thanks | |
| 17:34:51 | mriedem | i prefer herr riedemann btw | |
| 17:34:57 | mriedem | it is a german name | |
| 17:35:23 | hrw | print(cfg.to_xml()) in test ftw | |
| 17:35:45 | hrw | jaypipes: Fraü? | |
| 17:38:21 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Move resource provider objects into placement hierarchy https://review.openstack.org/540049 | |
| 17:40:42 | hrw | jaypipes: first version of patch was using model. Then libvirt guys told me about defaults | |
| 17:44:39 | hrw | ugh. I looked at tests and they look funny in some moments | |
| 17:47:01 | jaypipes | hrw: gotcha | |
| 17:47:24 | jaypipes | efried: reviewed. | |
| 17:47:30 | efried | jaypipes: thx | |
| 18:01:33 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Extending delete_cell --force to delete instance_mappings https://review.openstack.org/540073 | |
| 18:01:47 | efried | stephenfin: Remind me how to sphinx build a single document? | |
| 18:03:00 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console https://review.openstack.org/538003 | |
| 18:03:21 | hrw | jaypipes: your suggestions are in. Also test changed to test aarch64 with graphics | |
| 18:03:55 | hrw | probably should add new test so one will check base aarch64 no gfx + new for aarch64 with gfx | |
| 18:04:26 | jaypipes | hrw: lol, did you set the keymap to en-ie just for stephenfin? ;) | |
| 18:04:38 | hrw | jaypipes: copy/pasted from other test | |