Remove the unused codes in config handler

Bug #1835360 reported by Tao Li
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Won't Fix
Low
Tao Li

Bug Description

Brief Description
-----------------
In starlingx metal project, some codes are never be used,so we should remove them avoiding ambiguous.

For example the return statement in these methods as follows.

_config_handler
hbs_config_handler
mtc_config_handler
mtc_nfvi_handler
_nfvi_handler

Severity
--------
minor

Steps to Reproduce
------------------

Expected Behavior
------------------

Actual Behavior
----------------

Reproducibility
---------------

System Configuration
--------------------
STD system

Branch/Pull Time/Commit
-----------------------
master

Tags: stx.metal
Tao Li (eric-litao)
Changed in starlingx:
assignee: nobody → Tao Li (eric-litao)
status: New → In Progress
Tao Li (eric-litao)
summary: - Remove the unused codes in config handler
+ Remove the unused codes in metal
summary: - Remove the unused codes in metal
+ Remove the unused codes
summary: - Remove the unused codes
+ Remove the unused codes in config handler
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to metal (master)

Fix proposed to branch: master
Review: https://review.opendev.org/669073

Revision history for this message
Ghada Khalil (gkhalil) wrote :

Low priority / not gating any starlingx release -- code cleanup; doesn't have any system impact

tags: added: stx.metal
Changed in starlingx:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on metal (master)

Change abandoned by Bart Wensley (<email address hidden>) on branch: master
Review: https://review.opendev.org/669073
Reason: This patch has been idle for more than six months. I am abandoning it to keep the review queue sane. If you are still interested in working on this patch, please unabandon it and upload a new patchset.

Revision history for this message
Ramaswamy Subramanian (rsubrama) wrote :

No progress on this bug for more than 2 years. Candidate for closure.

If there is no update, this issue is targeted to be closed as 'Won't Fix' in 2 weeks.

Revision history for this message
Ramaswamy Subramanian (rsubrama) wrote :

Changing the status to 'Won't Fix' as there is no activity.

Changed in starlingx:
status: In Progress → Won't Fix
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.