Audit & action plans api controllers - unused parameter in get_all() and detail()

Bug #1534615 reported by Vincent Françoise
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
watcher
Fix Released
Medium
Gábor Antal

Bug Description

In the Audit controller (watcher/api/controllers/v1/audit.py), we have an unused "audit_uuid" parameter in both get_all() and detail()

Similarly, the Action Plan controller (watcher/api/controllers/v1/action_plan.py) has an unused "action_plan_uuid" parameter in both get_all() and detail().

We need to remove them as they are useless

Tags: api
Changed in watcher:
importance: Undecided → Medium
Changed in watcher:
status: New → Confirmed
Changed in watcher:
assignee: nobody → Gábor Antal (gabor.antal)
Changed in watcher:
milestone: none → mitaka-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to watcher (master)

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

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

Reviewed: https://review.openstack.org/269158
Committed: https://git.openstack.org/cgit/openstack/watcher/commit/?id=58fb86a3dae82efd0e6f1bdad7a8c1a2b787f1f3
Submitter: Jenkins
Branch: master

commit 58fb86a3dae82efd0e6f1bdad7a8c1a2b787f1f3
Author: Gábor Antal <email address hidden>
Date: Mon Jan 18 16:55:18 2016 +0100

    Removed unused parameters from api controllers

    In the Audit controller (watcher/api/controllers/v1/audit.py),
    we have an unused "audit_uuid" parameter in both get_all()
    and detail()

    Similarly, the Action Plan controller
    (watcher/api/controllers/v1/action_plan.py) has an unused
    "action_plan_uuid" parameter in both get_all() and detail().

    I also removed them from the @wsme_pecan.wsexpose decorator

    Change-Id: I7f1edfd44dd95c9768249650e19b16fcbe916b89
    Closes-Bug: #1534615

Changed in watcher:
status: In Progress → Fix Released
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.