yahoo-eng-team team mailing list archive

[OpenStack Infra <1792472@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/602694
Committed: https://git.openstack.org/cgit/openstack/neutron-tempest-plugin/commit/?id=af347da20f17dbbafb59bc1c0bc39101e9a612e1
Submitter: Zuul
Branch:    master

commit af347da20f17dbbafb59bc1c0bc39101e9a612e1
Author: Brian Haley <bhaley@xxxxxxxxxx>
Date:   Fri Sep 14 11:24:00 2018 -0600

    Wait for floating IP to be DOWN before checking port details
    
    Occasionally test_floatingip_port_details() fails, showing
    the floating IP attached port status as 'BUILD'.  Wait for the
    status to transition to 'DOWN' before checking other port details,
    similar as is done to the fixed IP port's device_id value.
    
    Changed afffected method to use constants for port status as well.
    
    Change-Id: I862115b37c97d5a3cad082d8cd4e7b855ef3f88b
    Closes-bug: #1792472


** 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/1792472

Title:
  neutron_tempest_plugin: test_floatingip_port_details occasionally
  fails

Status in neutron:
  Fix Released

Bug description:
  neutron_tempest_plugin.scenario.test_floatingip.FloatingIPPortDetailsTest.test_floatingip_port_details
  occasionally fails in the check queue.  An example:

  http://logs.openstack.org/97/602097/2/check/neutron-tempest-plugin-
  scenario-linuxbridge/849d93e/testr_results.html.gz

  Traceback (most recent call last):
    File "/opt/stack/neutron-tempest-plugin/neutron_tempest_plugin/scenario/test_floatingip.py", line 249, in test_floatingip_port_details
      fip, port, status='DOWN', device_id='', device_owner='')
    File "/opt/stack/neutron-tempest-plugin/neutron_tempest_plugin/scenario/test_floatingip.py", line 259, in _check_port_details
      self.assertEqual(status, port_details['status'])
    File "/opt/stack/tempest/.tox/tempest/local/lib/python2.7/site-packages/testtools/testcase.py", line 411, in assertEqual
      self.assertThat(observed, matcher, message)
    File "/opt/stack/tempest/.tox/tempest/local/lib/python2.7/site-packages/testtools/testcase.py", line 498, in assertThat
      raise mismatch_error
  testtools.matchers._impl.MismatchError: 'DOWN' != u'BUILD'

  The strange thing is that BUILD is not a valid state for a floating
  IP, only ACTIVE, DOWN and ERROR are.

  I believe this is due to this test using the compute API to get the
  floating IP state, it should instead use the neutron API if possible,
  for example get_floatingip(), so that a valid object is returned.
  I'll investigate changing the code.

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