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

mkfs should allow use of errors=panic

Details

    • Bug
    • Resolution: Fixed
    • Major
    • Lustre 2.8.0
    • Lustre 2.5.3
    • 3
    • 9223372036854775807

    Description

      mkfs.lustre defaults mountoptions to error=mount-ro and doesn't allow error=panic. Changing this using tunefs.lustre also gives warning about 'error=mount-ro' is missing.

      Is the use of error=panic discouraged? if so why?

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-6978 mkfs.lustre: LDISKFS-fs (loop0): Unrecognized mount option "force_over_128tb" or missing value

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. LU-6952 Mount options parsing problem

          • Major - Major loss of function.
          • Resolved
          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-6442 mkfs -G <value> parameter is not changed actually and default value is applied

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

          Activity

            [LU-6662] mkfs should allow use of errors=panic
            jgmitter Joseph Gmitter (Inactive) added a comment -

            Landed for 2.8

            jgmitter Joseph Gmitter (Inactive) added a comment - Landed for 2.8
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/15870/
            Subject: LU-6662 utils: allow overriding default mountopts
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 927effd2c2374508d904d2022c74574e508c07f1

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/15870/ Subject: LU-6662 utils: allow overriding default mountopts Project: fs/lustre-release Branch: master Current Patch Set: Commit: 927effd2c2374508d904d2022c74574e508c07f1
            pjones Peter Jones added a comment -

            Thanks Mahmoud but, given that there is a patch that - as far as I understand - you are using we should probably hold this open until that has landed to master

            pjones Peter Jones added a comment - Thanks Mahmoud but, given that there is a patch that - as far as I understand - you are using we should probably hold this open until that has landed to master
            mhanafi Mahmoud Hanafi added a comment -

            This LU can be closed

            mhanafi Mahmoud Hanafi added a comment - This LU can be closed
            adilger Andreas Dilger added a comment -

            Yes, each subtest should be self-contained.

            adilger Andreas Dilger added a comment - Yes, each subtest should be self-contained.
            jaylan Jay Lan (Inactive) added a comment -

            The conf-sanity.sh in 2.5.3 contains only up to test_77. Is it OK to just pick up test_88 added in #15870?

            jaylan Jay Lan (Inactive) added a comment - The conf-sanity.sh in 2.5.3 contains only up to test_77. Is it OK to just pick up test_88 added in #15870?

            People

              hongchao.zhang Hongchao Zhang
              mhanafi Mahmoud Hanafi
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: