yahoo-eng-team team mailing list archive

[zhanghao <1859258@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

The mtu is already non-nullable, so the writer context can be switched
to a reader context.

like this can be removed, and change it.
# NOTE(ihrachys) use writer manager to be able to update mtu
# TODO(ihrachys) remove in Queens when mtu is not nullable
with db_api.CONTEXT_WRITER.using(context):

In addition, redundant reader context can be removed.
def _get_subnet
    # TODO(ihrachys) remove explicit reader usage when subnet OVO switches
    # to engine facade by default
    with db_api.CONTEXT_READER.using(context):

** Affects: neutron
     Importance: Undecided
         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/1859258

Title:
  switch to reader context

Status in neutron:
  New

Bug description:
  The mtu is already non-nullable, so the writer context can be switched
  to a reader context.

  like this can be removed, and change it.
  # NOTE(ihrachys) use writer manager to be able to update mtu
  # TODO(ihrachys) remove in Queens when mtu is not nullable
  with db_api.CONTEXT_WRITER.using(context):

  In addition, redundant reader context can be removed.
  def _get_subnet
      # TODO(ihrachys) remove explicit reader usage when subnet OVO switches
      # to engine facade by default
      with db_api.CONTEXT_READER.using(context):

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