Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 1.0.0
    • Component/s: general
    • Labels:
      None

      Description

      To track Whirr retirement from the stack

      1. BIGTOP-1571.patch
        298 kB
        Dasha Boudnik
      2. BIGTOP-1571.patch
        285 kB
        Dasha Boudnik
      3. BIGTOP-1571.patch
        285 kB
        Dasha Boudnik

        Activity

        Hide
        rvs Roman Shaposhnik added a comment -

        Yeah! Lets make it happen!

        Show
        rvs Roman Shaposhnik added a comment - Yeah! Lets make it happen!
        Hide
        dasha.boudnik Dasha Boudnik added a comment -

        I think that takes care of it; let me know if I missed anything.

        Found a mistake in install-tomcat.sh while I was doing this:

        Required not-so-options:
             --build-dir=DIR             path to Whirr dist.dir
             --prefix=PREFIX             path to install into
        
          Optional options:
             --doc-dir=DIR               path to install docs into [/usr/share/doc/whirr]
             --lib-dir=DIR               path to install Whirr home [/usr/lib/whirr]

        Will open a separate ticket to fix these whirrs to bigtop-tomcats.

        Show
        dasha.boudnik Dasha Boudnik added a comment - I think that takes care of it; let me know if I missed anything. Found a mistake in install-tomcat.sh while I was doing this: Required not-so-options: --build-dir=DIR path to Whirr dist.dir --prefix=PREFIX path to install into Optional options: --doc-dir=DIR path to install docs into [/usr/share/doc/whirr] --lib-dir=DIR path to install Whirr home [/usr/lib/whirr] Will open a separate ticket to fix these whirrs to bigtop-tomcats.
        Hide
        rvs Roman Shaposhnik added a comment -

        Could you please post, whatchanged output so its easier to see what got removed (since its mostly file removal)?

        Show
        rvs Roman Shaposhnik added a comment - Could you please post, whatchanged output so its easier to see what got removed (since its mostly file removal)?
        Hide
        dasha.boudnik Dasha Boudnik added a comment -
        modified:   bigtop-deploy/live-cd/apache-bigtop.ks
        deleted:    bigtop-packages/src/common/whirr/do-component-build
        deleted:    bigtop-packages/src/common/whirr/install_whirr.sh
        deleted:    bigtop-packages/src/common/whirr/whirr.1
        deleted:    bigtop-packages/src/deb/whirr/changelog
        deleted:    bigtop-packages/src/deb/whirr/compat
        deleted:    bigtop-packages/src/deb/whirr/control
        deleted:    bigtop-packages/src/deb/whirr/copyright
        deleted:    bigtop-packages/src/deb/whirr/rules
        deleted:    bigtop-packages/src/deb/whirr/source/format
        modified:   bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/DeployCDH.groovy
        modified:   bigtop-tests/test-artifacts/package/src/main/resources/package_data.xml
        modified:   bigtop-tests/test-artifacts/package/src/main/resources/urpmi/package_data.xml
        modified:   bigtop-tests/test-artifacts/package/src/main/resources/yum/package_data.xml
        modified:   bigtop-tests/test-artifacts/package/src/main/resources/zypper/package_data.xml
        deleted:    bigtop-tests/test-artifacts/package/src/main/resources/zypper/whirr.xml
        modified:   bigtop.mk
        Show
        dasha.boudnik Dasha Boudnik added a comment - modified: bigtop-deploy/live-cd/apache-bigtop.ks deleted: bigtop-packages/src/common/whirr/do-component-build deleted: bigtop-packages/src/common/whirr/install_whirr.sh deleted: bigtop-packages/src/common/whirr/whirr.1 deleted: bigtop-packages/src/deb/whirr/changelog deleted: bigtop-packages/src/deb/whirr/compat deleted: bigtop-packages/src/deb/whirr/control deleted: bigtop-packages/src/deb/whirr/copyright deleted: bigtop-packages/src/deb/whirr/rules deleted: bigtop-packages/src/deb/whirr/source/format modified: bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/DeployCDH.groovy modified: bigtop-tests/test-artifacts/package/src/main/resources/package_data.xml modified: bigtop-tests/test-artifacts/package/src/main/resources/urpmi/package_data.xml modified: bigtop-tests/test-artifacts/package/src/main/resources/yum/package_data.xml modified: bigtop-tests/test-artifacts/package/src/main/resources/zypper/package_data.xml deleted: bigtop-tests/test-artifacts/package/src/main/resources/zypper/whirr.xml modified: bigtop.mk
        Hide
        cos Konstantin Boudnik added a comment - - edited

        Looks like bigtop-packages/src/rpm/whirr is missed

        Show
        cos Konstantin Boudnik added a comment - - edited Looks like bigtop-packages/src/rpm/whirr is missed
        Hide
        rvs Roman Shaposhnik added a comment -

        What Konstantin Boudnik said. Otherwise +1

        Show
        rvs Roman Shaposhnik added a comment - What Konstantin Boudnik said. Otherwise +1
        Hide
        dasha.boudnik Dasha Boudnik added a comment - - edited

        Konstantin Boudnik caught one I missed. Patch reattached, and here's the updated whatchanged:

        modified:   bigtop-deploy/live-cd/apache-bigtop.ks
        modified:   bigtop-packages/src/common/bigtop-tomcat/install_tomcat.sh
        deleted:    bigtop-packages/src/common/whirr/do-component-build
        deleted:    bigtop-packages/src/common/whirr/install_whirr.sh
        deleted:    bigtop-packages/src/common/whirr/whirr.1
        deleted:    bigtop-packages/src/deb/whirr/changelog
        deleted:    bigtop-packages/src/deb/whirr/compat
        deleted:    bigtop-packages/src/deb/whirr/control
        deleted:    bigtop-packages/src/deb/whirr/copyright
        deleted:    bigtop-packages/src/deb/whirr/rules
        deleted:    bigtop-packages/src/deb/whirr/source/format
        modified:   bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/DeployCDH.groovy
        modified:   bigtop-tests/test-artifacts/package/src/main/resources/package_data.xml
        modified:   bigtop-tests/test-artifacts/package/src/main/resources/urpmi/package_data.xml
        modified:   bigtop-tests/test-artifacts/package/src/main/resources/yum/package_data.xml
        modified:   bigtop-tests/test-artifacts/package/src/main/resources/zypper/package_data.xml
        deleted:    bigtop-tests/test-artifacts/package/src/main/resources/zypper/whirr.xml
        modified:   bigtop.mk
        deleted:    bigtop-packages/src/rpm/whirr/BUILD/.gitignore
        deleted:    bigtop-packages/src/rpm/whirr/RPMS/.gitignore
        deleted:    bigtop-packages/src/rpm/whirr/SOURCES/.gitignore
        deleted:    bigtop-packages/src/rpm/whirr/SPECS/whirr.spec
        deleted:    bigtop-packages/src/rpm/whirr/SRPMS/.gitignore
        Show
        dasha.boudnik Dasha Boudnik added a comment - - edited Konstantin Boudnik caught one I missed. Patch reattached, and here's the updated whatchanged: modified: bigtop-deploy/live-cd/apache-bigtop.ks modified: bigtop-packages/src/common/bigtop-tomcat/install_tomcat.sh deleted: bigtop-packages/src/common/whirr/do-component-build deleted: bigtop-packages/src/common/whirr/install_whirr.sh deleted: bigtop-packages/src/common/whirr/whirr.1 deleted: bigtop-packages/src/deb/whirr/changelog deleted: bigtop-packages/src/deb/whirr/compat deleted: bigtop-packages/src/deb/whirr/control deleted: bigtop-packages/src/deb/whirr/copyright deleted: bigtop-packages/src/deb/whirr/rules deleted: bigtop-packages/src/deb/whirr/source/format modified: bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/DeployCDH.groovy modified: bigtop-tests/test-artifacts/package/src/main/resources/package_data.xml modified: bigtop-tests/test-artifacts/package/src/main/resources/urpmi/package_data.xml modified: bigtop-tests/test-artifacts/package/src/main/resources/yum/package_data.xml modified: bigtop-tests/test-artifacts/package/src/main/resources/zypper/package_data.xml deleted: bigtop-tests/test-artifacts/package/src/main/resources/zypper/whirr.xml modified: bigtop.mk deleted: bigtop-packages/src/rpm/whirr/BUILD/.gitignore deleted: bigtop-packages/src/rpm/whirr/RPMS/.gitignore deleted: bigtop-packages/src/rpm/whirr/SOURCES/.gitignore deleted: bigtop-packages/src/rpm/whirr/SPECS/whirr.spec deleted: bigtop-packages/src/rpm/whirr/SRPMS/.gitignore
        Hide
        dasha.boudnik Dasha Boudnik added a comment -

        Whoops, didn't see the comments

        Show
        dasha.boudnik Dasha Boudnik added a comment - Whoops, didn't see the comments
        Hide
        cos Konstantin Boudnik added a comment -

        The patch doesn't apply with patch command - only with IDEA. Could you please add the git format-patch version instead.

        Show
        cos Konstantin Boudnik added a comment - The patch doesn't apply with patch command - only with IDEA. Could you please add the git format-patch version instead.
        Hide
        cos Konstantin Boudnik added a comment -

        Committed and pushed as 14a9857

        Show
        cos Konstantin Boudnik added a comment - Committed and pushed as 14a9857

          People

          • Assignee:
            dasha.boudnik Dasha Boudnik
            Reporter:
            cos Konstantin Boudnik
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development