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

user.job xattr is storing quotes around job name

Details

    • Bug
    • Resolution: Unresolved
    • Minor
    • None
    • Lustre 2.14.0, Lustre 2.16.1, Lustre 2.15.7
    • 3
    • 9223372036854775807

    Description

      bertschinger I noticed recently that the user.job xattr stored on disk is saving the " around the jobid:

      getfattr -d -m - /myth/tmp/foo 
      # file: myth/tmp/foo
      lustre.lov=0xd00bd70b0400000005da00003412000031323334
      trusted.link=0xdff1ea110100000036000000000000000000000000000000001e0000000200035f6200007fb000000000666f6f
      trusted.lma=0x0000000000000000315b0500020000000100000000000000
      trusted.lov=0xd00bd70b0400000005da00003412000031323334
      user.job=0x226d6f6f6b69653a6c66733a3022
      user.job="\"mookie:lfs:0\""
      

      The "0x22" at the start of the hex user.job show this as well as the ASCII version.

      It doesn't make sense to save the quotes into the xattr, since this makes the processing more complex since it doesn't match the actual jobid, and the printing is ugly (note escaped quotes while printing it).

      Attachments

        Issue Links

          Activity

            [LU-18798] user.job xattr is storing quotes around job name

            Hi Andreas – I would be happy to look into this. If I don't get to it in a week or so you can feel free to assign it to someone else but I'll try to get to it shortly.

            bertschinger Thomas Bertschinger added a comment - Hi Andreas – I would be happy to look into this. If I don't get to it in a week or so you can feel free to assign it to someone else but I'll try to get to it shortly.

            bertschinger is this something you could submit a patch to fix, or should I see if someone else could work on it? I suspect it is fairly trivial to fix.

            adilger Andreas Dilger added a comment - bertschinger is this something you could submit a patch to fix, or should I see if someone else could work on it? I suspect it is fairly trivial to fix.

            People

              bertschinger Thomas Bertschinger
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated: