Qpid
  1. Qpid
  2. QPID-4136

HA sporadic failures in ha_tests

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.7
    • Fix Version/s: 0.18
    • Component/s: C++ Clustering
    • Labels:
      None

      Description

      ha_tests.py fails sporadically. 2 different types of failure have been observed:

      ---- failure 1
      ha_tests.RecoveryTests.test_queue_hold ............................................ fail
      Error during test: Traceback (most recent call last):
      File "/home/aconway/qpid/dbg/src/tests/python/commands/qpid-python-test", line 340, in run
      phase()
      File "/home/aconway/qpid/qpid/cpp/src/tests/ha_tests.py", line 901, in test_queue_hold
      cluster[1].assert_browse_backup("q1", [str(i) for i in range(100)+range(100,200)])
      File "/home/aconway/qpid/qpid/cpp/src/tests/ha_tests.py", line 158, in assert_browse_backup
      assert_browse_retry(bs, queue, expected, **kwargs)
      File "/home/aconway/qpid/qpid/cpp/src/tests/brokertest.py", line 466, in assert_browse_retry
      assert expect_contents == actual_contents, msg
      AssertionError: browse failed: ['0', '1', '2', '3', '4', '5', '6', '7', '8', '9', '10', '11', '12', '13', '14', '15', '16', '17', '18', '19', '20', '21', '22', '23', '24', '25', '26', '27', '28', '29', '30', '31', '32', '33', '34', '35', '36', '37', '38', '39', '40', '41', '42', '43', '44', '45', '46', '47', '48', '49', '50', '51', '52', '53', '54', '55', '56', '57', '58', '59', '60', '61', '62', '63', '64', '65', '66', '67', '68', '69', '70', '71', '72', '73', '74', '75', '76', '77', '78', '79', '80', '81', '82', '83', '84', '85', '86', '87', '88', '89', '90', '91', '92', '93', '94', '95', '96', '97', '98', '99', '100', '101', '102', '103', '104', '105', '106', '107', '108', '109', '110', '111', '112', '113', '114', '115', '116', '117', '118', '119', '120', '121', '122', '123', '124', '125', '126', '127', '128', '129', '130', '131', '132', '133', '134', '135', '136', '137', '138', '139', '140', '141', '142', '143', '144', '145', '146', '147', '148', '149', '150', '151', '152', '153', '154', '155', '156', '157', '158', '159', '160', '161', '162', '163', '164', '165', '166', '167', '168', '169', '170', '171', '172', '173', '174', '175', '176', '177', '178', '179', '180', '181', '182', '183', '184', '185', '186', '187', '188', '189', '190', '191', '192', '193', '194', '195', '196', '197', '198', '199'] != []

      ----- failure 2

      ha_tests.RecoveryTests.test_queue_hold .................................. fail
      Error during test: Traceback (most recent call last):
      File "/var/lib/jenkins/workspace/Qpid-cpp/qpid/cpp/src/tests/python/commands/qpid-python-test", line 340, in run
      phase()
      File "/var/lib/jenkins/workspace/Qpid-cpp/qpid/cpp/src/tests/ha_tests.py", line 891, in test_queue_hold
      assertSyncTimeout(s1)
      File "/var/lib/jenkins/workspace/Qpid-cpp/qpid/cpp/src/tests/ha_tests.py", line 875, in assertSyncTimeout
      self.fail("Expected Timeout exception")
      File "/usr/lib64/python2.6/unittest.py", line 316, in fail
      raise self.failureException, msg
      AssertionError: Expected Timeout exception

        Activity

        Hide
        Alan Conway added a comment -

        Fixed on trunk, request permission for 0.18 branch

        ------------------------------------------------------------------------
        r1361323 | aconway | 2012-07-13 14:33:32 -0400 (Fri, 13 Jul 2012) | 5 lines

        QPID-4136: HA sporadic failures in ha_tests

        Caused by a test bug, default timeout was too short.
        Fixed the test to set a high timeout and increased default timeout.

        Show
        Alan Conway added a comment - Fixed on trunk, request permission for 0.18 branch ------------------------------------------------------------------------ r1361323 | aconway | 2012-07-13 14:33:32 -0400 (Fri, 13 Jul 2012) | 5 lines QPID-4136 : HA sporadic failures in ha_tests Caused by a test bug, default timeout was too short. Fixed the test to set a high timeout and increased default timeout.
        Hide
        Justin Ross added a comment -

        Trivial change, approved for 0.18.

        Show
        Justin Ross added a comment - Trivial change, approved for 0.18.
        Hide
        Alan Conway added a comment -

        Merged to 0.18 r1362654

        Show
        Alan Conway added a comment - Merged to 0.18 r1362654

          People

          • Assignee:
            Alan Conway
            Reporter:
            Alan Conway
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development