[RFE] Add string/pattern checks to LBaaSv2 HTTP/HTTPS health monitors

Bug #1523231 reported by Kobi Samoray
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Expired
Wishlist
Unassigned

Bug Description

Load balancer health monitors are usually capable of looking up a string or pattern in the HTTP/HTTPS reply, and pass the check only upon existence of it.
LBaaSv2 health monitor can only validate the HTTP/HTTPS return code, but cannot configure string lookup in the server response.

Currently, a health monitor an be configured to look for a 200 from "GET /", as an indicator that the backend member is working.

This bug adds the use case of checking for 200 + some string in the response. E.g. looking for 200 and "happy birthday", to avoid the case where a page decides to return a 200 but really contains an error string. Yes, loadbalancers have to work around lousy websites *all the time*.

Kobi Samoray (ksamoray)
Changed in neutron:
assignee: nobody → Kobi Samoray (ksamoray)
tags: added: lbaas
Revision history for this message
Brandon Logan (brandon-logan) wrote :

this is something others will want (us at rackspace too). rfe!

Kobi Samoray (ksamoray)
tags: added: rfe
Revision history for this message
Henry Gessau (gessau) wrote :

The title reads like a bug report, and the description adds very little detail. Can you word the title to describe what feature or enhancement you want? And perhaps add a little extra to the description, like a use-case.

Changed in neutron:
status: New → Confirmed
Henry Gessau (gessau)
Changed in neutron:
importance: Undecided → Wishlist
Doug Wiegley (dougwig)
Changed in neutron:
status: Confirmed → Triaged
tags: added: rfe-approved
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Can we get response on comment #2 before going ahead? I am personally unclear on the impact of this request.

tags: removed: rfe
tags: added: rfe
removed: rfe-approved
Revision history for this message
Doug Wiegley (dougwig) wrote : Re: Add string/pattern checks to LBaaSv2 HTTP/HTTPS health monitors

Henry, does that answer your question?

summary: - LBaaSv2 HTTP, HTTPS health monitor validates only return code
+ Add string/pattern checks to LBaaSv2 HTTP/HTTPS health monitors
description: updated
Revision history for this message
Henry Gessau (gessau) wrote :

Thanks Doug, LGTM.

summary: - Add string/pattern checks to LBaaSv2 HTTP/HTTPS health monitors
+ [RFE] Add string/pattern checks to LBaaSv2 HTTP/HTTPS health monitors
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

If Kobi has the code lined up and Doug is happy to support this, so am I, because this sounds reasonable enough. Mind you to preserve the old way for the health monitor to work.

tags: added: rfe-approved
removed: rfe
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

I expect both the cli and the api to change in order to accommodate for the specification of the string to be checked, correct?

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

@Doug: what is your recommendation for feature submission? Spec or no spec?

Changed in neutron:
milestone: none → mitaka-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron-lbaas (master)

Fix proposed to branch: master
Review: https://review.openstack.org/258560

Changed in neutron:
status: Triaged → In Progress
Changed in neutron:
milestone: mitaka-2 → mitaka-3
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :
Changed in neutron:
status: In Progress → Incomplete
assignee: Kobi Samoray (ksamoray) → nobody
milestone: mitaka-3 → none
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for neutron because there has been no activity for 60 days.]

Changed in neutron:
status: Incomplete → Expired
Changed in neutron:
assignee: nobody → Kobi Samoray (ksamoray)
status: Expired → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron-lbaas (master)

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/258560
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Perhaps the time for this has passed.

Changed in neutron:
status: In Progress → Incomplete
assignee: Kobi Samoray (ksamoray) → nobody
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for neutron because there has been no activity for 60 days.]

Changed in neutron:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.