| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-12-10 | |||
| 09:28:49 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Reject migration requests when source is down https://review.openstack.org/623489 | |
| 09:44:04 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | |
| 09:44:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Calculate RequestGroup resource provider mapping https://review.openstack.org/616239 | |
| 09:44:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Extend RequestGroup object for mapping https://review.openstack.org/619527 | |
| 09:44:06 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Pass resource provider mapping to neutronv2 api https://review.openstack.org/616240 | |
| 09:44:06 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fill the RequestGroup mapping during schedule https://review.openstack.org/619528 | |
| 09:44:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459 | |
| 09:44:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Recalculate request group - RP mapping during re-schedule https://review.openstack.org/619529 | |
| 09:44:08 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject interface attach with QoS aware port https://review.openstack.org/570078 | |
| 09:44:08 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | |
| 09:44:09 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421 | |
| 09:44:09 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject networks with QoS policy https://review.openstack.org/570079 | |
| 09:44:10 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ensure that allocated PF matches the used PF https://review.openstack.org/623543 | |
| 10:05:09 | openstackgerrit | Merged openstack/nova master: Add ratio online data migration when load compute node https://review.openstack.org/613499 | |
| 10:11:34 | openstackgerrit | Merged openstack/nova master: Add compute_node ratio online data migration script https://review.openstack.org/609995 | |
| 10:11:40 | openstackgerrit | Merged openstack/nova master: Note the aggregate allocation ratio restriction in scheduler docs https://review.openstack.org/620713 | |
| 10:17:52 | izza_ | hi | |
| 10:18:12 | izza_ | anyone here already deployed volume with attached image (windows) in openstack tripleo | |
| 10:18:16 | izza_ | need help pls | |
| 10:30:41 | openstackgerrit | Merged openstack/nova stable/rocky: Ignore MoxStubout deprecation warnings https://review.openstack.org/623545 | |
| 11:24:31 | openstackgerrit | Chris Dent proposed openstack/nova master: Add python 3.7 unit and functional tox jobs https://review.openstack.org/624055 | |
| 11:29:49 | openstackgerrit | Chris Dent proposed openstack/nova master: Use external placement in functional tests https://review.openstack.org/617941 | |
| 11:29:50 | openstackgerrit | Chris Dent proposed openstack/nova master: Delete the placement code https://review.openstack.org/618215 | |
| 11:46:55 | cdent | thanks for continuing to +2 the placement functional stuff gibi, we'll get it merged one of these days | |
| 11:48:23 | gibi | cdent: this morning when I did a git pull I got happy as the pull brought in the deletion of placement related files. But then I had to realize that it was just api sample removal not your patch (yet) | |
| 11:48:34 | cdent | :) | |
| 12:23:21 | openstackgerrit | Chris Dent proposed openstack/nova master: Add python 3.7 unit and functional tox jobs https://review.openstack.org/624055 | |
| 12:24:10 | cdent | dansmith looks like the multi_cell query job is not happy with python 3.7 ^ | |
| 12:24:16 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Add live migration timeout action https://review.openstack.org/619143 | |
| 12:24:34 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142 | |
| 13:20:01 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor PortResourceRequestBasedSchedulingTestBase https://review.openstack.org/624080 | |
| 13:20:01 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ensure that allocated PF matches the used PF https://review.openstack.org/623543 | |
| 14:02:04 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add workaround to cleanup instance dir during evac with rbd https://review.openstack.org/618478 | |
| 14:17:03 | lyarwood | melwitt: ^ when you're around would you mind taking another look at that? I honestly can't see a way for cleanup to be called erroneously on the source host during a evacuation failure. | |
| 14:42:56 | sean-k-mooney | bauzas: melwitt could one of ye apporve https://review.openstack.org/#/c/618239/ as release liaison/ptl when ye get a chance. | |
| 14:43:50 | openstack | Launchpad bug 1734320 in os-vif "Eavesdropping private traffic" [High,In progress] - Assigned to sean mooney (sean-k-mooney) | |
| 14:43:50 | sean-k-mooney | the main change is fixing the os-vif side of https://bugs.launchpad.net/neutron/+bug/1734320 + https://bugs.launchpad.net/os-vif/+bug/1801072 | |
| 14:43:51 | openstack | Launchpad bug 1801072 in os-vif "vif_plug_ovs.linux_net.delete_net_dev is called outside the privsep context" [Critical,Fix released] - Assigned to sean mooney (sean-k-mooney) | |
| 14:55:25 | KeithMnemonic | Is it possible to please get some reviews on this patch? https://review.openstack.org/#/c/573066/ | |
| 14:58:16 | sean-k-mooney | KeithMnemonic: you realise that libvirt/qemu has a limit on how many volumes you can attach to an instance too | |
| 14:58:33 | KeithMnemonic | yes there is a pci limit i thought | |
| 14:58:40 | KeithMnemonic | is that 26? | |
| 14:58:49 | sean-k-mooney | i think its 20 not 26 | |
| 14:59:49 | openstack | Launchpad bug 1770527 in OpenStack Compute (nova) "openstack server add volume fails over 26vols" [Wishlist,In progress] - Assigned to Tsuyoshi Nagata (yukari-papa) | |
| 14:59:49 | KeithMnemonic | it is more than 20 i am pretty sure, problem is this bug is still open https://bugs.launchpad.net/nova/+bug/1770527 so we have a customer asking for it | |
| 14:59:52 | sean-k-mooney | the 26 limit on device names i sobviousl just because tehre are 26 ascii/english letter | |
| 15:00:27 | sean-k-mooney | KeithMnemonic: it may depend on the qemu/libvirt version | |
| 15:00:31 | KeithMnemonic | https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/5/html/virtualization/sect-virtualization-virtualization_limitations-kvm_limitations | |
| 15:01:05 | KeithMnemonic | Hence, of the 32 available PCI devices for a guest, 4 are not removable. This means there are 28 PCI slots available for additional devices per guest. Every para-virtualized network or block device uses one slot. Each guest can use up to 28 additional devices made up of any combination of para-virtualized network, para-virtualized disk devices, or other PCI devices using VT-d. | |
| 15:03:31 | sean-k-mooney | does the 4 include the vnc/spice resouces | |
| 15:04:50 | sean-k-mooney | you can use virtio-scsi too which may allow mulitple volumes per pci device | |
| 15:18:59 | mriedem | lyarwood: surprise surprise multiattach swap volume across 2 hosts is broken | |
| 15:21:28 | lyarwood | mriedem: wasn't mdbooth looking at blocking all attempts to swap/migrate multiattach volumes? | |
| 15:21:49 | lyarwood | he isn't around today btw | |
| 15:22:04 | mriedem | he was for at least cases of the multiattach volume having >1 read/write attachment | |
| 15:22:17 | mriedem | https://review.openstack.org/#/c/572790/ | |
| 15:22:23 | mriedem | but this isn't that issue, | |
| 15:22:49 | mriedem | the swap volume code in compute updates the bdm record for the "old" volume and changes the volume_id to the "new" volume | |
| 15:22:52 | openstackgerrit | Merged openstack/nova master: Add docs for (initial) allocation ratio configuration https://review.openstack.org/622588 | |
| 15:22:56 | mriedem | essentially orphaning the old volume, so we don't cleanup properly | |
| 15:23:48 | mriedem | this https://github.com/openstack/nova/blob/ae3064b7a820ea02f7fc8a1aa4a41f35a06534f1/nova/compute/manager.py#L5798-L5806 | |
| 15:24:10 | mriedem | the bdm that gets updated is the old bdm (source volume), but save_volume_id is the new volume | |
| 15:24:41 | mriedem | we're also clearly wrongly updating the old bdm.connection_info with the new vol connection_info, | |
| 15:24:57 | mriedem | for new volume attach flows that doesn't matter as we don't use the bdm.connection_info, but it's still wrong | |
| 15:25:36 | mriedem | looks like that code assumes we did a cinder-induced retype/migration, "# correct volume_id returned by Cinder." | |
| 15:25:43 | mriedem | but that's not the case here | |
| 15:27:13 | mriedem | if only we had that volume_id uniqute constraint in the bdms table :) | |
| 15:27:20 | mriedem | the bdm update would blow up hard | |
| 15:28:41 | lyarwood | if only, sooooooooooo Matt also had a plan to block the direct use of this API FWIW at some point recently | |
| 15:28:58 | lyarwood | so only allow it to be used via volume migration API | |
| 15:29:20 | lyarwood | but this still sounds valid, do we have a launchpad bug for this? | |
| 15:29:47 | mriedem | not yet, i'm debugging the failures in the tempest-slow job here https://review.openstack.org/#/c/606981/ - i'll be dumping notes in a launchpad bug | |
| 15:30:46 | mriedem | i'm not sure how the compute api would block swap volume unless we checked the volume status to see if it's either 'retyping' or 'migrating' and fail otherwise | |
| 15:31:11 | mriedem | and if we'd restrict that based on a policy check | |
| 15:31:21 | mriedem | because swap volume is admin-only today already | |
| 15:32:34 | lyarwood | Yeah not sure what he had in mind tbh | |
| 15:43:55 | openstack | Launchpad bug 1807723 in OpenStack Compute (nova) "swap multiattach volume intermittently fails when servers are on different hosts" [Medium,Confirmed] | |
| 15:43:55 | mriedem | https://bugs.launchpad.net/nova/+bug/1807723 | |
| 15:46:21 | kashyap | lyarwood: What about Bregret? | |
| 15:46:46 | kashyap | lyarwood: I mean, what are you "dealing" with? Aren't you just supposed to just "suffer the consequences", or you have the power to "do something"? :D | |
| 15:47:33 | kashyap | lyarwood: Oh ... disregard me; just "caught up" with the Tories. | |
| 15:47:58 | stephenfin | jangutter: Reviewed https://review.openstack.org/#/c/607610/. Looks good to me, for the most part, the few updates suggested by others aside | |
| 15:48:10 | stephenfin | jangutter: If you rework today, let me know and I'll swing by again | |
| 15:48:53 | jangutter | stephenfin: thanks, respinning! | |
| 16:00:25 | jangutter | sean-k-mooney: any objection to choosing "Option 1" in the spec now? | |
| 16:00:54 | sean-k-mooney | jangutter: i have not looked in a while but didnt i say that in a previous version | |
| 16:01:55 | sean-k-mooney | ah its still at the same version so sure | |
| 16:02:04 | jangutter | sean-k-mooney: yeah, doing a respin now, just wanted to make doubly sure. I mean, I sold a _lot_ of tickets to "Sean vs Jay's punch-the-ginger contest". | |
| 16:03:49 | sean-k-mooney | jangutter: every know you dont punch ginger people in case its contagious :P | |
| 16:04:34 | jangutter | sean-k-mooney: that's why boxing gloves were developed to be so thick. | |
| 16:05:17 | sean-k-mooney | if you want to leave it till later to decide we can but you already have patches for option 1 | |
| 16:05:39 | sean-k-mooney | my main concern was serialiastion size | |
| 16:06:55 | sean-k-mooney | we can adress that at a later date in other ways | |
| 16:11:28 | openstackgerrit | Jack Ding proposed openstack/nova-specs master: Select cpu model from a list of cpu models https://review.openstack.org/620959 | |
| 16:14:55 | openstackgerrit | Ben Nemec proposed openstack/nova master: Migrate upgrade checks to oslo.upgradecheck https://review.openstack.org/603499 | |
| 16:40:13 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to implement os-vif generic datapath offloads https://review.openstack.org/607610 | |
| 16:58:17 | melwitt | o/ | |
| 16:59:03 | sean-k-mooney | melwitt: o/ | |
| 17:02:10 | sean-k-mooney | johnthetubaguy: o/ care to take another look at https://review.openstack.org/#/c/591607/11 | |
| 17:20:14 | melwitt | mriedem: I was thinking we should cancel the dec 20 nova meeting bc efried_cya_jan is out and I think dansmith is out that day too. and I was considering taking the day off as well | |