yahoo-eng-team team mailing list archive

[OpenStack Infra <1849547@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.opendev.org/690144
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=48730d9449c0e425637f0f1182d81f60a3d259ac
Submitter: Zuul
Branch:    master

commit 48730d9449c0e425637f0f1182d81f60a3d259ac
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date:   Tue Oct 22 17:51:18 2019 +0000

    Use threads insted of greethreads in IP monitor
    
    IP monitor is a method that is going to be executed in a separate
    process, to monitor the IP addresses changes in a namespace.
    
    This method spawns a thread to read from a socket opened by Pyroute2.
    The read function is a blocking method that will end only when the
    socket is closed. To avoid thread starvation that can happen using
    greenthreads, IP monitor will use kernel threads.
    
    This will increase the resources used but will ensure that no message
    is lost when reading the monitor socket.
    
    Reduced the number of IPs generated in "test_add_and_remove_multiple_ips"
    to shrink the testing time used.
    
    Change-Id: I3fbba2854d40ab0f683443aa30c2a95752345d2e
    Closes-Bug: #1849547


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

Title:
  Use threads insted of greethreads in IP monitor

Status in neutron:
  Fix Released

Bug description:
  IP monitor is a method that is going to be executed in a separate 
  process, to monitor the IP addresses changes in a namespace.

  This method spawns a thread to read from a socket opened by Pyroute2.
  The read function is a blocking method that will end only when the 
  socket is closed. To avoid thread starvation that can happen using 
  greenthreads, IP monitor should use kernel threads. 

  This will increase the resources used but will ensure that no message 
  is lost when reading the monitor socket.

  Reduced the number of IPs generated in "test_add_and_remove_multiple_ips"
  to shrink the testing time used.

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