| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-05 | |||
| 13:04:12 | openstackgerrit | Eric Young proposed openstack/nova master: Support extending attached ScaleIO volumes https://review.openstack.org/554679 | |
| 13:06:11 | openstackgerrit | Merged openstack/nova master: Remove duplicative implementation of temporary directories. https://review.openstack.org/554791 | |
| 13:10:07 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: WIP: Add notifications for removing a member from a server group https://review.openstack.org/559076 | |
| 13:42:55 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/virt/xenapi/test_vm_utils.py (1) https://review.openstack.org/558704 | |
| 13:43:28 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/virt/xenapi/test_vm_utils.py (2) https://review.openstack.org/558993 | |
| 13:46:52 | dansmith | kashyap: found some doc issues in your patch.. fix them up and I'll fast approve | |
| 13:49:09 | kashyap | dansmith: Hey there; thanks! Let me look | |
| 13:52:50 | kashyap | dansmith: When you're back, can you point me to the rendered conf link? I seem to be blind | |
| 13:53:12 | kashyap | Found it: http://logs.openstack.org/84/534384/23/check/build-openstack-sphinx-docs/2f47d33/html/configuration/config.html | |
| 13:55:12 | melwitt | nova meeting in 5 minutes | |
| 13:58:58 | kashyap | dansmith: For the rendering; I am aware of it, actually even discussed on PS-22 | |
| 13:59:14 | kashyap | dansmith: Replied on the review. It is a bug in oslo_config.sphinxext | |
| 13:59:41 | kashyap | dansmith: Also the bullet rendering I use is same as the one used by 'disk_cachemodes' | |
| 14:00:01 | dansmith | kashyap: I know and it looks bad there too | |
| 14:00:15 | dansmith | kashyap: as I said, the typo and the log message are the critical bits there | |
| 14:00:18 | kashyap | dansmith: Right; but it'll be fixed by the oslo_config patch series I pointed out | |
| 14:00:31 | kashyap | dansmith: Yep, that I already am fixing, as noted on the review :-) | |
| 14:00:41 | kashyap | But the config thing, we should leave it as-is | |
| 14:01:04 | kashyap | stephenfin: Since we discussed this here before, can you double-confirm I don't have to re-format the config options here: https://review.openstack.org/#/c/534384/23 | |
| 14:02:38 | stephenfin | kashyap: Yup, done | |
| 14:03:29 | kashyap | stephenfin: Saw that; merci. | |
| 14:03:41 | stephenfin | kashyap: As you noted, there are plenty other options displaying the exact same symptoms so I wouldn't block on that | |
| 14:06:40 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Allow to specify granular CPU feature flags https://review.openstack.org/534384 | |
| 14:08:15 | kashyap | dansmith: When you can, fixed the two critical bits ^. Appreciate the review! | |
| 14:09:29 | openstackgerrit | Eric Fried proposed openstack/nova master: update_provider_tree devref and docstring updates https://review.openstack.org/553476 | |
| 14:30:41 | kashyap | melwitt: I added a quick last item on the open disucssion, if there's time, about: http://lists.openstack.org/pipermail/openstack-dev/2018-April/129048.html | |
| 14:37:02 | openstackgerrit | sahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | |
| 14:37:39 | sahid | mriedem, stephenfin, can you remove your -2 on ^ - the spec has been accepted and that could help to get more reviews | |
| 14:38:14 | stephenfin | sahid: Sure, done | |
| 14:40:18 | mriedem | sahid: done | |
| 14:40:36 | sahid | thakns | |
| 14:47:29 | openstackgerrit | sahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | |
| 14:50:00 | mriedem | efried: setting CONF.neutron.auth_type is going to be required to talk to neutron using ksa right? | |
| 14:50:14 | efried | mriedem: I believe so, yes. | |
| 14:50:17 | efried | uh | |
| 14:50:22 | efried | unless you're going the admin path. | |
| 14:50:31 | mriedem | that's what i was worried about, | |
| 14:50:35 | efried | then you're just using admin auth. | |
| 14:50:45 | mriedem | but if you get here and auth_type is None, http://git.openstack.org/cgit/openstack/nova/tree/nova/network/neutronv2/api.py#n75 | |
| 14:50:48 | efried | which is CONF.something-other-than-neutron.auth_type, I imagine. | |
| 14:50:50 | mriedem | then you've f'cked up right? | |
| 14:51:51 | efried | hum, that code shouldn't be in nova at all. If you had no auth type, that ksa loading shoulda kicked an exception. | |
| 14:52:21 | efried | I'll have to go paw through the ksa code again, gimme a few... | |
| 14:52:22 | openstackgerrit | sahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | |
| 14:53:22 | imacdonn | mriedem: on a very-slightly related note; take a peek at https://review.openstack.org/#/c/558089/ when you get a chance | |
| 14:53:53 | efried | Yes, mriedem, ^ is truly excellent work :) | |
| 14:54:02 | imacdonn | :) | |
| 14:55:50 | efried | mriedem: Okay, I was mistaken. No exception; it returns None. https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/loading/conf.py#L122-L124 | |
| 14:56:47 | mriedem | efried: right, ok, point is i'm trying to know if/when i can log something useful in the logs for when you don't configure nova to talk to neutron https://bugs.launchpad.net/nova/+bug/1761487 | |
| 14:56:47 | openstack | Launchpad bug 1761487 in OpenStack Compute (nova) "Nova, Unexpected API Error while creating VM" [Undecided,Invalid] | |
| 14:56:51 | mriedem | we get that same bug at least weekly | |
| 14:58:16 | efried | mriedem: This would be very similar in spirit to what imacdonn noted above. We used to try to determine whether you'd configured placement based on whether [placement]os_region_name was present. Which isn't the best way to do it, because that guy can default - auth_type would have been a better one to check. | |
| 14:58:57 | efried | mriedem: Which leads me to this one: you could have a similar check for neutron config. Though I don't know whether neutron is required to make the world work, so having the check up front may be too aggressive. | |
| 14:59:13 | efried | mriedem: But yeah, you could at least make that error message that you pointed out more clear. | |
| 14:59:44 | efried | mriedem: "You need to configure the [neutron] section of your conf with ksa Adapter options." | |
| 14:59:49 | efried | kind of thing. | |
| 15:00:01 | zigo | o/ | |
| 15:00:02 | mriedem | neutron is required yeah, because we assume port binding is available in neutron and the port binding extension is an admin-only extension by default | |
| 15:00:19 | kashyap | zigo: Hi there ... so to follow-up, all the distributions in the DistroSupportMatrix have libvirt 3.8.0 and QEMU 2.9.0 | |
| 15:00:20 | mriedem | efried: so i'll start small with that kind of log message if we hit this, | |
| 15:00:25 | zigo | I haven't tried backporting libvirt & qemu to Stretch, could you give me some time so I can tell if it's painful or not? | |
| 15:00:27 | kashyap | zigo: This is the wiki: https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix | |
| 15:00:29 | efried | mriedem: ++ | |
| 15:00:38 | mriedem | efried: and i also noticed during triage of that bug that our install guide, at least for ubuntu, doesn't tell you that you need to configure nova with neutron creds | |
| 15:00:45 | kashyap | zigo: Sure. Please respond here: http://lists.openstack.org/pipermail/openstack-dev/2018-April/129048.html | |
| 15:00:53 | zigo | Will do. | |
| 15:01:00 | zigo | I should be able to give an answer tomorrow. | |
| 15:01:06 | kashyap | zigo: I spent a couple of hours duking around various Debian packaging URLs & wrote that up | |
| 15:01:09 | kashyap | Thanks! | |
| 15:01:14 | zigo | (today, got 2 hours driving back home from work in Geneva) | |
| 15:01:32 | mriedem | dansmith: https://review.openstack.org/#/c/555416/ is ready to go if you want to re-apply your +2 | |
| 15:01:51 | tssurya | thanks ^^ | |
| 15:05:29 | mriedem | tssurya: unrelated, but would you mind pushing a simple patch to add a .. note:: to http://logs.openstack.org/59/556459/9/check/build-openstack-sphinx-docs/66f4bf4/html/admin/configuration/schedulers.html#cell-filters that those filters are only for cells v1 which is deprecated | |
| 15:06:03 | tssurya | mriedem: sure, looking at it now.. | |
| 15:12:21 | zigo | kashyap: You can just scrap Jessie from your table, it's unsupported already, and maybe add Sid. | |
| 15:12:25 | zigo | That's what counts for me. | |
| 15:12:47 | kashyap | zigo: "Sid" == Testing / unstable, isn't it? | |
| 15:12:55 | kashyap | zigo: Isn't it "Buster" that's the upcoming release? | |
| 15:13:21 | zigo | kashyap: Generally, we have sid == buster, package migrate in 5 days from Sid to Buster if there's no RC bug. | |
| 15:13:31 | kashyap | zigo: "Also ... it's a wiki, feel free to update it" :-) | |
| 15:13:36 | zigo | :) | |
| 15:14:05 | kashyap | zigo: When I checked earlier this week, there was no ETA for Buster (love saying it out loud) | |
| 15:14:32 | zigo | Sure, that's what Debian is about: it's ready when it's ready, no commercial agenda. | |
| 15:14:33 | bauzas | dansmith: mriedem: melwitt: FYI, I had a family loss this week, so I won't be around tomorrow | |
| 15:14:45 | zigo | What's surprising is that there's still no freeze date. | |
| 15:14:46 | kashyap | zigo: Either way, if you see, we're trying to converge on a reasonable least common denonimnator version for libvirt / QEMU across releases | |
| 15:14:54 | dansmith | bauzas: ack :( | |
| 15:14:58 | zigo | I guess the release team has a few more transition to get done before announcing the freeze date. | |
| 15:15:12 | bauzas | hence also me not around this afternoon :( | |
| 15:15:15 | melwitt | sorry to hear that bauzas :( | |
| 15:15:20 | zigo | bauzas: Oh, sorry to read that. Moral support and all... | |
| 15:15:21 | kashyap | bauzas: Take care of that; IRC can wait | |
| 15:15:53 | bauzas | np, it was my grand father who was 91 | |
| 15:19:40 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Update the Cell filters section of the scheduler docs https://review.openstack.org/559108 | |
| 15:21:04 | tssurya | mriedem: ^ | |
| 15:25:29 | mriedem | bauzas: ack; making it that long is a feat | |
| 15:26:00 | bauzas | yup, and with a way to leave us quickly without illness | |
| 15:26:10 | bauzas | just because the body stopds | |
| 15:26:21 | bauzas | I just hope everyone can leave the world like him | |
| 15:27:33 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Log a more useful error when neutron isn't configured https://review.openstack.org/559111 | |