Public bug reported:

In test_ha_router_failover function, w have the below test conditions


        utils.wait_until_true(lambda: router1.ha_state == 'master')
        utils.wait_until_true(lambda: router2.ha_state == 'backup')

        self.fail_ha_router(router1)

        utils.wait_until_true(lambda: router2.ha_state == 'master')
        utils.wait_until_true(lambda: router1.ha_state != 'backup')


I think the last test condition is incorrect and it should be

        utils.wait_until_true(lambda: router1.ha_state == 'backup')

instead of

        utils.wait_until_true(lambda: router1.ha_state != 'backup')

** Affects: neutron
     Importance: Undecided
         Status: New


** Tags: l3-ha low-hanging-fruit

** Tags added: low-hanging-fruit

** Tags added: l3-ha

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

Title:
  Invalid test condition in test_ha_router_failover

Status in neutron:
  New

Bug description:
  In test_ha_router_failover function, w have the below test conditions

  
          utils.wait_until_true(lambda: router1.ha_state == 'master')
          utils.wait_until_true(lambda: router2.ha_state == 'backup')

          self.fail_ha_router(router1)

          utils.wait_until_true(lambda: router2.ha_state == 'master')
          utils.wait_until_true(lambda: router1.ha_state != 'backup')

  
  I think the last test condition is incorrect and it should be

          utils.wait_until_true(lambda: router1.ha_state == 'backup')

  instead of

          utils.wait_until_true(lambda: router1.ha_state != 'backup')

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1476714/+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

Reply via email to