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

jobstats are not expired if no new jobs arrive

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.8.0
    • Lustre 2.6.0, Lustre 2.5.3
    • None
    • 3
    • 16603

    Description

      The jobstats data is not expired for a device if new jobids are not arriving at the server. This means that the stats may stick around far longer than necessary. They should be expired when they get too old.

      Attachments

        Activity

          [LU-5946] jobstats are not expired if no new jobs arrive
          pjones Peter Jones added a comment -

          Landed for 2.8

          pjones Peter Jones added a comment - Landed for 2.8

          Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/12821/
          Subject: LU-5946 lprocfs: free expired jobstats after /proc read
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: dd99ddf9f22921535c3f083e91bfc06a566e382b

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/12821/ Subject: LU-5946 lprocfs: free expired jobstats after /proc read Project: fs/lustre-release Branch: master Current Patch Set: Commit: dd99ddf9f22921535c3f083e91bfc06a566e382b
          pjones Peter Jones added a comment -

          Right As per Andreas, the patch that landed was cleanup and the remaining patch is the primary bug.

          pjones Peter Jones added a comment - Right As per Andreas, the patch that landed was cleanup and the remaining patch is the primary bug.

          Peter, the patch has been landed to master, what else would you like me to do?

          Ah, there is sill one patch not landed, I'll work on that one.

          niu Niu Yawei (Inactive) added a comment - Peter, the patch has been landed to master, what else would you like me to do? Ah, there is sill one patch not landed, I'll work on that one.

          Peter, the patch has been landed to master, what else would you like me to do?

          niu Niu Yawei (Inactive) added a comment - Peter, the patch has been landed to master, what else would you like me to do?
          pjones Peter Jones added a comment -

          Niu

          Could you please complete the work Andreas has started on this issue?

          Thanks

          Peter

          pjones Peter Jones added a comment - Niu Could you please complete the work Andreas has started on this issue? Thanks Peter

          Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/12872/
          Subject: LU-5946 lprocfs: cleanup stats locking code
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: c3e8e9d37fa6ea74d10b850a6e02af9fc477e719

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/12872/ Subject: LU-5946 lprocfs: cleanup stats locking code Project: fs/lustre-release Branch: master Current Patch Set: Commit: c3e8e9d37fa6ea74d10b850a6e02af9fc477e719

          Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: http://review.whamcloud.com/12872
          Subject: LU-5946 lprocfs: cleanup stats locking code
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 91e7986a81b0643ffb363d5c290ed06da12e2ea7

          gerrit Gerrit Updater added a comment - Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: http://review.whamcloud.com/12872 Subject: LU-5946 lprocfs: cleanup stats locking code Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 91e7986a81b0643ffb363d5c290ed06da12e2ea7

          Completely untested, but probably reasonably close patch at http://review.whamcloud.com/12821

          adilger Andreas Dilger added a comment - Completely untested, but probably reasonably close patch at http://review.whamcloud.com/12821

          People

            niu Niu Yawei (Inactive)
            adilger Andreas Dilger
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: