Uploaded image for project: 'C Driver'
  1. C Driver
  2. CDRIVER-3334

Test with Cyrus SASL on Windows in evergreen

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.17.0-beta, 1.17.0
    • Component/s: auth, libmongoc, tests
    • Labels:
      None

      Description

      There are two types of SASL tasks for Windows:

      debug-compile-sasl-xyz
      debug-compile-sspi-xyz

      However, the "sasl" tasks default to building with SSPI (not Cyrus). Based on our cmake logic, defaulting to SSPI seems intended when ENABLE_SASL=AUTO on Windows. But this means we aren't testing Cyrus SASL on evergreen. Let's consider making these changes:

      • Rename those "debug-compile-sasl-xyz" tasks to be "debug-compile-cyrus-xyz" and have them build with ENABLE_SASL=Cyrus, and rename the corresponding tests tasks. This way, we're not relying on the value of AUTO. This would be a rename for tasks on all variants, not just Windows.
      • Perhaps leave one "debug-compile-sasl-xyz" to check that configuring with ENABLE_SASL=AUTO works (though tests need not use it) since it'll default to either Cyrus or SSPI, both of which should be tested independantly.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              kevin.albertson Kevin Albertson
              Reporter:
              kevin.albertson Kevin Albertson
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: