← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 2071920] Re: The periodic workers service should run only ``PeriodicWorker`` instances

 

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

commit d2e53f83f792e9fc7d6cd444bc35581d25e38a5f
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date:   Thu Jul 4 10:53:14 2024 +0000

    Limit the worker type in neutron-periodic-workers service
    
    The service "neutron-periodic-workers" is in charge of executing the
    ML2 plugin periodic workers (usually ``PeriodicWorker`` type). These
    workers have a worker_process_count=0.
    
    Before this patch, the method ``start_periodic_workers`` was also
    spawning other ML2 plugin workers, as the ML2/OVN ``MaintenanceWorker``,
    not expected to be running in this service.
    
    Closes-Bug: #2071920
    Related-Bug: #2069581
    Change-Id: Ic87ec5cc5037cab6eb0e24746a95ab4c8447870b


** 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/2071920

Title:
  The periodic workers service should run only ``PeriodicWorker``
  instances

Status in neutron:
  Fix Released

Bug description:
  In [1], a new service was created to execute the ML2 plugin periodic
  workers. However, this is not only executing the periodic workers but
  also other plugin workers (for example, in ML2/OVN, the maintenance
  worker). Same as done in [2], the method retrieving the plugin workers
  should filter out those with worker_process_count>=1.

  [1]https://review.opendev.org/c/openstack/neutron/+/922110
  [2]https://github.com/openstack/neutron/blob/cfab008eef3d11055ad16b8dd0d49aaa1801bc95/neutron/service.py#L278-L281

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



References