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_tests.patch
        99 kB
        Arun C Murthy
      2. cs_test.patch
        264 kB
        Arun C Murthy

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        4d 21h 18m 1 Arun C Murthy 10/Aug/11 07:04
        Patch Available Patch Available Resolved Resolved
        17s 1 Arun C Murthy 10/Aug/11 07:05
        Resolved Resolved Closed Closed
        96d 18h 44m 1 Arun C Murthy 15/Nov/11 00:49
        Arun C Murthy made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Arun C Murthy made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        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.
        Arun C Murthy made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Arun C Murthy made changes -
        Attachment cs_tests.patch [ 12489942 ]
        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 -

        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
        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.
        Arun C Murthy made changes -
        Field Original Value New Value
        Attachment cs_test.patch [ 12489448 ]
        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.
        Arun C Murthy created issue -

          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