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

Tachyon webapp directory not located where expected

    Details

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

      Description

      Here is where the RPM places the webapp

      bash-4.1# rpm -ql tachyon | grep webapp
      /usr/share/tachyon/web/webapp
      /usr/share/tachyon/web/webapp/browse-pagination-header.jsp
      /usr/share/tachyon/web/webapp/browse.jsp
      /usr/share/tachyon/web/webapp/configuration.jsp
      /usr/share/tachyon/web/webapp/css
      /usr/share/tachyon/web/webapp/css/bootstrap-responsive.min.css
      /usr/share/tachyon/web/webapp/css/bootstrap.min.css
      /usr/share/tachyon/web/webapp/css/tachyoncustom.min.css
      /usr/share/tachyon/web/webapp/dependency.jsp
      /usr/share/tachyon/web/webapp/footer.jsp
      /usr/share/tachyon/web/webapp/general.jsp
      /usr/share/tachyon/web/webapp/header.jsp
      /usr/share/tachyon/web/webapp/img
      /usr/share/tachyon/web/webapp/img/amplab_logo.png
      /usr/share/tachyon/web/webapp/img/glyphicons-halflings-white.png
      /usr/share/tachyon/web/webapp/img/glyphicons-halflings.png
      /usr/share/tachyon/web/webapp/img/logo.png
      /usr/share/tachyon/web/webapp/index.html
      /usr/share/tachyon/web/webapp/js
      /usr/share/tachyon/web/webapp/js/bootstrap.min.js
      /usr/share/tachyon/web/webapp/js/cookies.min.js
      /usr/share/tachyon/web/webapp/js/jquery-1.9.1.min.js
      /usr/share/tachyon/web/webapp/memory-pagination-header.jsp
      /usr/share/tachyon/web/webapp/memory.jsp
      /usr/share/tachyon/web/webapp/pagination-component.jsp
      /usr/share/tachyon/web/webapp/pagination-control.jsp
      /usr/share/tachyon/web/webapp/viewFile.jsp
      /usr/share/tachyon/web/webapp/workers.jsp
      

      Here is where tachyon expects it

      # less /var/lib/tachyon/logs/master.log\@172.17.0.3_01-17-2015
      015-01-17 23:36:56,897 WARN  webapp.WebAppContext (WebAppContext.java:doStart) - Failed startup of context o.e.j.w.WebAppContext{/,null},/var/lib/tachyon/core/src/main/webapp
      java.io.FileNotFoundException: /var/lib/tachyon/core/src/main/webapp
      

        Issue Links

          Activity

          Hide
          dcapwell David Capwell added a comment -

          This looks to be what gets set when bigtop runs on centos6

          bash /home/pivotal/jenkins_home/workspace/BDAS-tachyon-bigtop-repo/build/tachyon/rpm/SOURCES/install_tachyon.sh --build-dir=/home/pivotal/jenkins_home/workspace/BDAS-tachyon-bigtop-repo/build/tachyon/rpm/BUILDROOT/tachyon-0.6.0-19.el6.x86_64 --bin-dir=/usr/bin --data-dir=/usr/share --libexec-dir=/usr/libexec --var-dir=/var --prefix=/home/pivotal/jenkins_home/workspace/BDAS-tachyon-bigtop-repo/build/tachyon/rpm/BUILDROOT/tachyon-0.6.0-19.el6.x86_64
          
          Show
          dcapwell David Capwell added a comment - This looks to be what gets set when bigtop runs on centos6 bash /home/pivotal/jenkins_home/workspace/BDAS-tachyon-bigtop-repo/build/tachyon/rpm/SOURCES/install_tachyon.sh --build-dir=/home/pivotal/jenkins_home/workspace/BDAS-tachyon-bigtop-repo/build/tachyon/rpm/BUILDROOT/tachyon-0.6.0-19.el6.x86_64 --bin-dir=/usr/bin --data-dir=/usr/share --libexec-dir=/usr/libexec -- var -dir=/ var --prefix=/home/pivotal/jenkins_home/workspace/BDAS-tachyon-bigtop-repo/build/tachyon/rpm/BUILDROOT/tachyon-0.6.0-19.el6.x86_64
          Hide
          dcapwell David Capwell added a comment -

          The above is with tachyon master branch. The webapp default location logic is different between 0.5 release and master. The value can be overridden with tachyon.web.resources property. The RPM could set this property to be /usr/share/tachyon/web/webapp, but how to do this is also different between 0.5 and soon to be master (0.5 and master are system properties, Henry Saputra finished off the work for using a config file rather than system properties).

          jay vyas what would you recommend in this case?

          Show
          dcapwell David Capwell added a comment - The above is with tachyon master branch. The webapp default location logic is different between 0.5 release and master. The value can be overridden with tachyon.web.resources property. The RPM could set this property to be /usr/share/tachyon/web/webapp, but how to do this is also different between 0.5 and soon to be master (0.5 and master are system properties, Henry Saputra finished off the work for using a config file rather than system properties). jay vyas what would you recommend in this case?
          Hide
          dcapwell David Capwell added a comment -

          Oh perfect, seems that the path did not change between 0.5 and master, so this patch should be possible between both. Ill take a look at that in a bit.

          Show
          dcapwell David Capwell added a comment - Oh perfect, seems that the path did not change between 0.5 and master, so this patch should be possible between both. Ill take a look at that in a bit.
          Hide
          jayunit100 jay vyas added a comment -

          David Capwell do you want to merge this into BIGTOP1607 as a general tachyon improvement ? If so we can close this JIRA .

          Show
          jayunit100 jay vyas added a comment - David Capwell do you want to merge this into BIGTOP1607 as a general tachyon improvement ? If so we can close this JIRA .
          Hide
          dcapwell David Capwell added a comment -

          This one I think ill have a patch in a moment. BIGTOP-1607 I think requires future fixes in tachyon (unless you can think of simpler fix), which might be problematic for bigtop with the current rate of releases for tachyon.

          Show
          dcapwell David Capwell added a comment - This one I think ill have a patch in a moment. BIGTOP-1607 I think requires future fixes in tachyon (unless you can think of simpler fix), which might be problematic for bigtop with the current rate of releases for tachyon.
          Hide
          dcapwell David Capwell added a comment -

          jay vyas Tested on a centos6 docker container and the tachyon logs are now clean and tachyon runTests now passes (couldn't connect before)

          Show
          dcapwell David Capwell added a comment - jay vyas Tested on a centos6 docker container and the tachyon logs are now clean and tachyon runTests now passes (couldn't connect before)
          Hide
          dcapwell David Capwell added a comment - - edited

          Tested on both 0.5 and master

          Show
          dcapwell David Capwell added a comment - - edited Tested on both 0.5 and master
          Hide
          rvs Roman Shaposhnik added a comment -

          +1. Any reason we shouldn't be committing this?

          Show
          rvs Roman Shaposhnik added a comment - +1. Any reason we shouldn't be committing this?
          Hide
          jayunit100 jay vyas added a comment -

          sure ill commit !

          Show
          jayunit100 jay vyas added a comment - sure ill commit !
          Hide
          jayunit100 jay vyas added a comment -

          thanks david ! commited.

          i guess we are pretty close now for this to be a first class demo of tachyon's capabilities.

          Lets move the rest of the patches to BIGTOP-1572, shall we ? I think that will give us a good final perfection of tachyon packaging, maybe at that point we can make a splash on the blog.

          Show
          jayunit100 jay vyas added a comment - thanks david ! commited. i guess we are pretty close now for this to be a first class demo of tachyon's capabilities. Lets move the rest of the patches to BIGTOP-1572 , shall we ? I think that will give us a good final perfection of tachyon packaging, maybe at that point we can make a splash on the blog.

            People

            • Assignee:
              dcapwell David Capwell
              Reporter:
              dcapwell David Capwell
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development