| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-05 | |||
| 04:35:20 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514 | |
| 04:53:45 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/virt/xenapi/test_vm_utils.py (2) https://review.openstack.org/558993 | |
| 05:37:18 | openstackgerrit | Nakanishi Tomotaka proposed openstack/nova master: Test availability zone in multiple cells https://review.openstack.org/558998 | |
| 05:38:55 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | |
| 07:04:45 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772 | |
| 07:57:49 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Check pinning support in NUMATopologyFilter https://review.openstack.org/531049 | |
| 07:57:50 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Enable cpu pinning with libvirt/QEMU driver https://review.openstack.org/554076 | |
| 07:57:50 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add NumaTopology support for libvirt/qemu driver https://review.openstack.org/530451 | |
| 08:43:59 | kashyap | alex_xu_: Hi there, will you be able to merge this: https://review.openstack.org/#/c/534384/ | |
| 08:44:17 | kashyap | It also got thorough review from johnthetubaguy ^ | |
| 08:45:30 | kashyap | We should be able to merge ready patches (with thorough reviews) during CET / UTC hours. | |
| 08:49:02 | kashyap | cdent: This is one of the fundamental blocking points that DanPB raised many moons ago | |
| 08:56:34 | lyarwood | mdbooth: https://review.openstack.org/#/c/543569/ was rebased and lost your +1 if you have time today, I'll push for more reviews later once NA are online. | |
| 09:02:14 | kashyap | lyarwood: Saw it last night; I have a small remark | |
| 09:02:28 | kashyap | Oh, not this one, it's the other | |
| 09:03:30 | kashyap | It's this: https://review.openstack.org/#/c/544238/ I'll write in the review | |
| 09:11:16 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Enable NUMA Features for Libvirt/QEMU Driver https://review.openstack.org/533077 | |
| 09:15:16 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/559032 | |
| 09:18:27 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/559034 | |
| 09:18:27 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: libvirt: log vm and task state when vif plugging times out https://review.openstack.org/559033 | |
| 09:35:19 | openstackgerrit | Merged openstack/nova master: Move configurable mkfs to privsep. https://review.openstack.org/551921 | |
| 09:35:27 | openstackgerrit | Merged openstack/nova master: Move xenapi xenstore_read's to privsep. https://review.openstack.org/552241 | |
| 10:15:18 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: libvirt: log vm and task state when vif plugging times out https://review.openstack.org/559033 | |
| 10:15:19 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/559034 | |
| 11:12:47 | openstackgerrit | Doug Hellmann proposed openstack/nova master: add lower-constraints job https://review.openstack.org/555961 | |
| 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 | openstack | Launchpad bug 1761487 in OpenStack Compute (nova) "Nova, Unexpected API Error while creating VM" [Undecided,Invalid] | |
| 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: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. | |