Details

    • Type: Test Test
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.23.0
    • Component/s: mrv2
    • Labels:
      None

      Description

      Add (true) unit tests for CapacityScheduler

      1. cs_test.patch
        264 kB
        Arun C Murthy
      2. cs_tests.patch
        99 kB
        Arun C Murthy

        Activity

        Hide
        Arun C Murthy added a comment -

        Bunch of unit tests for CS covering user limits, queue limits, multiple applications per queue, reservations etc.

        I'll add more soon for scheduling across queue hierarchies, priorities etc.

        Show
        Arun C Murthy added a comment - Bunch of unit tests for CS covering user limits, queue limits, multiple applications per queue, reservations etc. I'll add more soon for scheduling across queue hierarchies, priorities etc.
        Hide
        Mahadev konar added a comment -

        arun,
        I think you generated the diff across an older version of MR-279. There's lot of accidental changes regarding securityinfo (in common) in your patch.

        Show
        Mahadev konar added a comment - arun, I think you generated the diff across an older version of MR-279. There's lot of accidental changes regarding securityinfo (in common) in your patch.
        Hide
        Arun C Murthy added a comment -

        Teaches me to stop working before it's too late. I'll fix it as part of the next patch which has more tests. Thanks.

        Show
        Arun C Murthy added a comment - Teaches me to stop working before it's too late. I'll fix it as part of the next patch which has more tests. Thanks.
        Hide
        Arun C Murthy added a comment -

        Updated patch, completes unit tests for CS for all runtime resource limits, reservations for high-ram apps etc.

        Show
        Arun C Murthy added a comment - Updated patch, completes unit tests for CS for all runtime resource limits, reservations for high-ram apps etc.
        Hide
        Arun C Murthy added a comment -

        I just committed this since it's only tests.

        Show
        Arun C Murthy added a comment - I just committed this since it's only tests.

          People

          • Assignee:
            Arun C Murthy
            Reporter:
            Arun C Murthy
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development