Qpid
  1. Qpid
  2. QPID-4159

HA missing messages in failover test.

    Details

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

      Description

      Running this test:

      ha_tests.py *.test_failover_send_receive -DDURATION=2

      in a loop eventually fails with something like:

      AssertionError: test8 missing message 17529>17454

      The test should run indefinitely with no missing messages.

        Activity

        Hide
        Alan Conway added a comment -

        Fix for review: https://reviews.apache.org/r/6093/

        Request merge to 0.18

        Show
        Alan Conway added a comment - Fix for review: https://reviews.apache.org/r/6093/ Request merge to 0.18
        Hide
        Alan Conway added a comment -

        Comitted to trunk 1364804

        Show
        Alan Conway added a comment - Comitted to trunk 1364804
        Hide
        Gordon Sim added a comment -

        Change poses no risk for 0.18 if there is still time to squeeze it in. It is a clear defect so would be very nice to see it fixed in this release.

        Show
        Gordon Sim added a comment - Change poses no risk for 0.18 if there is still time to squeeze it in. It is a clear defect so would be very nice to see it fixed in this release.
        Hide
        Justin Ross added a comment -

        Reviewed by Gordon. Approved for 0.18.

        Show
        Justin Ross added a comment - Reviewed by Gordon. Approved for 0.18.
        Hide
        Alan Conway added a comment -

        Committed to 0.18 branch r1365044

        Show
        Alan Conway added a comment - Committed to 0.18 branch r1365044
        Hide
        Alan Conway added a comment -

        The failures are still occuring, but less frequently. Reopening.

        Show
        Alan Conway added a comment - The failures are still occuring, but less frequently. Reopening.
        Hide
        Alan Conway added a comment -

        Additional patch to finally fix the issue, review at https://reviews.apache.org/r/6153/
        Request 0.18 merge.

        Show
        Alan Conway added a comment - Additional patch to finally fix the issue, review at https://reviews.apache.org/r/6153/ Request 0.18 merge.
        Hide
        Gordon Sim added a comment -

        Same comment as above: Change poses no risk for 0.18 if there is still time to squeeze it in. It is a clear defect so would be very nice to see it fixed in this release.

        Show
        Gordon Sim added a comment - Same comment as above: Change poses no risk for 0.18 if there is still time to squeeze it in. It is a clear defect so would be very nice to see it fixed in this release.
        Hide
        Justin Ross added a comment -

        Further change is approved for 0.18.

        Show
        Justin Ross added a comment - Further change is approved for 0.18.
        Hide
        Alan Conway added a comment - - edited

        Committed to trunk r1366179 and 0.18 branch r1366206

        Show
        Alan Conway added a comment - - edited Committed to trunk r1366179 and 0.18 branch r1366206

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development