Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-19
14:17:50 sahid dansmith: nothing is merged
14:18:17 dansmith sahid: I know, I mean before +2ing.. I expected you were testing these other configs locally before review
14:18:30 sahid i expected to have that env with DPDK ready before the final +2+W
14:19:36 sahid i'm going to fix the issue in libvirt but for nova i guess we should just avoid using name attribute for vhostuser interface
14:24:26 dansmith sean-k-mooney[m]: are there jobs anywhere in neutron or experimental that use vhost/vhostuser that we can depends-on changes like this to get at least a smoke test?
14:28:58 jaypipes sahid: quick update needed on https://review.openstack.org/#/c/562565/ commit message and then I'm +2 on the whole series.
14:30:54 sahid jaypipes: cool, i will fix that, thanks
14:30:57 Nisha_Agarwal mriedem, then i will go ahhead n raise the defect?
14:32:01 mriedem Nisha_Agarwal: i'm in a meeting so don't have time to investigate this, nor the ironic background, so for tracking it yes if you want to report a bug that's fine
14:32:37 openstackgerrit Merged openstack/nova stable/ocata: Avoid unnecessary lazy-loads in mutated_migration_context https://review.openstack.org/515581
14:32:45 openstackgerrit Merged openstack/nova stable/ocata: Only attempt a rebuild claim for an evacuation to a new host https://review.openstack.org/550560
14:49:05 openstackgerrit zhangyangyang proposed openstack/nova master: Deprecate auth_uri option https://review.openstack.org/562537
15:00:47 stephenfin jaypipes, bauzas: Tell me more about https://review.openstack.org/#/c/557065/2/specs/rocky/approved/vgpu-rocky.rst
15:01:13 stephenfin jaypipes, bauzas: That's a very similar issue we have for NUMA-aware vSwitch
15:01:26 bauzas it's about configuring things
15:01:43 stephenfin I've been trying to get dynamic '[physnet_$PHYSNET_NAME]' groups created on the fly but oslo.config isn't designed that way
15:01:57 bauzas stephenfin: yeah I remember your spec
15:02:19 stephenfin So what I've done is ugly enough to not even push to Gerrit, heh https://github.com/stephenfin/nova/commit/b2c0421e64d627aaad2cf9ac71c6e1c79357a970
15:02:23 bauzas looks like we're limited for describing resources or topologies
15:02:51 bauzas stephenfin: the problem is that oslo.config is pretty freeform
15:03:06 bauzas stephenfin: and IMHO the pci whitelist is a terrible interface
15:03:50 stephenfin Right, well I'd be interested in figuring out what you do. I don't think what I have is good enough so I'm open to ideas
15:04:09 stephenfin Until then, I'll probably revert back to a JSON option just so I can proceed
15:05:08 jaypipes stephenfin: dansmith is gonna hate that.
15:05:16 mriedem stephenfin: as in StrOpt with serialized json?
15:05:18 mriedem i also hate that
15:05:20 takashin python-novaclient stable maintainers for pike, would you review https://review.openstack.org/#/c/562500/ ? This patch fixes pike gate job failure.
15:05:25 mriedem i would rather us load in a yaml file
15:05:28 stephenfin jaypipes: Oh, I'm not planning to keep it
15:05:31 mriedem if that's the alternative
15:05:51 melwitt takashin: I think you need to backport that to queens first, then to pike
15:06:06 stephenfin It just seems like we don't have a decided-upon alternative and I need to load _something_
15:06:11 mriedem takashin: done
15:06:13 dansmith stephenfin: I thought I suggested the cinder model for encoding single-level dicts in config with sections?
15:06:21 mriedem melwitt: fix came from queens
15:06:21 dansmith it's stupid simple
15:06:32 takashin melwitt: it has already merged in queens.
15:06:37 stephenfin dansmith: Yeah, it seems horrible though -> https://github.com/stephenfin/nova/commit/b2c0421e64d627aaad2cf9ac71c6e1c79357a970
15:06:44 melwitt mriedem, takashin: oh, okay. my bad
15:07:05 takashin mriedem: Thank you.
15:07:12 stephenfin dansmith: With cinder, they still know the group names ahead of time. We're having to build that based on another value
15:07:21 stephenfin It works, but it seem...wrong (IMO, anyway)
15:07:42 dansmith stephenfin: eh? they let you set your own section names.. backend_foo and what not
15:08:42 bauzas dansmith: I don't have the context, what were you suggesting and do you think it could apply for https://review.openstack.org/#/c/557065/ ?
15:09:18 dansmith bauzas: stephenfin: https://wiki.openstack.org/wiki/Cinder-multi-backend
15:09:24 mriedem bauzas: http://logs.openstack.org/54/554154/27/check/tempest-full/fbfe7c5/controller/logs/etc/cinder/cinder_conf.txt.gz
15:09:43 mriedem enabled_backends = lvmdriver-1
15:09:47 mriedem [lvmdriver-1]
15:09:51 dansmith exactly that
15:10:03 bauzas I see
15:10:13 bauzas so dynamically importing opt groups ?
15:10:16 mriedem http://logs.openstack.org/54/554154/27/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/b88aa0b/logs/etc/cinder/cinder.conf.txt.gz for ceph
15:10:19 stephenfin dansmith: I need to figure out the heck that is working so. All I could find was this https://github.com/openstack/cinder/blob/master/cinder/volume/configuration.py
15:10:27 stephenfin and all calls to that were statically defined
15:10:44 bauzas mriedem: dansmith: okay, I need to consider that for https://review.openstack.org/#/c/557065/
15:10:47 stephenfin dansmith, mriedem, bauzas: but yeah, my configuration was looking like this http://paste.openstack.org/show/719567/
15:11:09 stephenfin think that was what was proposed in the review
15:11:25 dansmith stephenfin: right I think that's highly desirable over a separate yaml or json blob unless there's a really good reason not to
15:11:34 stephenfin dansmith: I concur
15:12:19 mriedem you could always....ask someone in -cinder about how that impl works
15:12:26 dansmith WAT
15:12:29 dansmith can you do that?
15:12:34 mriedem or if it can be shared into oslo.config so we can re-use it
15:12:39 mriedem remember oslo-incubator?
15:12:44 stephenfin but what if they're mean to me?
15:12:50 mriedem they are pretty nice
15:12:52 mriedem nicer than us
15:12:59 mriedem maybe...too nice
15:15:54 bauzas I have one terrible thought
15:16:08 bauzas if we say we want to configure things on the fly
15:16:22 bauzas wouldn't that be something in nova-manage ?
15:16:46 stephenfin bauzas: Isn't this stuff based on pretty static configuration though?
15:17:07 stephenfin I mean, unless you're hotplugging vGPUs/NICs
15:17:41 bauzas isn't the purpose of nova-manage to make nova aware of things that you want to set either once or multiple times ?
15:19:33 stephenfin OK, solved it (the missing piece was knowing that 'enabled_backends' was where this was used)
15:19:48 stephenfin https://github.com/openstack/cinder/blob/master/cinder/cmd/volume.py#L161-L162 and https://github.com/openstack/cinder/blob/master/cinder/cmd/volume.py#L89 for anyone interested
15:21:30 openstackgerrit Balazs Gibizer proposed openstack/nova-specs master: Suggest when to propose notification changes https://review.openstack.org/562265
15:25:15 arvindn0_ mriedem: i see you replied to https://review.openstack.org/#/c/560718/
15:25:33 arvindn0_ any thoughts on the approach i proposed in the spec?
15:28:45 arvindn0_ trying to reduce the chance of encountering the rebuild bug...but cant seem to avoid the bug completely...
15:28:50 openstackgerrit Eric Fried proposed openstack/nova-specs master: Granular: group_policy={none|isolate} https://review.openstack.org/562687
15:35:03 openstackgerrit sahid proposed openstack/nova master: libvirt: fix wrong driver name for vhostuser interface https://review.openstack.org/561562
15:35:04 openstackgerrit sahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997
15:35:05 openstackgerrit sahid proposed openstack/nova master: libvirt: don't set interface backend for vhostuser https://review.openstack.org/562708
15:36:09 mriedem arvindn0_: i will have to get to it later, but it's in the queue
15:38:06 arvindn0_ got it. i put 2 alternatives in there along with why we may not want to go with them based on previous discussions..fyi
15:52:36 openstackgerrit Balazs Gibizer proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306
15:58:24 openstackgerrit Merged openstack/nova master: Deduplicate notification samples Rocky - 7 https://review.openstack.org/560748
16:09:10 openstackgerrit zhangyangyang proposed openstack/nova master: Deprecate auth_uri option https://review.openstack.org/562537
16:09:14 openstackgerrit Chris Friesen proposed openstack/nova stable/pike: libvirt: disconnect volume from host during detach https://review.openstack.org/560690
16:09:22 lyarwood http://logs.openstack.org/04/516404/3/gate/legacy-tempest-dsvm-neutron-full/6d5d851/logs/screen-n-cpu.txt.gz?level=ERROR#_2018-04-19_14_41_33_712 - `AttributeError: 'NoneType' object has no attribute 'generation'` is leading to stable/ocata gate failures FWIW
16:10:10 melwitt urgh
16:10:48 melwitt how did that happen
16:12:55 lyarwood https://review.openstack.org/#/c/525309/ maybe?
16:13:02 lyarwood efried: ^ any ideas?
16:19:38 melwitt hm, maybe. a resource provider is ending up as None
16:23:32 efried lyarwood: looking...
16:29:04 openstackgerrit Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757
16:29:20 openstackgerrit zhangyangyang proposed openstack/nova master: Deprecate auth_uri option https://review.openstack.org/562537
16:30:17 cfriesen mriedem: melwitt: Do we want to address bug 1764556? If so, is there a better option than changing the online data migration in Pike to add UUIDs to deleted services?

Earlier   Later