lbaas v2.0 api-ref inconsistencies

Bug #1719259 reported by brenda
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Wishlist
yanpuqing

Bug Description

This bug tracker is for errors with the documentation, use the following as a template and remove or add fields as you see fit. Convert [ ] into [x] to check boxes:

- [x] This doc is inaccurate in this way: ______
- [ ] This is a doc addition request.
- [x] I have a fix to the document that I can paste below including example: input and output.

If you have a troubleshooting or support issue, use the following resources:

 - Ask OpenStack: http://ask.openstack.org
 - The mailing list: http://lists.openstack.org
 - IRC: 'openstack' channel on Freenode

-----------------------------------
Release: 1.8.1.dev40 on 'Tue Jul 18 18:01:21 2017, commit 7581af4'
SHA:
Source: Can't derive source file URL
URL: https://developer.openstack.org/api-ref/networking/v2/

Tags: api-ref
Revision history for this message
brenda (tian-mingming) wrote :

there are three problems with lbaas v2.0 api:
1.There is no "name" attribute in "POST /v2.0/lbaas/health_monitors" request body, but from the lbaas v2.0 code and datamodel , there is a "name" attribute added in to healthmonitor object, and in fact, we can use "name" attribute in request body when creating a healthmonitor.
2. the attribute usage description is not right about "protocol" attribute of listener, you can find this error in "POST /v2.0/lbaas/listeners",like below:
protocol body string The IP protocol. Valid value is icmp, tcp, udp, or null. No default.
3. There is a "subnet_id" attribute in ""POST /v2.0/lbaas/pools", but both the code and datamodel has no "subnet_id" in
lbaas v2 pool object. It's confusing. Why remove "subnet_id" from pool?

Boden R (boden)
summary: - Networking API v2.0 in Networking API Reference
+ lbaas v2.0 api inconsistencies
summary: - lbaas v2.0 api inconsistencies
+ lbaas v2.0 api-ref inconsistencies
Changed in neutron:
status: New → Triaged
importance: Undecided → Wishlist
yanpuqing (ycx)
Changed in neutron:
assignee: nobody → yanpuqing (ycx)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron-lib (master)

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

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

Reviewed: https://review.openstack.org/536288
Committed: https://git.openstack.org/cgit/openstack/neutron-lib/commit/?id=2c98c01e9907248de247b751a8deee1461f8be5d
Submitter: Zuul
Branch: master

commit 2c98c01e9907248de247b751a8deee1461f8be5d
Author: yanpuqing <email address hidden>
Date: Thu Jan 4 08:23:42 2018 +0000

    Fix the mistake of lbaas api-ref

    There are some mistakes of lbaas api-ref.
    1.There is no definition for "status", "status_description",
    "healthmonitors_status", "provider", "subnet_id", "vip_id"
    attributes in the pool, and "listeners", "session_persistence"
    should be defined in the pool.
    2.The protocol_port of listener and the delay of healthmonitor is an
    integer.
    3.The id of healthmonitor of the pool named "healthmonitor_id"
    rather than "healthmonitors".
    4.There is no "name" attribute in "POST /v2.0/lbaas/health_monitors"
    request body, but from the lbaas v2.0 code and datamodel, there is
     a "name" attribute added in to healthmonitor object.
    The patch fixes those mistakes.

    Change-Id: Ia7417c818f6646c419f734d7273fd3bf8eb5aad5
    Closes-Bug: #1719259

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

This issue was fixed in the openstack/neutron-lib 1.19.0 release.

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.