yahoo-eng-team team mailing list archive

[OpenStack Infra <2097257@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.opendev.org/c/openstack/neutron/+/940707
Committed: https://opendev.org/openstack/neutron/commit/d98d028527fb959f89cc4c3f5a28337b1e61c2fd
Submitter: "Zuul (22348)"
Branch:    master

commit d98d028527fb959f89cc4c3f5a28337b1e61c2fd
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date:   Tue Feb 4 15:12:14 2025 +0000

    [OVN] Change ``PeriodicWorker`` executor to ``ThreadPoolExecutor``
    
    The default executor factory for the ``periodics.PeriodicWorker``
    instance is ``futurist.SynchronousExecutor``. That creates a caller that
    is executed synchronously with the current thread. The
    ``futurist.ThreadPoolExecutor`` instead creates a new thread pool to
    execute the calls asynchronously. Now eventlet is removed from the
    Neutron API and kernel threads are using, this executor will be more
    accurate when calling the periodic methods.
    
    Closes-Bug: #2097257
    Change-Id: Ie01fd776856463593e016a0a22a29cb5666ea896


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/2097257

Title:
  [eventlet-removal] Use ``ThreadPoolExecutor`` for
  ``periodics.PeriodicWorker``

Status in neutron:
  Fix Released

Bug description:
  The default ``executor_factory`` in
  ``futurist.periodics.PeriodicWorker`` is ``SynchronousExecutor``. This
  executor is not reliable to provide an on-time periodic execution for
  the requested method. When the eventlet removal finishes and all
  workers are using kernel threads, it will be recommended to switch to
  ``ThreadPoolExecutor`` by default.

  This class is used in the OVN ``MaintenanceThread`` class.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/2097257/+subscriptions