yahoo-eng-team team mailing list archive

[SamP <sam47priya@xxxxxxxxx>]

Public bug reported:

In following fix,
https://review.openstack.org/#/c/259406/
markus_z marked these codes need to remove in separate patch.

In nova/nova/conf/paths.py,
# TODO(markus_z): This needs to be removed in a new patch. No one uses this.
def basedir_rel(*args):
   :omitted for simplicity
# TODO(markus_z): This needs to be removed in a new patch. No one uses this.
def bindir_rel(*args):
   :omitted for simplicity
# TODO(markus_z): This needs to be removed in a new patch. No one uses this.
def state_path_rel(*args):
   :omitted for simplicity

No one using them and codes are still there.
IMHO, need to remove them.

** Affects: nova
     Importance: Undecided
     Assignee: SamP (sampath-priyankara)
         Status: New

** Changed in: nova
     Assignee: (unassigned) => SamP (sampath-priyankara)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1598655

Title:
  Need to remove unused codes in paths.py

Status in OpenStack Compute (nova):
  New

Bug description:
  In following fix,
  https://review.openstack.org/#/c/259406/
  markus_z marked these codes need to remove in separate patch.

  In nova/nova/conf/paths.py,
  # TODO(markus_z): This needs to be removed in a new patch. No one uses this.
  def basedir_rel(*args):
     :omitted for simplicity
  # TODO(markus_z): This needs to be removed in a new patch. No one uses this.
  def bindir_rel(*args):
     :omitted for simplicity
  # TODO(markus_z): This needs to be removed in a new patch. No one uses this.
  def state_path_rel(*args):
     :omitted for simplicity

  No one using them and codes are still there.
  IMHO, need to remove them.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1598655/+subscriptions