yahoo-eng-team team mailing list archive

[Salvatore Orlando <1449462@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

According to the docstring at [1] I can specify read_delete=yes or only to see deleted records when performing queries.
However, Neutron does not perform soft deletes.
Also I kind of know a little Neutron's DB management layer and I'm pretty sure it never uses read_deleted anywhere.
As far as I remember no plugin makes use of it either.
According to git history this was added with an initial commit for Neutron context [2], which was probably more or less a cut & paste from nova.

It is worth removing that parameter before somebody actually and tries
to set it to 'yes' or 'only'

[1] http://git.openstack.org/cgit/openstack/neutron/tree/neutron/context.py#n44
[2] https://review.openstack.org/#/c/7952/

** Affects: neutron
     Importance: Low
     Assignee: Salvatore Orlando (salvatore-orlando)
         Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1449462

Title:
  read_deleted in neutron context serves no purpose

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  According to the docstring at [1] I can specify read_delete=yes or only to see deleted records when performing queries.
  However, Neutron does not perform soft deletes.
  Also I kind of know a little Neutron's DB management layer and I'm pretty sure it never uses read_deleted anywhere.
  As far as I remember no plugin makes use of it either.
  According to git history this was added with an initial commit for Neutron context [2], which was probably more or less a cut & paste from nova.

  It is worth removing that parameter before somebody actually and tries
  to set it to 'yes' or 'only'

  [1] http://git.openstack.org/cgit/openstack/neutron/tree/neutron/context.py#n44
  [2] https://review.openstack.org/#/c/7952/

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