Redundant code in storage_backend API

Bug #1836678 reported by junboli
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Low
junboli

Bug Description

Brief Description
-----------------
The base code already provide storage_ceph, storage_lvm, storage_file..., and
already returned by bellow code:
   'return _patch(storage_backend_uuid, patch)'
So, after the return code, there is no use and we don't need keep those code again.

Severity
--------
Major

Steps to Reproduce
------------------
Just remove those dead code.

Tags: stx.config
junboli (junboli)
Changed in starlingx:
assignee: nobody → junboli (junboli)
Changed in starlingx:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to config (master)

Reviewed: https://review.opendev.org/668583
Committed: https://git.openstack.org/cgit/starlingx/config/commit/?id=0e625db2b13c42aba04e8e9424f1a1aaeacd8886
Submitter: Zuul
Branch: master

commit 0e625db2b13c42aba04e8e9424f1a1aaeacd8886
Author: lijunbo <email address hidden>
Date: Mon Jul 15 23:08:29 2019 -0400

    Cleanup dead code in storage_backend.py

    The base code already provide storage_ceph,
    storage_lvm, storage_file..., and already
    returned by bellow code:
       'return _patch(storage_backend_uuid, patch)'
    So, after the return code, there is no use and
    we don't need keep those code again.

    Closes-bug: #1836678
    Change-Id: Ia83e674688b583a40205da5d26d26de150e326a7
    Signed-off-by: junboli <email address hidden>

Changed in starlingx:
status: In Progress → Fix Released
Revision history for this message
Ghada Khalil (gkhalil) wrote :

Low priority / not gating - code cleanup with no functional impact

tags: added: stx.config
Changed in starlingx:
importance: Undecided → Low
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.