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

Eliminate acl warning for acl mount option in ll_options

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • Lustre 2.7.0
    • Lustre 2.6.0
    • None
    • 3
    • 11469

    Description

      The ll_options has a version check that emits a warning once we are post 2.5.51 (or some later version after I change it).
      Since I have kicking this can down the road we need to eliminate this code.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-2369 'noacl' mount option doesn't take effect

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Activity

            [LU-4217] Eliminate acl warning for acl mount option in ll_options
            adilger Andreas Dilger made changes -
            Link New: This issue is related to LU-2369 [ LU-2369 ]
            jamesanunez James Nunez (Inactive) made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: Reopened [ 4 ] New: Resolved [ 5 ]
            jamesanunez James Nunez (Inactive) added a comment -

            Landed to master (pre-2.7)

            jamesanunez James Nunez (Inactive) added a comment - Landed to master (pre-2.7)
            jamesanunez James Nunez (Inactive) made changes -
            Assignee Original: WC Triage [ wc-triage ] New: James Nunez [ jamesanunez ]
            jamesanunez James Nunez (Inactive) made changes -
            Assignee Original: James Nunez [ jamesanunez ] New: WC Triage [ wc-triage ]
            jamesanunez James Nunez (Inactive) made changes -
            Assignee Original: WC Triage [ wc-triage ] New: James Nunez [ jamesanunez ]
            jamesanunez James Nunez (Inactive) added a comment -

            Patch to remove "acl" mount option at http://review.whamcloud.com/12154

            jamesanunez James Nunez (Inactive) added a comment - Patch to remove "acl" mount option at http://review.whamcloud.com/12154
            adilger Andreas Dilger added a comment -

            I pushed http://review.whamcloud.com/8630 to bump the build versions further out. It finally appears that the http://review.whamcloud.com/9251 patch from TEI-974 has been deployed, so the "acl" mount option can likely be removed. In the meantime, I've changed the #ifdef to not fail the build once we pass 2.6.53 and hopefully we can remove the (now-inactive) code after that point.

            adilger Andreas Dilger added a comment - I pushed http://review.whamcloud.com/8630 to bump the build versions further out. It finally appears that the http://review.whamcloud.com/9251 patch from TEI-974 has been deployed, so the "acl" mount option can likely be removed. In the meantime, I've changed the #ifdef to not fail the build once we pass 2.6.53 and hopefully we can remove the (now-inactive) code after that point.
            adilger Andreas Dilger made changes -
            Fix Version/s New: Lustre 2.7.0 [ 10631 ]
            Fix Version/s Original: Lustre 2.6.0 [ 10595 ]
            adilger Andreas Dilger made changes -
            Resolution Original: Fixed [ 1 ]
            Status Original: Resolved [ 5 ] New: Reopened [ 4 ]

            People

              jamesanunez James Nunez (Inactive)
              green Oleg Drokin
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: