watcher

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

Bug #1534615 reported by Vincent Françoise on 2016-01-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
​ watcher
Fix Released
Medium
Gábor Antal
watcher mitaka-2 "m2"
Also affects project (?) Also affects distribution/package Nominate for series

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 Edit Tag help
Vincent Françoise (vincent-francoise) on 2016-01-15
Changed in watcher:
importance: Undecided → Medium
David TARDIVEL (david-tardivel) on 2016-01-15
Changed in watcher:
status: New → Confirmed
Gábor Antal (gabor.antal) on 2016-01-18
Changed in watcher:
assignee: nobody → Gábor Antal (gabor.antal)
Antoine Cabot (antoinecabot) on 2016-01-18
Changed in watcher:
milestone: none → mitaka-2
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
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  Edit
Everyone can see this information.
Subscribing...

Other bug subscribers