Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-03
19:59:34 gibi_cape mriedem: I guess I accepted that we cannot make our notification consumers to change to versioned.
20:00:07 gibi_cape mriedem: and emiting both notification by default is not logical as most of the consumers consume either legacy or versioned but noth both
20:06:07 openstackgerrit Dan Smith proposed openstack/nova master: Add docs for image type support request filter https://review.opendev.org/657025
20:08:29 mriedem gibi_cape: ah, did that come up during your talk?
20:08:39 mriedem i haven't seen the recording yet (probably not posted)
20:13:09 tssurya gibi_cape: for the record I don't think you should do this upstream as a default, if deployments are annoyed with it they can just set it right ?
20:13:51 mriedem tssurya: most projects consuming nova notifications aren't consuming versioned notifications, right?
20:14:05 mriedem so if the default is versioned, most deployments have to change,
20:14:13 tssurya mriedem: yea they are not
20:14:21 mriedem if the default is both (current), most deployments have extra traffic/performance hit of generating an unused versioned notification
20:14:29 mriedem hence the bug
20:14:44 tssurya but if we have put in a lot of work to get it done we have it available, if they want they can set it in the config
20:14:52 tssurya yea i know the bug :) was there was for his presentation
20:15:20 mriedem i'd argue that we (nova) should work on getting versioned notification support in other projects in openstack
20:15:24 mriedem if we want people using it
20:15:35 mriedem once that is done, then change the default
20:16:10 bauzas uh, my cellsv2 skills are rusty
20:16:12 tssurya sure :) that effort has been going on I guess. I agree, so I think we are saying the same thing right ? keep the default as both for now ?
20:16:33 bauzas I wonder whether the migration.execute() code is run under the super-cond or the cond
20:16:37 bauzas I guess the latter
20:16:45 bauzas since we used the cell MQ
20:16:54 mriedem bauzas: what migration.execute() code?
20:17:03 mriedem tssurya: no i'm not saying that
20:17:05 bauzas mriedem: the conductor task
20:17:16 mriedem "if the default is both (current), most deployments have extra traffic/performance hit of generating an unused versioned notification"
20:17:26 tssurya okay so you want the default to be unversioned and then move when ll projects have moved to both ?
20:17:32 bauzas oh shit
20:17:34 bauzas nevermind
20:17:39 bauzas it calls the scheduler
20:17:42 bauzas so it's super-cond
20:17:45 bauzas silly question it is
20:17:49 tssurya all*
20:18:02 mriedem bauzas: conductor task will be targeted to the source cell
20:18:07 mriedem the context that is
20:18:18 bauzas mmm ?
20:18:20 mriedem i.e. @targes_cell
20:18:26 bauzas oh that yeah
20:18:26 mriedem *@targets_cell
20:20:09 bauzas just to make it clear, this section is run by super-cond (IMO) https://github.com/openstack/nova/blob/master/nova/conductor/tasks/migrate.py#L187
20:20:16 bauzas we don't target the cell yet
20:21:05 mriedem um
20:21:16 mriedem well resize can reschedule and run on a conductor in the cell
20:22:10 mriedem bauzas: whether or not that works depends on your cell conductor config being able to hit the api db https://bugs.launchpad.net/nova/+bug/1781286
20:22:11 openstack Launchpad bug 1781286 in OpenStack Compute (nova) "CantStartEngineError in cell conductor during reschedule - get_host_availability_zone up-call" [Medium,Triaged]
20:22:12 bauzas I'm confused then, but nevermind, I'll just take my devstack and try
20:22:29 mriedem bauzas: devstack will fail on ^ during a resize reschedule
20:22:41 mriedem if there is no reschedule, then yes it's just super-conductor
20:23:12 bauzas ok, the context is just a resize
20:23:25 bauzas and yeah, gotcha
20:26:44 gibi_cape mriedem, tssurya: I don't think we have manpower to transform ceilometer to versioned notifications.
20:29:45 tssurya gibi_cape: agreed i'm just saying changing this to unversioned by default basically means no one will take efforts to move maybe, I do understand the bug scenario and perf issues which is why we have the workaround option for the deployments to set it to unversioned, I wasn't just sure if we should go out and make that the default
20:30:20 tssurya but yea its your call :)
20:30:31 openstackgerrit Merged openstack/nova master: Trivial: use default value in next() func https://review.opendev.org/653362
20:33:31 gibi_cape tssurya: for me it is OK to keep it 'both' and keep the perf bug open. What I cannot do is to promise I will change ceilometer over to versioned
20:33:56 tssurya gibi_cape: fair enough
20:34:07 mriedem it'd be interesting to know if deployers (1) don't even notice the change or (2) noticed and changed to unversioned
20:34:12 mriedem i know godaddy found out and changed it
20:34:21 mriedem since they are heavy on notifications for internal stuff
20:34:37 mriedem by "the change" i mean to 'both' long ago
20:34:51 mriedem there is a guy 2 over from me from godaddy so he'd probably know better
20:43:34 openstackgerrit Merged openstack/nova stable/stein: Fix help for ironic.peer_list config https://review.opendev.org/656632
20:56:42 dansmith \\
21:01:21 openstackgerrit melanie witt proposed openstack/nova master: Revert "Fix target_cell usage for scatter_gather_cells" https://review.opendev.org/653894
21:01:21 openstackgerrit melanie witt proposed openstack/nova master: Fix SynchronousThreadPoolExecutorFixture mock of Future https://review.opendev.org/650171
21:01:22 openstackgerrit melanie witt proposed openstack/nova master: Use futurist.ThreadPoolExecutor in scatter_gather_cells https://review.opendev.org/650172
21:05:29 openstackgerrit Merged openstack/nova master: libvirt: drop MIN_LIBVIRT_POSTCOPY_VERSION https://review.opendev.org/649671
21:08:15 openstackgerrit Merged openstack/nova master: libvirt: remove conditional on VIR_DOMAIN_EVENT_SUSPENDED_POSTCOPY https://review.opendev.org/649674
21:08:24 openstackgerrit Merged openstack/nova master: Add Venn diagram showing taxonomy of traits and capabilities https://review.opendev.org/656032
21:08:32 openstackgerrit Merged openstack/nova master: libvirt: Stop ignoring unknown libvirtError exceptions during volume attach https://review.opendev.org/655696
21:08:40 openstackgerrit Merged openstack/nova master: Remove [ironic]api_endpoint option https://review.opendev.org/643483
21:10:42 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: libvirt: Stop ignoring unknown libvirtError exceptions during volume attach https://review.opendev.org/657049
21:11:22 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: libvirt: Stop ignoring unknown libvirtError exceptions during volume attach https://review.opendev.org/657050
21:14:45 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Stop ignoring unknown libvirtError exceptions during volume attach https://review.opendev.org/657051
21:18:23 mriedem dansmith: nits in the docs patch https://review.opendev.org/#/c/657025/ not sure if you want to modify, but should probably make that a proper subsection so people can link to it
21:29:59 openstackgerrit Matt Riedemann proposed openstack/nova master: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656
21:32:49 lbragstad johnthetubaguy https://opendev.org/openstack/oslo.policy/src/branch/master/oslo_policy/policy.py#L633
22:05:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Add InstanceList.get_uuids_by_host_and_node https://review.opendev.org/657070
22:06:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Optimize SchedulerReportClient.delete_resource_provider https://review.opendev.org/657070
22:11:49 openstackgerrit Merged openstack/nova master: Include all network devices in nova diagnostics https://review.opendev.org/648123
22:11:56 openstackgerrit Merged openstack/nova master: Add ids to sections of flavors guide to allow deep-linking https://review.opendev.org/653458
22:15:31 efried mriedem: where you at? Compute capabilities as traits in 207
22:18:23 mriedem right next to you
22:26:57 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Make nova-tox-functional-py36 reusable https://review.opendev.org/657074
22:31:20 openstackgerrit Dan Smith proposed openstack/nova master: Add a workaround config toggle to refuse ceph image upload https://review.opendev.org/657078
22:35:50 openstackgerrit Balazs Gibizer proposed openstack/nova master: Log when port resource is leaked during port delete https://review.opendev.org/657079
22:54:50 sean-k-mooney https://github.com/openstack/nova/blob/master/nova/scheduler/filters/affinity_filter.py#L108-L111
22:55:39 gmann johnthetubaguy: melwitt this is how currently or in past, we handle the deprecated policy check -L47 https://etherpad.openstack.org/p/ptg-train-xproj-nova-keystone
22:56:28 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rework the PCI passthrough guides https://review.opendev.org/635243
22:57:50 melwitt gmann: thanks
23:20:03 openstackgerrit Chris Dent proposed openstack/nova master: Make nova-tox-functional-py36 reusable https://review.opendev.org/657074
#openstack-nova - 2019-05-04
02:22:16 openstackgerrit Merged openstack/nova master: libvirt: Always disconnect volumes after libvirtError exceptions https://review.opendev.org/655712
03:05:43 openstackgerrit Merged openstack/nova master: Add nova-status upgrade check for minimum required cinder API version https://review.opendev.org/649759
09:16:43 openstackgerrit Merged openstack/nova master: Delete the placement code https://review.opendev.org/618215
12:14:59 lyarwood \o/
12:23:25 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: libvirt: Always disconnect volumes after libvirtError exceptions https://review.opendev.org/657109
12:23:55 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Always disconnect volumes after libvirtError exceptions https://review.opendev.org/657110
12:24:32 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: libvirt: Always disconnect volumes after libvirtError exceptions https://review.opendev.org/657111
12:58:33 openstackgerrit sean mooney proposed openstack/nova master: Libvirt: add nfv job https://review.opendev.org/652197
13:21:18 openstackgerrit sean mooney proposed openstack/nova master: Libvirt: add nfv job https://review.opendev.org/652197
13:39:25 openstackgerrit Matt Riedemann proposed openstack/nova master: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656

Earlier   Later