| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-23 | |||
| 20:49:33 | mriedem | now it's moved to the front | |
| 20:49:37 | dansmith | "No change is necessary as this was never accounted for properly using | |
| 20:49:38 | dansmith | the overhead model" | |
| 20:50:00 | mriedem | where do you see that? | |
| 20:50:09 | dansmith | that was in the reno before I cut out the virt stuff | |
| 20:50:19 | dansmith | which is why I thought having it missing from the admin docs made sense | |
| 20:50:41 | dansmith | because it's not per-instance unless you have dedicated CPUs, but even still, it wasn't accounted for by the overhead calcs anyway | |
| 20:51:15 | mriedem | ok this reno https://review.opendev.org/#/c/672065/2..3/releasenotes/notes/remove-core-ram-disk-filters-9510cbe5b4e295b6.yaml@a21 | |
| 20:51:34 | dansmith | yeah | |
| 20:53:16 | mriedem | and the overhead isn't used here https://github.com/openstack/nova/blob/383a4cf3710b5da2fe0b580bc25783ec8fefed48/nova/compute/claims.py#L108 | |
| 20:53:18 | mriedem | like for disk and ram above | |
| 20:53:26 | mriedem | wtf, why was it even in the gd driver | |
| 20:53:40 | mriedem | anyway, i see now | |
| 20:54:26 | openstackgerrit | sean mooney proposed openstack/nova master: libvirt: use domain capablites to get supported device models https://review.opendev.org/666915 | |
| 20:54:27 | openstackgerrit | sean mooney proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775 | |
| 21:30:07 | mriedem | dansmith: on the claims removal one what am i not understanding here? https://review.opendev.org/#/c/551026/4//COMMIT_MSG@16 | |
| 21:35:17 | openstackgerrit | Adam Spiers proposed openstack/nova master: libvirt: harden Host.get_domain_capabilities() https://review.opendev.org/670189 | |
| 21:40:47 | aspiers | efried: ^^^ I've beefed up the commit message so it may be slightly less impenetrable now :) | |
| 21:41:37 | efried | noted | |
| 21:42:55 | aspiers | but since this is now a joint effort between sean-k-mooney and myself, maybe it's enough to get a +1 from kashyap | |
| 21:45:14 | mriedem | dustinc: efried: a few questions in the bottom sdk patch https://review.opendev.org/#/c/643664/ | |
| 21:46:48 | efried | mriedem: I have just emerged from the ironic-to-novaclient rabbit hole you sent me down. | |
| 21:46:55 | mriedem | haha | |
| 21:46:56 | efried | thanks (sincerely, no sarcasm) for that. | |
| 21:46:57 | mriedem | i knew it would happen | |
| 21:47:02 | efried | see https://review.opendev.org/#/c/672369/ | |
| 22:01:43 | dansmith | mriedem: regarding my BFV comment? | |
| 22:02:12 | mriedem | yeah | |
| 22:02:46 | dansmith | yes, that's obviously old from the first WIPpy version of this where I was just taking shots against the caching scheduler | |
| 22:03:02 | mriedem | dustinc: efried: also some questions in the ironic driver + sdk patch https://review.opendev.org/#/c/642899/33 - looks like at least in our ci where nova-compute is started once before ironic, we now fail to start the service that first time unlike before where we'd swallow the VirtDriverNotReady error | |
| 22:03:03 | efried | mriedem: responded on https://review.opendev.org/#/c/643664/ | |
| 22:03:23 | dustinc | mriedem: checking them out now, thanks | |
| 22:05:22 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.opendev.org/576020 | |
| 22:05:32 | efried | mriedem: Note that the series forks; if you want an easy +Win, the placement one should be ready (zuul connection borkage notwithstanding) https://review.opendev.org/#/c/656023/ | |
| 22:05:37 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.opendev.org/576027 | |
| 22:05:54 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.opendev.org/576031 | |
| 22:06:09 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.opendev.org/576299 | |
| 22:06:14 | mriedem | dansmith: poor caching scheduler | |
| 22:06:23 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.opendev.org/576344 | |
| 22:06:31 | dansmith | we could work up a remix of the office space printer scene with the caching scheduler | |
| 22:06:44 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.opendev.org/576673 | |
| 22:07:04 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.opendev.org/576676 | |
| 22:07:11 | dansmith | mriedem: https://imgflip.com/i/36cp5t | |
| 22:31:06 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.opendev.org/576689 | |
| 22:31:19 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.opendev.org/576709 | |
| 22:32:16 | mriedem | oh dan | |
| 22:32:26 | mriedem | there are things i've hated in nova more than that | |
| 22:34:22 | redkrieg | mriedem: I was looking at your blueprint for improving rescue mode and it's exactly what I need to offer ISO based OS installs for my clients. I see it's Abandoned for Train, but I was wondering if you knew off-hand what else needs to be done. https://blueprints.launchpad.net/nova/+spec/virt-rescue-stable-disk-devices | |
| 22:34:39 | mriedem | redkrieg: that's not mine | |
| 22:34:43 | mriedem | it's lyarwood's i think | |
| 22:34:55 | redkrieg | my apoligies, I just saw "Started by | |
| 22:35:00 | redkrieg | had your name | |
| 22:35:09 | redkrieg | I pinged him earlier but I think he's away today | |
| 22:35:40 | mriedem | he's in....UK timezone (can't remember if he's london or dublin, i know that's a grave error if i get it wrong) | |
| 22:35:45 | mriedem | dansmith: is lyarwood out this week? ^ | |
| 22:36:09 | dansmith | mriedem: not according to the vacay calendar | |
| 22:36:38 | mriedem | ok i saw that he had re-proposed the spec for train https://review.opendev.org/#/c/651151/ but then abandoned it without any comment why, so idk | |
| 22:36:40 | dansmith | lyarwood is in angleand, not eye'r'land | |
| 22:36:42 | mriedem | but spec freeze is thursday | |
| 22:36:54 | mriedem | can i say he's in the "british isles"? | |
| 22:37:14 | mriedem | he's in a cold dark land of boiled meats and dark brews | |
| 22:38:57 | redkrieg | I pinged him earlier today with a similar message, hopefully he'll see it when he gets in tomorrow. | |
| 22:43:47 | openstackgerrit | Merged openstack/nova master: Drop delete_build_requests_with_no_instance_uuid online migration https://review.opendev.org/651001 | |
| 22:43:57 | openstackgerrit | Merged openstack/nova master: Handle Invalid exceptions as expected in attach_interface https://review.opendev.org/650077 | |
| 22:46:37 | openstackgerrit | Eric Fried proposed openstack/nova master: Remove fake_libvirt_utils users in functional testing. https://review.opendev.org/644793 | |
| 22:48:28 | efried | stephenfin: I put that ^ back to PS10, which was passing tests, thereby resurrecting johnthetubaguy's +2. You were +2 at PS8; would you mind sending it? TIA. | |
| 22:49:12 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.opendev.org/576712 | |
| 22:49:50 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Handle Invalid exceptions as expected in attach_interface https://review.opendev.org/672384 | |
| 23:23:13 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Deprecate the '--config-drive' option https://review.opendev.org/653683 | |
| 23:24:17 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add database schema upgrade check https://review.opendev.org/667047 | |
| 23:36:07 | openstackgerrit | Merged openstack/nova master: Remove deprecated Core/Ram/DiskFilter https://review.opendev.org/672065 | |
| #openstack-nova - 2019-07-25 | |||
| 04:01:16 | openstackgerrit | ZHOU YAO proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/621646 | |
| 05:05:24 | openstackgerrit | melanie witt proposed openstack/nova-specs master: Propose policy rule for host status UNKNOWN https://review.opendev.org/666181 | |
| 07:04:52 | openstackgerrit | ZHOU YAO proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/621646 | |
| 07:25:42 | ChipOManiac | Hi guys. We have an openstack cluster with three KVM compute hosts that we setup via openstack-ansible. We've set up a single Nova-LXD compute unit and then imported a tgz ubuntu cloud image into our images list. | |
| 07:26:07 | ChipOManiac | Our problem seems to be with launching any LXD instances on this new Nova-LXD compute host. | |
| 07:27:23 | ChipOManiac | If we try starting an LXD instance with this image. Nova creates a KVM instance and tries to boot the tgz with it. | |
| 07:27:31 | ChipOManiac | Obviously that won't work. | |
| 07:28:10 | ChipOManiac | Is there any way to make the LXD instance work here? | |
| 07:29:49 | ChipOManiac | I've seen Ubuntu charms deployments have a 'root-tar' image format. I don't see that here in our openstack-ansubile. Is there any way for me to add this disk format? | |
| 08:20:16 | kashyap | aspiers: Catching up with the long scroll here...buried in several things | |
| 08:58:06 | openstackgerrit | Merged openstack/nova master: Correct project/user id descriptions for os-instance-actions https://review.opendev.org/670027 | |
| 09:01:19 | stephenfin | alex_xu: Would you be okay with me fixing this up in a follow-up? | |
| 09:01:20 | stephenfin | <mschuppert> kashyap: yes. when you'd reserve one host per supported version and can not get multiple versions on one host. | |
| 09:01:20 | stephenfin | <mschuppert> kashyap: like I mentioned yesterday . the devnest pool is a shared pool for multiple DFGs. there are only 2 systems or so which are really exclusive for compute. | |
| 09:01:21 | stephenfin | <kashyap> mschuppert: Hmm, didn't realize that fully | |
| 09:01:24 | stephenfin | whoops | |
| 09:01:33 | stephenfin | https://review.opendev.org/#/c/551026/ | |
| 09:01:48 | stephenfin | HexChat's copy-paste behaviour is FUBAR | |
| 09:02:17 | alex_xu | stephenfin: ok, i'm cool with that | |
| 09:08:23 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Follow-up for I2936ce8cb293dc80e1a426094fdae6e675461470 https://review.opendev.org/672669 | |
| 09:08:27 | stephenfin | alex_xu: ^ | |
| 09:08:59 | alex_xu | thanks | |
| 09:17:34 | kashyap | aspiers: On "what's wrong with option 1 -- do we need to pass virt_type?" -- if libvirt "knows" that KVM is available, then if you pass virt_type as None to getDomanCapabilities(), it defaults to KVM | |
| 09:17:46 | kashyap | I've done a bunch of quick tests | |
| 09:29:14 | aspiers | kashyap, sean-k-mooney: I got a reply from our libvirt guy | |
| 09:29:35 | aspiers | he said "domcapabilities should be the same with or without virttype" | |
| 09:29:45 | aspiers | so I'm not sure why it's a parameter in the API call | |
| 09:30:03 | aspiers | "default on a kvm host would be virttype=kvm" | |
| 09:31:53 | kashyap | aspiers: I'm getting you some diffs and results. 1 sec, uploading the files | |