Uploaded image for project: 'Drivers'
  1. Drivers
  2. DRIVERS-2834

Revise runOnRequirements for transaction tests incompatible with load balancers

XMLWordPrintableJSON

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Unknown Unknown
    • None
    • Component/s: Transactions
    • Labels:
      None
    • Needed
    • Hide

      Commits for syncing spec/prose tests

      Context for other referenced/linked tickets

      • This ticket should be completed alongside DRIVERS-1709, which ported the legacy transaction spec tests to the unified format.
      Show
      Commits for syncing spec/prose tests Sync unified transactions spec tests with mongodb/specifications@a7696bc Context for other referenced/linked tickets This ticket should be completed alongside DRIVERS-1709, which ported the legacy transaction spec tests to the unified format.
    • $i18n.getText("admin.common.words.hide")
      Key Status/Resolution FixVersion
      CDRIVER-4836 Backlog
      CXX-2831 Backlog
      CSHARP-4966 Backlog
      GODRIVER-3130 Backlog
      JAVA-5326 Done
      NODE-5927 Gone away
      MOTOR-1256 Duplicate
      PYTHON-4214 Backlog
      PHPLIB-1397 Fixed 1.18.0
      RUBY-3398 Backlog
      RUST-1848 Fixed 3.0.0
      $i18n.getText("admin.common.words.show")
      #scriptField, #scriptField *{ border: 1px solid black; } #scriptField{ border-collapse: collapse; } #scriptField td { text-align: center; /* Center-align text in table cells */ } #scriptField td.key { text-align: left; /* Left-align text in the Key column */ } #scriptField a { text-decoration: none; /* Remove underlines from links */ border: none; /* Remove border from links */ } /* Add green background color to cells with FixVersion */ #scriptField td.hasFixVersion { background-color: #00FF00; /* Green color code */ } /* Center-align the first row headers */ #scriptField th { text-align: center; } Key Status/Resolution FixVersion CDRIVER-4836 Backlog CXX-2831 Backlog CSHARP-4966 Backlog GODRIVER-3130 Backlog JAVA-5326 Done NODE-5927 Gone away MOTOR-1256 Duplicate PYTHON-4214 Backlog PHPLIB-1397 Fixed 1.18.0 RUBY-3398 Backlog RUST-1848 Fixed 3.0.0

      Summary

      DRIVERS-2713 added "load-balanced" to the runOnRequirements of legacy transaction tests. These tests were later ported to the unified format in DRIVERS-1709. In that issue's PR, jeff.yemin@mongodb.com noted that some tests are incompatible with LB topologies due to their use of targetedFailPoint operations (see: this comment in mongodb/specifications#1502).

      Those tests should not be permitted to run on LB topologies. This ticket has been opened to track that modification, since the legacy transaction tests modified by DRIVERS-2713 no longer exist and there is nothing to revert.

      As part of this ticket, we may consider adding LB topologies to runOnRequirements for withTransaction spec tests, which were also ported to the unified format in DRIVERS-1709. The withTransaction spec tests were inadvertently never considered in DRIVERS-2713, which likely works to our advantage as it did not introduce possible test failures; however, that does mean some withTransaction could be run on LB and serverless environments.

      Motivation

      Is this issue urgent?

      Yes. Most drivers have not handled downstream changes for DRIVERS-2713, and will have no reason to do so once DRIVERS-1709 is merged and all transaction tests are in the unified format.

      Is this ticket required by a downstream team?

      No.

      Is this ticket only for tests?

      Yes.

            Assignee:
            jmikola@mongodb.com Jeremy Mikola
            Reporter:
            jmikola@mongodb.com Jeremy Mikola
            Jeffrey Yemin Jeffrey Yemin
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: