| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-23 | |||
| 14:28:14 | kashyap | It is so cathartic to "remove" code... | |
| 14:28:32 | kashyap | Reminds me of the quote from UNIX hero, Douglas Mcilroy: https://en.wikipedia.org/wiki/Douglas_McIlroy#Views_on_computing | |
| 14:31:10 | cdent | kashyap: quite: https://review.openstack.org/#/c/618215/ | |
| 14:31:38 | kashyap | Also as Elements of Style says: "Omit needless words" | |
| 14:31:48 | kashyap | cdent: Whoa ... /me bows | |
| 14:31:56 | kashyap | It is nearing merging | |
| 14:32:17 | cdent | that's the smaller of two. the one to remove the placement tests (already merged) was more lines (but much of it yaml) | |
| 14:33:11 | kashyap | That `diff stat` ... nice work. I guess the new repo is shaping up nice and well | |
| 14:34:11 | cdent | yeah, external placement is what runs in the intergrated gate now | |
| 14:35:28 | kashyap | Naice | |
| 14:50:50 | openstackgerrit | Eric Fried proposed openstack/nova master: ksa auth conf and client for cyborg access https://review.openstack.org/631242 | |
| 15:09:45 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: [WIP] libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Stein" https://review.openstack.org/632507 | |
| 15:46:27 | efried | cdent, melwitt: I probably won't make the nova IRC meeting tomorrow; will be on the road again. Not much worth reporting from the n-sch meeting. | |
| 15:47:03 | cdent | ✔ | |
| 15:47:44 | sean-k-mooney | cdent: do you keep that tick copy pasted or is there a keyboard sort cut for it | |
| 15:48:29 | cdent | i've got my keyboard setup to allow inputting unicode codepoints and have that one memorize opt/alt+2714 | |
| 15:59:00 | melwitt | efried: ack | |
| 16:02:30 | prometheanfire | new libvirt-python (5.0.0) has been proposed to upperconstaints, just an fyi | |
| 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++ | |