Details

    • 9106

    Description

      Make testing more robust by extracting changelog record fields by name where applicable instead of using their position in the list.

      Attachments

        Activity

          [LU-3589] Extract changelog fields by name
          pjones Peter Jones added a comment -

          Landed for 2.4.1 and 2.5. This will be included in a 2.1.7 if need be.

          pjones Peter Jones added a comment - Landed for 2.4.1 and 2.5. This will be included in a 2.1.7 if need be.

          Reducing from blocker as the patch has landed to master. Will close when other patches have landed.

          jlevi Jodi Levi (Inactive) added a comment - Reducing from blocker as the patch has landed to master. Will close when other patches have landed.

          Uploaded new patchsets that perform the operation via a dedicated function. Abandoned and re-opened gerrits for b2_4 and b2_1 as these were accidentally sharing the same change-id as the one against master.

          master is still at: http://review.whamcloud.com/#/c/6998/
          b2_4: http://review.whamcloud.com/#/c/7020/
          b2_1: http://review.whamcloud.com/#/c/7021/

          hdoreau Henri Doreau (Inactive) added a comment - Uploaded new patchsets that perform the operation via a dedicated function. Abandoned and re-opened gerrits for b2_4 and b2_1 as these were accidentally sharing the same change-id as the one against master. master is still at: http://review.whamcloud.com/#/c/6998/ b2_4: http://review.whamcloud.com/#/c/7020/ b2_1: http://review.whamcloud.com/#/c/7021/
          hdoreau Henri Doreau (Inactive) added a comment - Patch against b2_1 is at http://review.whamcloud.com/#/c/6999/
          hdoreau Henri Doreau (Inactive) added a comment - Patch against b2_4 is at http://review.whamcloud.com/#/c/7000/
          hdoreau Henri Doreau (Inactive) added a comment - Patch against master is at http://review.whamcloud.com/#/c/6998/

          As advised by Andreas on http://review.whamcloud.com/#/c/4060/14/lustre/tests/sanity.sh
          We will provide a patch.

          hdoreau Henri Doreau (Inactive) added a comment - As advised by Andreas on http://review.whamcloud.com/#/c/4060/14/lustre/tests/sanity.sh We will provide a patch.

          People

            jamesanunez James Nunez (Inactive)
            hdoreau Henri Doreau (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: