yahoo-eng-team team mailing list archive

[OpenStack Infra <2100806@xxxxxxxxxxxxxxxxxx>]

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.dev@xxxxxxxxx>
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