← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1706285] Re: FirewallDriver's defer_apply might need better exception handling

 

This bug has not been attended in several years. I'm closing it. Please,
feel free to reopen if needed.

** Changed in: neutron
       Status: New => Won't Fix

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

Title:
  FirewallDriver's defer_apply might need better exception handling

Status in neutron:
  Won't Fix

Bug description:
  The current code is:

      @contextlib.contextmanager
      def defer_apply(self):
          """Defer apply context."""
          self.filter_defer_apply_on()
          try:
              yield
          finally:
              self.filter_defer_apply_off()

  This means filter_defer_apply_off will be called even if there is an
  exception and the exception will be reraised.

  Assuming the SG rules were consistent before this contextmanager call,
  it might make more sense to just discard deferred SG programming.

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



References