Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-1866

openssl-devel is required by hue RPM package

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 1.0.0
    • Component/s: build
    • Labels:
      None

      Description

      The hue package doesn't build on centos 7 and fedora 20 container:

      18:35:00  :hue-rpmerror: Failed build dependencies:
      18:35:00  	openssl is needed by hue-3.7.1-1.el7.centos.x86_64
      18:35:00  
      18:35:00  Installing /ws/output/hue/hue-3.7.1-1.el7.centos.src.rpm
      18:35:00  :hue-rpm FAILED
      18:35:00  
      18:35:00  FAILURE: Build failed with an exception.
      18:35:00  
      18:35:00  * Where:
      18:35:00  Script '/ws/packages.gradle' line: 420
      18:35:00  
      18:35:00  * What went wrong:
      18:35:00  Execution failed for task ':hue-rpm'.
      18:35:00  > Process 'command 'rpmbuild'' finished with non-zero exit value 1
      18:35:00  
      18:35:00  * Try:
      18:35:00  Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
      18:35:00  
      18:35:00  BUILD FAILED
      18:35:00  
      18:35:00  Total time: 59.451 secs
      18:35:05  Build step 'Execute shell' marked build as failure
      18:35:05  Archiving artifacts
      18:35:10  Finished: FAILURE
      

      The openssl package can be installed as openssl-devel's dependency on most of the Linux distro., but this does not stand in centos 7 and fedora 20. We should explicitly specify the package to be installed in toolchain.

      1. BIGTOP-1866.patch
        2 kB
        Evans Ye
      2. BIGTOP-1866.patch
        0.9 kB
        Evans Ye

        Activity

        Hide
        evans_ye Evans Ye added a comment -

        patch uploaded. Just to add openssl into the package list.

        BTW, I hit this when setting up Docker-bigtop-trunk builds:
        *centos 7 build failure
        *fedora 20 build failure

        Show
        evans_ye Evans Ye added a comment - patch uploaded. Just to add openssl into the package list. BTW, I hit this when setting up Docker-bigtop-trunk builds: * centos 7 build failure * fedora 20 build failure
        Hide
        warwithin YoungWoo Kim added a comment -

        Evans Ye, I'm looking into this. Your patch looks good but actually HUE depends on openssl-libs, so what about update the spec file or dependency for HUE itself? Like following:

        -BuildRequires: openssl 
        +BuildRequires: openssl-libs
        

        BTW, Installing openssl is a workaround for HUE because openssl-libs is installed as a dependency of openssl. I'm not sure, we need openssl for other components?

        Show
        warwithin YoungWoo Kim added a comment - Evans Ye , I'm looking into this. Your patch looks good but actually HUE depends on openssl-libs, so what about update the spec file or dependency for HUE itself? Like following: -BuildRequires: openssl +BuildRequires: openssl-libs BTW, Installing openssl is a workaround for HUE because openssl-libs is installed as a dependency of openssl. I'm not sure, we need openssl for other components?
        Hide
        oflebbe Olaf Flebbe added a comment -

        I am pretty sure that none of the Java code (i.e. the rest of bigtop) will ever call the openssl command.
        +1 for YoungWoo Kim suggestion.

        Show
        oflebbe Olaf Flebbe added a comment - I am pretty sure that none of the Java code (i.e. the rest of bigtop) will ever call the openssl command. +1 for YoungWoo Kim suggestion.
        Hide
        evans_ye Evans Ye added a comment -

        OK. I'll refine my patch. Thank you both for providing insightful comments. I'm not knowledgeable in hue as you, so thanks for helping.

        Show
        evans_ye Evans Ye added a comment - OK. I'll refine my patch. Thank you both for providing insightful comments. I'm not knowledgeable in hue as you, so thanks for helping.
        Hide
        plinnell Peter Linnell added a comment -

        Adding:

        BuildRequires: openssl-libs to the spec file should work correctly.

        Show
        plinnell Peter Linnell added a comment - Adding: BuildRequires: openssl-libs to the spec file should work correctly.
        Hide
        evans_ye Evans Ye added a comment -

        Well, when I update the spec file to require openssl-libs and ran through a test, the package can't be found on centos 6.
        Therefore I went to hue's website and found that it list openssl-devel as its dependency package.
        I change the buildrequires from openssl to openssl-devel and have the patch tested on centos 6, 7, and ubuntu.

        Show
        evans_ye Evans Ye added a comment - Well, when I update the spec file to require openssl-libs and ran through a test, the package can't be found on centos 6. Therefore I went to hue's website and found that it list openssl-devel as its dependency package. I change the buildrequires from openssl to openssl-devel and have the patch tested on centos 6, 7, and ubuntu.
        Hide
        oflebbe Olaf Flebbe added a comment -

        I have an – unrelated – build error on centos7 .. Will adress in a seperate JIRA. Thanks! WIll commit it.

        Show
        oflebbe Olaf Flebbe added a comment - I have an – unrelated – build error on centos7 .. Will adress in a seperate JIRA. Thanks! WIll commit it.

          People

          • Assignee:
            evans_ye Evans Ye
            Reporter:
            evans_ye Evans Ye
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development