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
            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) 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 added a comment -

            The root of this problem was not actually fixed.

            adilger Andreas Dilger added a comment - The root of this problem was not actually fixed.
            jlevi Jodi Levi (Inactive) added a comment -

            Patch landed to Master.

            jlevi Jodi Levi (Inactive) added a comment - Patch landed to Master.
            adilger Andreas Dilger added a comment -

            Pushed a new patch to bump the 2.5.53 build warnings out to 2.6.53: http://review.whamcloud.com/8627

            I'd rather see them fixed, but I don't want to turn them off or they never will be.

            adilger Andreas Dilger added a comment - Pushed a new patch to bump the 2.5.53 build warnings out to 2.6.53: http://review.whamcloud.com/8627 I'd rather see them fixed, but I don't want to turn them off or they never will be.
            adilger Andreas Dilger added a comment -

            This depends on TEI-974 actually removing the "acl" mount option from the autotest config. Otherwise, this will result in mounts failing due to an unknown mount option.

            adilger Andreas Dilger added a comment - This depends on TEI-974 actually removing the "acl" mount option from the autotest config. Otherwise, this will result in mounts failing due to an unknown mount option.

            People

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

              Dates

                Created:
                Updated:
                Resolved: