[RFE] Port based HealthMonitor in neutron_lbaas

Bug #1541579 reported by Maharajan Nachiappan
52
This bug affects 7 people
Affects Status Importance Assigned to Milestone
octavia
Fix Released
Wishlist
cheng

Bug Description

Summary:
Neutron LbaaS lags port based monitoring.

Description:
The current HealthMonitory attached to pool that monitors the member port by default. But some use case may run their service monitoring in a different port rather than the service port, these type of ECV is incapable in the current HealthMonitoring Object.

Expected:
We have to have a new field called 'destination': 'ip:port', since most of the external LBs support this feature and organizations uses it. since pool can have multiple HealthMonitors attached to it.

'destination': {'allow_post': True, 'allow_put': True,
                        'validate': {'type:string': None},
                        'default': '*:*',
                        'is_visible': True},

Version: Kilo/Liberty

Tags: rfe
Revision history for this message
Brandon Logan (brandon-logan) wrote :

this should be an rfe as it will basically be an addition to the api and as such would need to be done as an extension.

Changed in neutron:
importance: Undecided → Wishlist
status: New → Confirmed
tags: added: lbaas rfe
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Let's see if we can get an opinion from Doug.

Changed in neutron:
status: Confirmed → Triaged
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :
tags: added: rfe-approved
removed: rfe
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Due to current backlog, this is most likely deferred to N.

Henry Gessau (gessau)
summary: - Port based HealthMonitor in neutron_lbaas
+ [RFE] Port based HealthMonitor in neutron_lbaas
affects: neutron → octavia
Revision history for this message
Michael Johnson (johnsom) wrote :

Removing rfe-approved tag and lbaas tag, we should target this for the octavia API.

Also, I would like to see a better name, maybe monitoring_port and monitoring_address on the member. This way it is optional to only list one and leave the other the default member IP or port.

tags: added: rfe
removed: lbaas rfe-approved
Revision history for this message
Phil (philsttr) wrote :

Can you confirm that the neutron lbaas v2 api will be modified to support this?

If I understand correctly, octavia is only one specific implementation. I just want to make sure that this functionality is not specific to octavia, and can be implemented in other drivers.

Revision history for this message
Matt Greene (m-greene) wrote :

Octavia is the name of the (new) endpoint and architecture, but also the reference implementation for current LBaaSv2 plugin driver in neutron-lbaas. AFAIK the plan it to keep LBaaSv2 API for both, so would be a generic modification to the model available to all service providers (octavia and 3rd party vendors).

Revision history for this message
Michael Johnson (johnsom) wrote :
Revision history for this message
Michael Johnson (johnsom) wrote :

Octavia will support this for providers as well as the octavia driver.

cheng (tangch318)
Changed in octavia:
assignee: nobody → cheng (tangch318)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to octavia (master)

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

Changed in octavia:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to octavia (master)

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

commit 594a5935e89ee1bbbf6728aa5014458bf7d64837
Author: cheng <email address hidden>
Date: Wed Apr 26 23:25:23 2017 -0400

    Add monitor address and port to pool member

    Closes-Bug: #1541579

    Change-Id: I8b49726bc2b7fbc37e1cd203a405f4bcbd0b7125
    Signed-off-by: cheng <email address hidden>

Changed in octavia:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/octavia 1.0.0.0b2

This issue was fixed in the openstack/octavia 1.0.0.0b2 development milestone.

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.