Cluster-list does not have few parameters implemented for filters option

Bug #1548081 reported by Ayush Garg
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
senlin
Fix Released
Low
Ayush Garg

Bug Description

As per the document http://docs.openstack.org/developer/senlin/developer/cluster.html, in listing clusters filters option can have parameters name, status, project and user. But in implementation, https://github.com/openstack/senlin/blob/master/senlin/api/openstack/v1/clusters.py#L132, It does not contain parameters project and user.

Ayush Garg (ayushgrg93)
Changed in senlin:
assignee: nobody → Ayush Garg (ayushgrg93)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to senlin (master)

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

Changed in senlin:
status: New → In Progress
Revision history for this message
Qiming Teng (tengqim) wrote :

Please fix the documentation instead of the code. We have an API published here: http://developer.openstack.org/api-ref-clustering-v1.html

We would need to cross-verify the code and documentation to make a decision.

In this case, filtering clusters by 'project' and/or 'user' is not that interesting a use case to support.

In future, when there is a need, we can add them back.

Changed in senlin:
status: In Progress → Triaged
importance: Undecided → Low
milestone: none → mitaka-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on senlin (master)

Change abandoned by Qiming Teng (<email address hidden>) on branch: master
Review: https://review.openstack.org/283718
Reason: superceded by https://review.openstack.org/288269

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to senlin (master)

Reviewed: https://review.openstack.org/288269
Committed: https://git.openstack.org/cgit/openstack/senlin/commit/?id=7f97ab6e5ea44627cc56f1a5cf19c55fe862d6bc
Submitter: Jenkins
Branch: master

commit 7f97ab6e5ea44627cc56f1a5cf19c55fe862d6bc
Author: tengqm <email address hidden>
Date: Fri Mar 4 01:43:53 2016 -0500

    Fix cluster list documentation

    There is an inconsistency between doc and implementation regarding
    cluster list. This patch fixes the doc.

    Change-Id: I4d07447ef81236ad5b02855d65d5d1239aef1357
    Closes-Bug: #1548081

Changed in senlin:
status: Triaged → Fix Released
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/senlin 1.0.0.0b3

This issue was fixed in the openstack/senlin 1.0.0.0b3 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.