| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-23 | |||
| 16:11:27 | kashyap | prometheanfire: Got the change URL handy? | |
| 16:11:51 | kashyap | The next min libvirt version for "Train" release is 4.0.0. | |
| 16:12:13 | kashyap | IIRC, we should stick to matching libvirt-python binding. | |
| 16:14:21 | prometheanfire | https://www.libvirt.org/news.html ? | |
| 16:14:56 | prometheanfire | kashyap: iirc libvirt-python maintains backwards compat | |
| 16:15:10 | kashyap | prometheanfire: Oh, certainly it does | |
| 16:15:37 | kashyap | But, I'm forgetting some "fine print" that a libvirt dev told on IRC some months ago | |
| 16:16:05 | kashyap | When proposing a higher libvirt-python version, I'd appreciate people noting rationale in the commit message, too. | |
| 16:16:09 | prometheanfire | LOL | |
| 16:16:10 | prometheanfire | https://github.com/libvirt/libvirt-python/compare/v4.10.0...v5.0.0 | |
| 16:16:13 | prometheanfire | one change | |
| 16:17:48 | kashyap | prometheanfire: You should compare 4.0.0, not 4.10.0. | |
| 16:17:50 | kashyap | https://github.com/libvirt/libvirt-python/compare/v4.0.0...v5.0.0 | |
| 16:17:59 | kashyap | Many many changes in there. | |
| 16:18:21 | prometheanfire | libvirt-python is already on 4.10.0 | |
| 16:18:30 | prometheanfire | for master | |
| 17:07:22 | kashyap | Sure. I said v4.4.0 because that's the version we are targetting for "Train". If there is a review change, post it there. /me needs to be AFK soon. | |
| 17:11:50 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Make 'plugin' a required argument for '_get_vif_instance' https://review.openstack.org/608279 | |
| 17:11:51 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove get_config_vhostuser https://review.openstack.org/565471 | |
| 17:18:18 | belmoreira | jaypipes: by the end of week we expect to test the "cherry-pick" of efried patch for placement requests (tssurya is working on it). After, we will do yours (we need to check how much different is your approach comparing with what we currently have and how difficult it is to have it in rocky) | |
| 17:18:39 | jaypipes | belmoreira: cool :) thanks! | |
| 17:21:42 | openstackgerrit | Jack Ding proposed openstack/nova master: Flavor extra spec and image properties validation https://review.openstack.org/620706 | |
| 17:53:35 | canori01 | hey guys, if I need to send nova notifications to multiple rabbit queues, is there more two it than just adding the notification topic? I am running both ceilometer and designate so I have "notification_topics = notifications,notifications_designate" in my config...but while the notifications queue gets the messages, notifications_designate doesn't get anything | |
| 18:49:36 | stephenfin | canori01: Maybe try again tomorrow earlier in UTC when gibi is about? | |
| 20:18:40 | openstackgerrit | Hang Yang proposed openstack/nova master: Fix port dns_name reset https://review.openstack.org/631684 | |
| 21:21:20 | edleafe | gs | |
| 22:05:45 | openstackgerrit | Jack Ding proposed openstack/nova master: Flavor extra spec and image properties validation https://review.openstack.org/620706 | |
| 23:35:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for libvirt vgpu reshape https://review.openstack.org/631559 | |
| 23:35:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: implement reshaper for vgpu https://review.openstack.org/599208 | |
| 23:35:11 | mriedem | artom: bauzas: jaypipes: consider the libvirt reshaper patch reshaped ^ | |
| 23:36:11 | mriedem | actually i removed _get_vgpu_total so need to remove some test stuff with it | |
| 23:36:15 | mriedem | not sure why bauzas didn't just remove _get_vgpu_total | |
| 23:54:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for libvirt vgpu reshape https://review.openstack.org/631559 | |
| 23:54:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: implement reshaper for vgpu https://review.openstack.org/599208 | |
| #openstack-nova - 2019-01-24 | |||
| 03:26:09 | openstackgerrit | melanie witt proposed openstack/nova master: Add configuration of maximum disk devices to attach https://review.openstack.org/616777 | |
| 03:26:10 | openstackgerrit | melanie witt proposed openstack/nova master: Raise 403 instead of 500 error from attach volume API https://review.openstack.org/632904 | |
| 05:17:32 | openstackgerrit | Yongli He proposed openstack/nova master: Adds the server group info into show server detail API. https://review.openstack.org/621474 | |
| 05:59:42 | openstackgerrit | Yongli He proposed openstack/nova master: Add server subresouce toplogy API https://review.openstack.org/621476 | |
| 07:09:55 | openstackgerrit | Asmita Singh proposed openstack/python-novaclient master: Handle unicode multi-byte characters https://review.openstack.org/632942 | |
| 09:23:41 | gibi | canori01: hi! I'm not sure either about how to send nova notifications to multiple queues | |
| 09:25:09 | gibi | canori01: the nova legacy notifications and the new versioned notifications are separated already to different queues | |
| 09:26:06 | gibi | canori01: there is an oslo messaging notificatin driver called 'routing' but it seems it is undocumented https://docs.openstack.org/oslo.messaging/ocata/notifier.html#routing | |
| 11:02:56 | openstackgerrit | Maria Luisa Arches proposed openstack/nova master: Skip checking of target_dev for vhostuser https://review.openstack.org/627540 | |
| 11:29:34 | openstackgerrit | Maria Luisa Arches proposed openstack/nova master: Skip checking of target_dev for vhostuser https://review.openstack.org/627540 | |
| 12:39:39 | openstackgerrit | Jan Gutter proposed openstack/nova master: Add support for vrouter HW datapath offloads https://review.openstack.org/572082 | |
| 14:00:51 | gibi | nova meeting just started on #openstack-meeting | |
| 14:28:49 | mriedem | alex_xu: what do you think about the suggestion in https://review.openstack.org/#/c/619953/7/nova/virt/libvirt/driver.py@4894 | |
| 14:29:08 | mriedem | change hw:unique_serial from a boolean to an enum - hw:guest_serial=unique | |
| 14:29:19 | mriedem | and that's the only value allowed | |
| 14:31:37 | mriedem | gibi: comments in your e-r query for the subunit parser failure https://review.openstack.org/#/c/632977/1 | |
| 14:32:13 | gibi | mriedem: looking | |
| 14:33:26 | gibi | mriedem: the hits are mostly nova but other projects shows up as well | |
| 14:34:10 | gibi | mriedem: it is interesting the sometimes there is no Captured Parser Error in the logs but besides that the symptoms are the same | |
| 14:34:52 | gibi | mriedem: execution aborted, and the output of the test case caused the error are truncated | |
| 14:50:09 | mriedem | gibi: ack approved | |
| 14:50:18 | gibi | mriedem: thanks | |
| 15:00:20 | alex_xu | mriedem: I like the enum than the boolean | |
| 15:02:11 | mriedem | ok i'll send something to the ML to confirm | |
| 15:02:11 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Introduce the PENDING instance state https://review.openstack.org/566473 | |
| 15:02:20 | alex_xu | cool, thanks | |
| 15:04:16 | alex_xu | mriedem: when the user specific 'false' explicitly, but in the end he get the unique value, that is confuse. Althought I'm not sure any user want to use False value, but it is just a confuse value. | |
| 15:09:10 | mriedem | alex_xu: yeah, email sent | |
| 15:09:40 | openstackgerrit | Paul-Emile Element proposed openstack/nova master: Add emulated TPM support to Nova https://review.openstack.org/631363 | |
| 15:18:03 | Kevin_Zheng | mriedem: thanks for doing this | |
| 15:18:51 | mriedem | np | |
| 15:21:56 | mriedem | Kevin_Zheng: i guess let's give it a day to see if anyone has a strong objection to that alternative proposal and if not, just make that change and we/i can amend the spec | |
| 15:22:49 | Kevin_Zheng | Sure, I can amend it in Saturday as we are working on Saturday this week | |
| 15:25:59 | BjoernT | Hello is there anyone here familiar with instance store baked by NFS and larger volume, as I see performance issues with the image cache manager ? | |
| 15:26:08 | BjoernT | https://bugs.launchpad.net/nova/+bug/1804262/ is the original bug I had opened | |
| 15:26:09 | openstack | Launchpad bug 1804262 in OpenStack Compute (nova) "ComputeManager._run_image_cache_manager_pass times out when running on NFS" [Undecided,New] | |
| 15:26:20 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Skip checking of target_dev for vhostuser https://review.openstack.org/627540 | |
| 15:29:43 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421 | |
| 15:29:43 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | |
| 15:29:44 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311 | |
| 15:29:44 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310 | |
| 15:29:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543 | |
| 15:31:13 | stephenfin | alex_xu: Regarding https://review.openstack.org/#/c/631038/3/nova/virt/libvirt/driver.py@6328 | |
| 15:31:31 | stephenfin | alex_xu: Is it not a reasonable assumption that there will be some small pages on each NUMA node? | |
| 15:31:45 | stephenfin | Far as I know, you can't boot a system with only large/huge pages | |
| 15:33:02 | sean-k-mooney | stephenfin: correct you cant | |
| 15:34:01 | sean-k-mooney | also event if there are 0 available i think you will still have the mempage listed | |
| 15:36:28 | stephenfin | sean-k-mooney++ | |
| 15:49:17 | jangutter | stephenfin: allocating everything to hugepages is the quickest way to get a system unbootable by hitting the OOM killer. | |
| 15:49:57 | sean-k-mooney | jangutter: yep the kernel cant allocate its own memory form hugepages so it gets grumpy | |
| 15:51:14 | stephenfin | sean-k-mooney: When we move hugepage tracking to placement, we should get rid of this thing https://review.openstack.org/#/c/631053/1/nova/virt/hardware.py@1059 | |
| 15:51:45 | stephenfin | (the second part of your comment, that is | |
| 15:51:46 | stephenfin | ) | |
| 15:52:06 | sean-k-mooney | stephenfin we can only do that if we track all memory as meory of a set size | |
| 15:52:40 | sean-k-mooney | so we would need to get rid of memory_mb from the compute node and always afine guest to numa nodes | |
| 15:54:14 | sean-k-mooney | i personally woudl prefer to do that but i dont think we can just remove it simply | |
| 15:54:22 | sean-k-mooney | maybe we can | |
| 15:54:25 | stephenfin | Hmm that's a good point | |
| 15:55:00 | sean-k-mooney | honestly i would prefer to have a singel code path used for dedicate/shared cpus and all mempages sizes | |
| 15:55:15 | sean-k-mooney | instead of the 2 we have now for numa guests and non numa guests | |
| 16:07:28 | bauzas | folks, FWIW, I'm on a conference today and tomorow named 'SnowCamp' where I'm an organizer | |
| 16:07:31 | bauzas | mriedem: ^ | |
| 16:12:07 | mriedem | mdbooth: fyi image cache manager perf bug https://bugs.launchpad.net/nova/+bug/1804262/ - i put some thoughts on how to optimize where they are hitting timeouts | |
| 16:12:08 | openstack | Launchpad bug 1804262 in OpenStack Compute (nova) "ComputeManager._run_image_cache_manager_pass times out when running on NFS" [Medium,Triaged] | |
| 16:13:17 | BjoernT | Thanks for the feedback, I did not know about the GMR report (good old amiga times reflect back on me, lol) | |
| 16:14:25 | mriedem | BjoernT: i think i've sorted out the issue though so the gmr isn't really necessary | |