Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-04
23:28:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc (3/3) https://review.openstack.org/550659
23:29:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
23:29:34 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
#openstack-nova - 2018-04-05
02:29:36 openstackgerrit Merged openstack/nova master: Default to py3 for the pep8 tox env because it's stricter https://review.openstack.org/558648
03:44:33 openstackgerrit Merged openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260
04:35:19 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450
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!

Earlier   Later