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

l_getsepol does not build due to not requiring openssl-devel

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.16.0, Lustre 2.15.5
    • Lustre 2.15.3
    • llnl build farm

      lustre 2.15.3_3.llnl
    • 3
    • 9223372036854775807

    Description

      The l_getsepol utility does not build in our buildfarm because openssl-devel is not installed. The only "BuildRequires: openssl-devel" is for "with gss" and we aren't using gss.

      Attachments

        Issue Links

          is related to

          Improvement - An improvement or enhancement to an existing feature or task. LU-11914 Build error for l_getsepol.c due to missing openssl/evp.h

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

          Activity

            [LU-17226] l_getsepol does not build due to not requiring openssl-devel
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/54190/
            Subject: LU-17226 build: create config option for l_getsepol
            Project: fs/lustre-release
            Branch: b2_15
            Current Patch Set:
            Commit: e12e831794cd172ba687567b7b50548740d83b49

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/54190/ Subject: LU-17226 build: create config option for l_getsepol Project: fs/lustre-release Branch: b2_15 Current Patch Set: Commit: e12e831794cd172ba687567b7b50548740d83b49
            defazio Gian-Carlo Defazio added a comment -

            Can we get a second review on the b2_15 backport this week if possible? Thanks!

            defazio Gian-Carlo Defazio added a comment - Can we get a second review on the b2_15 backport this week if possible? Thanks!
            gerrit Gerrit Updater added a comment -

            "Gian-Carlo DeFazio <defazio1@llnl.gov>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/54190
            Subject: LU-17226 build: create config option for l_getsepol
            Project: fs/lustre-release
            Branch: b2_15
            Current Patch Set: 1
            Commit: 2c3e803bea474fc141702925e6c846e709fc2323

            gerrit Gerrit Updater added a comment - "Gian-Carlo DeFazio <defazio1@llnl.gov>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/54190 Subject: LU-17226 build: create config option for l_getsepol Project: fs/lustre-release Branch: b2_15 Current Patch Set: 1 Commit: 2c3e803bea474fc141702925e6c846e709fc2323
            pjones Peter Jones added a comment -

            Landed for 2.16

            pjones Peter Jones added a comment - Landed for 2.16
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/52849/
            Subject: LU-17226 build: create config option for l_getsepol
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 2777adcabd1032ddb886f913fa04d82a292ab379

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/52849/ Subject: LU-17226 build: create config option for l_getsepol Project: fs/lustre-release Branch: master Current Patch Set: Commit: 2777adcabd1032ddb886f913fa04d82a292ab379
            gerrit Gerrit Updater added a comment -

            "Gian-Carlo DeFazio <defazio1@llnl.gov>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52849
            Subject: LU-17226 build: create config option for l_getsepol
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: deddcb57ab27ba7fb4b961ce0aa51db7f1129612

            gerrit Gerrit Updater added a comment - "Gian-Carlo DeFazio <defazio1@llnl.gov>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52849 Subject: LU-17226 build: create config option for l_getsepol Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: deddcb57ab27ba7fb4b961ce0aa51db7f1129612
            sebastien Sebastien Buisson added a comment -

            Yes, good point Olaf.

            There is currently no config flag to disable l_getsepol build. Would that help with "mock", if we build l_getsepol by default but give the ability to disable via --disable-l_getsepol or something?

            Otherwise we can add openssl-devel to the BuildRequires as default, if it is not a too strong requirement.

            Thanks!

            sebastien Sebastien Buisson added a comment - Yes, good point Olaf. There is currently no config flag to disable l_getsepol build. Would that help with "mock", if we build l_getsepol by default but give the ability to disable via --disable-l_getsepol or something? Otherwise we can add openssl-devel to the BuildRequires as default, if it is not a too strong requirement. Thanks!

            People

              sebastien Sebastien Buisson
              defazio Gian-Carlo Defazio
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: