← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1600326] Re: neutron-lbaas health monitor timeout and delay values interpreted as milliseconds

 

Reviewed:  https://review.openstack.org/380660
Committed: https://git.openstack.org/cgit/openstack/octavia/commit/?id=ef11747a56ed210d9ae59e7c08b08fb6b6dc604b
Submitter: Jenkins
Branch:    master

commit ef11747a56ed210d9ae59e7c08b08fb6b6dc604b
Author: Paul Glass <paul.glass@xxxxxxxxxxxxx>
Date:   Fri Sep 30 21:39:52 2016 +0000

    Switch HAProxy health check timeout to seconds
    
    Change-Id: If8166b8e76ca6c1b15963ef99bec07ff2a6fb118
    Closes-Bug: #1600326


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

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

Title:
  neutron-lbaas health monitor timeout and delay values interpreted as
  milliseconds

Status in neutron:
  Fix Released
Status in octavia:
  Fix Released

Bug description:
  The timeout and delay values on the health monitor objects in Neutron
  LBaaS are purportedly in units of seconds, but the numeric value is
  passed all the the way down to the HAProxy configuration[1] file (in
  both the HAProxy namespace driver and Octavia) where it is interpreted
  in milliseconds:

  * https://cbonte.github.io/haproxy-dconv/configuration-1.5.html#4.2-timeout%20check
  * https://cbonte.github.io/haproxy-dconv/configuration-1.5.html#5.2-inter

  Due to this unit mismatch, a user may configure a pool with a
  reasonable 10 second timeout, and the service may appear to function
  normally until even a small load causes the backend servers to exceed
  a 10 millisecond timeout and then they are removed from the pool.

  A timeout value less than one second is useful some settings, such as
  monitoring a pool of backend servers serving static content, let the
  database field stores this value as an integer.

  1: https://github.com/openstack/neutron-
  lbaas/blob/b322615e4869eb42ed7888a3492eae4a52f3b4db/neutron_lbaas/services/loadbalancer/drivers/haproxy/templates/haproxy_proxies.j2#L72

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


References