Reviewed: https://review.opendev.org/c/openstack/neutron/+/943212
Committed:
https://opendev.org/openstack/neutron/commit/71c83f43934a9de219c450e993f81302abec682e
Submitter: "Zuul (22348)"
Branch: master
commit 71c83f43934a9de219c450e993f81302abec682e
Author: Brian Haley <haleyb....@gmail.com>
Date: Mon Mar 3 16:43:43 2025 -0500
Increase batch notifier interval in unit tests
Change batch notifier interval to 0.2 seconds to better
let the test enqueue events under that time interval, to
avoid issue seen in requirements repo changes.
Closes-bug: #2100806
Change-Id: I73f6f51e33c3289ddae0125cdad79e4d8e031d70
** 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/2100806
Title:
Intermittent test_batch_notifier failure
Status in neutron:
Fix Released
Bug description:
The cross-neutron-py12 job was intermittently failing in the
requirements repo:
https://review.opendev.org/c/openstack/requirements/+/942959
Below is the failure:
==============================
Failed 1 tests - output below:
==============================
neutron.tests.unit.notifiers.test_batch_notifier.TestBatchNotifier.test_queue_event_multiple_events_notify_method
-----------------------------------------------------------------------------------------------------------------
Captured traceback:
~~~~~~~~~~~~~~~~~~~
Traceback (most recent call last):
File
"/home/zuul/src/opendev.org/openstack/neutron/neutron/tests/base.py",
line 178, in func
return f(self, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^
File
"/home/zuul/src/opendev.org/openstack/neutron/neutron/tests/unit/notifiers/test_batch_notifier.py",
line 66, in test_queue_event_multiple_events_notify_method
self.assertEqual(2, c_mock.call_count)
File
"/home/zuul/src/opendev.org/openstack/neutron/.tox/py312/lib/python3.12/site-packages/testtools/testcase.py",
line 419, in assertEqual
self.assertThat(observed, matcher, message)
File
"/home/zuul/src/opendev.org/openstack/neutron/.tox/py312/lib/python3.12/site-packages/testtools/testcase.py",
line 509, in assertThat
raise mismatch_error
testtools.matchers._impl.MismatchError: 2 != 3
This test and code was recently updated to remove eventlet, and it was noted
that the test is timing-sensitive, so at the CI meeting we agreed to increase
the sleep in the test from zero (to just trigger a yield) to either 0.1 or 0.2
to let the consumer execute. This bug is to track that work.
To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/2100806/+subscriptions
--
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help : https://help.launchpad.net/ListHelp
