← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1714389] Re: Pecan is missing some quota dirtying and resync calls

 

Reviewed:  https://review.openstack.org/499434
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=e76a575b0570880d4209ff1ea73f82c01e4a6e24
Submitter: Jenkins
Branch:    master

commit e76a575b0570880d4209ff1ea73f82c01e4a6e24
Author: Kevin Benton <kevin@xxxxxxxxxx>
Date:   Wed Aug 30 20:16:59 2017 -0700

    Pecan: add calls to resync/dirty quota usages
    
    Add calls to dirty and resync the quota engine during
    DELETEs and GETs respectively. This matches the old API controller
    behavior and will be covered when the unit tests are switched to
    use pecan in [1].
    
    1. I76dc23fb7b96d82b0da50285bd0aac76142e81e5
    
    Closes-Bug: #1714389
    Change-Id: I1fb6b471e0e7cbf9b10dd14233931be94af38a41


** Changed in: neutron
       Status: In Progress => Fix Released

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

Title:
  Pecan is missing some quota dirtying and resync calls

Status in neutron:
  Fix Released

Bug description:
  The legacy API controller would resync and dirty the quotas at
  particular intervals that pecan is missing. In particular it would
  resync on GET operations[1] and dirty on deletes[2]. The pecan hook is
  missing both of these cases.

  
  1. https://github.com/openstack/neutron/blob/8d2c1bd88b14eefbea74c72f384cb9952e7ee62e/neutron/api/v2/base.py#L338-L339
  2. https://github.com/openstack/neutron/blob/8d2c1bd88b14eefbea74c72f384cb9952e7ee62e/neutron/api/v2/base.py#L589-L591

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


References