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

sanity-hsm uses /etc/motd (which is probably of size 0)

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.12.1
    • Lustre 2.11.0, Lustre 2.10.4
    • 3
    • 15912

    Description

      In various tests sanity-hsm copies /etc/hosts, /etc/passwd, or /etc/modt for use as the test file. /etc/motd is likely of size 0 and may defeat the use of bandwidth control for timing. Uses of /etc/* should be replaced with a generated files of known suitable size. We should also add explicit tests using files of size 0.

      Attachments

        Issue Links

          Activity

            [LU-5680] sanity-hsm uses /etc/motd (which is probably of size 0)

            Anyone looking at this ?

            simmonsja James A Simmons added a comment - Anyone looking at this ?

            Reopening ticket to allow for patches to replace all uses of /etc files.

            jamesanunez James Nunez (Inactive) added a comment - Reopening ticket to allow for patches to replace all uses of /etc files.

            John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30795/
            Subject: LU-5680 tests: Remove use of /etc/motd from sanity-hsm
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set:
            Commit: 158b6382704947c0d23d5b79d2b94bdc7a04af10

            gerrit Gerrit Updater added a comment - John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30795/ Subject: LU-5680 tests: Remove use of /etc/motd from sanity-hsm Project: fs/lustre-release Branch: b2_10 Current Patch Set: Commit: 158b6382704947c0d23d5b79d2b94bdc7a04af10

            There are still a lot of place in sanity-hsm where /etc/{passwd,hosts} are used. Maybe this ticket should be reopened.

            bougetq Quentin Bouget (Inactive) added a comment - There are still a lot of place in sanity-hsm where /etc/{passwd,hosts} are used. Maybe this ticket should be reopened.

            Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/30795
            Subject: LU-5680 tests: Remove use of /etc/motd from sanity-hsm
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set: 1
            Commit: 05b19b2fe88732e36694ce738a915ccd5c0506f5

            gerrit Gerrit Updater added a comment - Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/30795 Subject: LU-5680 tests: Remove use of /etc/motd from sanity-hsm Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 05b19b2fe88732e36694ce738a915ccd5c0506f5
            mdiep Minh Diep added a comment -

            Landed for 2.11

            mdiep Minh Diep added a comment - Landed for 2.11

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/14021/
            Subject: LU-5680 tests: Remove use of /etc/motd from sanity-hsm
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 715c7e9166a34c1f2b781f3314d8b7c02e10950c

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/14021/ Subject: LU-5680 tests: Remove use of /etc/motd from sanity-hsm Project: fs/lustre-release Branch: master Current Patch Set: Commit: 715c7e9166a34c1f2b781f3314d8b7c02e10950c

            Patch http://review.whamcloud.com/14021 replaces all uses of /etc/* with creation of a small file of known size. A second patch will be created to add archiving of zero byte files to sanity-hsm.

            jamesanunez James Nunez (Inactive) added a comment - Patch http://review.whamcloud.com/14021 replaces all uses of /etc/* with creation of a small file of known size. A second patch will be created to add archiving of zero byte files to sanity-hsm.

            James Nunez (james.a.nunez@intel.com) uploaded a new patch: http://review.whamcloud.com/14021
            Subject: LU-5680 tests: Remove all use of /etc from sanity-hsm
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: d6545c1ffb2ded84df550a0e2b810f52f6f4d078

            gerrit Gerrit Updater added a comment - James Nunez (james.a.nunez@intel.com) uploaded a new patch: http://review.whamcloud.com/14021 Subject: LU-5680 tests: Remove all use of /etc from sanity-hsm Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: d6545c1ffb2ded84df550a0e2b810f52f6f4d078

            Hi James,
            Could you have a look at this one?
            Thank you!

            jlevi Jodi Levi (Inactive) added a comment - Hi James, Could you have a look at this one? Thank you!

            People

              jamesanunez James Nunez (Inactive)
              jhammond John Hammond
              Votes:
              0 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: