Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-7627

Lustre sanity tests 130b, 130c, 130d, and 130e have a faulty comparison that causes them to fail

Details

    • Bug
    • Resolution: Fixed
    • Major
    • Lustre 2.9.0
    • Lustre 2.8.0
    • 3
    • 9223372036854775807

    Description

      Test 130c-130e failures are all the same coding issue, where a comparison is done on two hex numbers without prefacing them with '0x' so that they are treated as hex:
      test 130b:
      7178 if (( $frag_lun != $last_lun )); then
      should be
      7178 if (( 0x$frag_lun != 0x$last_lun )); then
      test 130c:
      7227 if (( $frag_lun != $last_lun )); then
      should be:
      7227 if (( 0x$frag_lun != 0x$last_lun )); then
      test 130d:
      7282 if (( $frag_lun != $last_lun )); then
      should be
      7282 if (( 0x$frag_lun != 0x$last_lun )); then
      test 130e:
      7336 if (( $frag_lun != $last_lun )); then
      should be
      7336 if (( 0x$frag_lun != 0x$last_lun )); then

      Tests 130[bcde] failed because numbers parsed with leading zeros
      treated as octal number so there is error value too great for base
      when parsed lun id greater than 0007 value,

      Attachments

        Issue Links

          Activity

            People

              jamesanunez James Nunez (Inactive)
              jadhav.vikram VIKRAM BABASO JADHAV (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: