| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-23 | |||
| 12:50:01 | mdbooth | kashyap: But you need to change it... | |
| 12:50:32 | kashyap | Yes, yes. I was isolating the problem | |
| 12:50:47 | kashyap | mdbooth: For example, see this small PDB session w/ & without my patch for guest.devices: http://paste.openstack.org/show/743172/ | |
| 12:51:08 | kashyap | This is a different test that also fails due to: | |
| 12:51:10 | kashyap | self.assertEqual(2, len(guest.devices)) | |
| 12:51:19 | kashyap | [...] | |
| 12:51:24 | kashyap | testtools.matchers._impl.MismatchError: 2 != 1 | |
| 12:52:06 | kashyap | The fakelibvirt.py is "misbehaving" due to the version change. | |
| 12:59:59 | mdbooth | kashyap: There are way too many version checks for me to audit quickly. Honestly I'm not sure it's worth it, though. Could you not just change the test to scan the list of the devices for the one it's looking for? | |
| 13:05:47 | kashyap | I think I've got a lead; will update once done. | |
| 13:05:55 | kashyap | mdbooth: Please disregard me for now. | |
| 13:07:28 | mdbooth | kashyap: If I wanted to track it down quickly, I'd modify the unit test to spit out the device list, compare the generated devices with and without the bumped version number, then go find the guard in that code. | |
| 13:08:40 | kashyap | mdbooth: I don't know how to get the device list. | |
| 13:08:52 | kashyap | That's why I went the PDB route with and without my patch | |
| 13:09:16 | kashyap | And got that: dir(guest.devices[0]) output at the bottom: http://paste.openstack.org/raw/743168/ | |
| 13:09:35 | mdbooth | kashyap: Right, so you've worked out what device was removed? | |
| 13:10:07 | kashyap | 'port' seems to be missing on the second iteration | |
| 13:10:19 | kashyap | mdbooth: And ... weirdly enough, see this comment from migi on the review: | |
| 13:10:24 | kashyap | https://review.openstack.org/#/c/632507/2/nova/virt/libvirt/driver.py | |
| 13:11:16 | kashyap | He says updating MIN_QEMU_VIRTLOGD from 2.7.0 --> 2.9.0 "fixes" it. | |
| 13:11:31 | kashyap | But...that constant will also be removed, as the new MIN satisfies the condition. | |
| 13:11:45 | kashyap | If you're on something else, ignore it. Don't split your attention | |
| 13:29:15 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Convert port to str when validate console port https://review.openstack.org/632720 | |
| 13:51:10 | kashyap | Haha, I was wondering why on _earth_ a particular console code behaviour is the way it is ... and the code comment points to a RHT Bugzilla. | |
| 13:51:37 | kashyap | I click it ... and it points to a bug I filed in Jan-2012! -- https://bugzilla.redhat.com/show_bug.cgi?id=781467 | |
| 13:51:38 | openstack | bugzilla.redhat.com bug 781467 in libvirt "RFE: virsh: 'console' support for non-PTY (unix, tcp)" [Medium,New] - Assigned to libvirt-maint | |
| 14:12:39 | openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: Add fill_virtual_interface_list online_data_migration script https://review.openstack.org/614167 | |
| 14:12:39 | openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal https://review.openstack.org/591607 | |
| 14:13:11 | jaypipes | belmoreira: good afternoon, sir. just wondering if you'd found any time to test out that single-query-per-cell instance info fetcher patch (https://review.openstack.org/#/c/623558/)? no worries if not, I know you're super busy | |
| 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: libvirt: implement reshaper for vgpu https://review.openstack.org/599208 | |
| 23:35:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for libvirt vgpu reshape https://review.openstack.org/631559 | |
| 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: libvirt: implement reshaper for vgpu https://review.openstack.org/599208 | |
| 23:54:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for libvirt vgpu reshape https://review.openstack.org/631559 | |
| #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 | |