[GitHub] [hadoop-ozone] elek commented on a change in pull request #445: HDDS-2849. OM and SCM Web-server report HTTP 404 error when accessing…

[GitBox <gi...@apache.org>]

elek commented on a change in pull request #445: HDDS-2849. OM and SCM Web-server report HTTP 404 error when accessing…
URL: https://github.com/apache/hadoop-ozone/pull/445#discussion_r368479586
 
 

 ##########
 File path: hadoop-hdds/framework/src/main/java/org/apache/hadoop/hdds/server/BaseHttpServer.java
 ##########
 @@ -114,8 +118,13 @@ public BaseHttpServer(Configuration conf, String name) throws IOException {
                 + "production!");
         httpServer.addServlet("profile", "/prof", ProfileServlet.class);
       }
-    }
 
+      String baseDir = conf.get(OzoneConfigKeys.OZONE_HTTP_BASEDIR);
+      if (!StringUtils.isEmpty(baseDir)) {
+        httpServer.getWebAppContext().setAttribute(JETTY_BASETMPDIR, baseDir);
 
 Review comment:
   From an offline discussion I learned the problem: In some environments the `/tmp` cleaned up regularly (For me it's just at the restart but in some environments it's cleaned up more frequently). In these environments the web UI is not available after a while (when jetty temporary files are deleted).
    
   I am fine with this patch. We can remove this configuration setting long-term as we are planning to unify all the webui under the recon. Until that we can make it configurable...

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org