test_dualnet_multi_prefix_dhcpv6_stateless failed due to "RuntimeError: Set changed size during iteration" in quota code

Bug #1485969 reported by Ihar Hrachyshka
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Ihar Hrachyshka

Bug Description

The failure is: http://logs.openstack.org/13/213913/3/check/gate-tempest-dsvm-neutron-full/1d73a46/logs/testr_results.html.gz

Nova failed to build an instance because neutron crashed with InternalError. In neutron-server.log, we see:

2015-08-18 05:08:05.737 ERROR neutron.api.v2.resource [req-2fb6728e-f7f0-492c-bdf6-0b76d553932b neutron service] create failed
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource Traceback (most recent call last):
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource File "/opt/stack/new/neutron/neutron/api/v2/resource.py", line 83, in resource
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource result = method(request=request, **args)
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource File "/usr/local/lib/python2.7/dist-packages/oslo_db/api.py", line 146, in wrapper
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource ectxt.value = e.inner_exc
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource File "/usr/local/lib/python2.7/dist-packages/oslo_utils/excutils.py", line 119, in __exit__
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource six.reraise(self.type_, self.value, self.tb)
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource File "/usr/local/lib/python2.7/dist-packages/oslo_db/api.py", line 136, in wrapper
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource return f(*args, **kwargs)
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource File "/opt/stack/new/neutron/neutron/api/v2/base.py", line 495, in create
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource obj)})
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource File "/opt/stack/new/neutron/neutron/api/v2/base.py", line 459, in notify
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource resource_registry.set_resources_dirty(request.context)
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource File "/opt/stack/new/neutron/neutron/quota/resource_registry.py", line 70, in set_resources_dirty
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource res.mark_dirty(context, nested=True)
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource File "/usr/local/lib/python2.7/dist-packages/oslo_concurrency/lockutils.py", line 252, in inner
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource return f(*args, **kwargs)
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource File "/opt/stack/new/neutron/neutron/quota/resource.py", line 185, in mark_dirty
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource for tenant_id in self._dirty_tenants:
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource RuntimeError: Set changed size during iteration
2015-08-18 05:08:05.737 26601 ERROR neutron.api.v2.resource

Apparently, we should make sure the loop does not operate on a set that is changing under the hood.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/214133

Changed in neutron:
assignee: nobody → Ihar Hrachyshka (ihar-hrachyshka)
status: New → In Progress
Revision history for this message
Matt Riedemann (mriedem) wrote :

Yeah I'm seeing this elsewhere, quite a bit in the last couple of days, but since logstash has been basically MIA for a week I can't tell how widespread this is.

Revision history for this message
Matt Riedemann (mriedem) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

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

commit 852752edad15e13ab08aee67ea54a1fd2e829942
Author: Ihar Hrachyshka <email address hidden>
Date: Tue Aug 18 12:35:15 2015 +0200

    quota: synchronize resync and count with other dirty_tenants code

    We should synchronize every access or modification of
    self._dirty_tenants or self._out_of_sync_tenants.

    Closes-Bug: #1485969
    Change-Id: If17f57e8905fd8d13438d0421f73468e77f723d9

Changed in neutron:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (feature/pecan)

Fix proposed to branch: feature/pecan
Review: https://review.openstack.org/218710

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (feature/pecan)
Download full text (155.6 KiB)

Reviewed: https://review.openstack.org/218710
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=2c5f44e1b3bd4ed8a0b7232fd293b576cc8c1c87
Submitter: Jenkins
Branch: feature/pecan

commit f35d1c5c50dccbef1a2e079f967b82f0df0e22e9
Author: Adelina Tuvenie <email address hidden>
Date: Thu Aug 27 02:27:28 2015 -0700

    Fixes wrong neutron Hyper-V Agent name in constants

    Change Id03fb147e11541be309c1cd22ce27e70fadc28b5 moved the
    AGENT_TYPE_HYPERV constant from common.constants to
    plugins.ml2.drivers.hyperv.constants but change the value of the
    constant from 'HyperV agent' to 'hyperv'. This patch changes
    the name back to 'HyperV agent'

    Change-Id: If74b4b2a84811e266c8b12e70bf6bfe74ed4ea21
    Partial-Bug: #1487598

commit de604de334854e2eb6b4312ff57920564cbd4459
Author: OpenStack Proposal Bot <email address hidden>
Date: Sun Aug 30 01:39:06 2015 +0000

    Updated from global requirements

    Change-Id: Ie52aa3b59784722806726e4046bd07f4a4d97328

commit f0415ac20eaf5ab4abb9bd4839bf6d04ceee85d0
Author: armando-migliaccio <email address hidden>
Date: Fri Aug 28 13:53:04 2015 -0700

    Revert "Add support for unaddressed port"

    This implementation may expose a vulnerability where a malicious
    user can sieze the opportunity of a time window where a port
    may land unaddressed on a shared network, thus allowing him/her
    to suck up all the tenant traffic he/she wants....oh the shivers.

    This reverts commit d4c52b7f5a36a103a92bf9dcda7f371959112292.

    Change-Id: I7ebdaa8d3defa80eab90e460fde541a5bdd8864c

commit 013fdcd2a6d45dbe4de5d6e7077e5e9b60985ef9
Author: Assaf Muller <email address hidden>
Date: Fri Aug 28 16:41:07 2015 -0400

    Improve logging upon failure in iptables functional tests

    This will help us nail down a more accurate and efficient logstash
    query.

    Change-Id: Iee4238e358f7b056e373c7be8d6aa3202117a680
    Related-Bug: #1478847

commit 622dea818d851224a43d5276a81d5ce8a6eebb76
Author: Ivar Lazzaro <email address hidden>
Date: Mon Aug 17 17:17:42 2015 -0700

    handle gw_info outside of the db transaction on router creation

    Move the gateway interface creation outside the DB transaction
    to avoid lock timeout.

    Change-Id: I5a78d7f32e8ca912016978105221d5f34618af19
    Closes-bug: 1485809

commit 5b27d290a0a95f6247fc5a0fe6da1e7d905e6b2d
Author: Assaf Muller <email address hidden>
Date: Wed Aug 26 10:07:03 2015 -0400

    Remove ml2 resource extension success logging

    This is the cause of a tremendous amount of logs, for no
    perceivable gain. A normal dvr run in the gate shows this debug
    message around 120K times, which is way too much.

    Closes-Bug: #1489952

    Change-Id: I26fca8515d866a7cc1638d07fa33bc04479ae221

commit 8d3faf549cba2f58c872ef4121b2481e73464010
Author: huangpengtao <email address hidden>
Date: Fri Aug 28 23:20:46 2015 +0800

    Replace "prt" variable by "port"

    the local variable prt is meaningless,
    and port is used popular.

    Change-Id: I20849102cf5b4d84433c46791b4b1e2a22dc4739

commit ee374e7a5f4dea538fcd942f5...

tags: added: in-feature-pecan
Thierry Carrez (ttx)
Changed in neutron:
milestone: none → liberty-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: liberty-3 → 7.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.