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

ACLs aren't nodemapped properly in mdt_finish_open

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • Lustre 2.7.0
    • Lustre 2.7.0
    • 3
    • 16796

    Description

      mdt_finish_open's ACL handling was overlooked in the original implementation, but it also needs to map the ACLs before sending them to the client. The ACL code in mdt_finish_open is identical to the non-nodemapped version of mdt_getattr_internal, so I created a patch that just copies the nodemapped version over. Would it be worthwhile to split this out into its own function?

      Attachments

        Activity

          [LU-6024] ACLs aren't nodemapped properly in mdt_finish_open

          Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13062/
          Subject: LU-6024 nodemap: fix ACL handling in mdt_finish_open
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 228946bf41edeb39f5c992ba0781340d5428aa9c

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13062/ Subject: LU-6024 nodemap: fix ACL handling in mdt_finish_open Project: fs/lustre-release Branch: master Current Patch Set: Commit: 228946bf41edeb39f5c992ba0781340d5428aa9c

          Nathaniel,
          Can you please keep an eye on this patch?
          Thank you!

          jlevi Jodi Levi (Inactive) added a comment - Nathaniel, Can you please keep an eye on this patch? Thank you!

          Kit Westneat (kit.westneat@gmail.com) uploaded a new patch: http://review.whamcloud.com/13062
          Subject: LU-6024 nodemap: fix ACL handling in mdt_finish_open
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 87447355537386ea5e69edd1c70e1d2d600a1986

          gerrit Gerrit Updater added a comment - Kit Westneat (kit.westneat@gmail.com) uploaded a new patch: http://review.whamcloud.com/13062 Subject: LU-6024 nodemap: fix ACL handling in mdt_finish_open Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 87447355537386ea5e69edd1c70e1d2d600a1986

          People

            utopiabound Nathaniel Clark
            kit.westneat Kit Westneat (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: