Skip to content

HDDS-6176. Ozone service WebUI is not accessible with 404 error. #4308

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jojochuang merged 10 commits into from
Mar 2, 2023
Merged

HDDS-6176. Ozone service WebUI is not accessible with 404 error. #4308

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
e36981f
HDDS-6176. Ozone service WebUI is not accessible with 404 error.
ArafatKhan2198 Feb 24, 2023
8570c04
HDDS-6176. Ozone service WebUI is not accessible with 404 error.
ArafatKhan2198 Feb 24, 2023
78940e9
Made changes to the description
ArafatKhan2198 Feb 24, 2023
2afb4be
Made code review changes
ArafatKhan2198 Feb 25, 2023
41ccee9
Fixed checkstyle issue
ArafatKhan2198 Feb 26, 2023
ccf72ec
Code review changes approved
ArafatKhan2198 Feb 27, 2023
0255993
UT fails fixed
ArafatKhan2198 Mar 1, 2023
6220c8a
Added unit tests for jetty path creation
ArafatKhan2198 Mar 1, 2023
2f93080
Fixed Bugs
ArafatKhan2198 Mar 1, 2023
f3ed598
Changed method name and updated discription
ArafatKhan2198 Mar 2, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion hadoop-hdds/common/src/main/resources/ozone-default.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2935,7 +2935,7 @@
<description>
The base dir for HTTP Jetty server to extract contents. If this property
is not configured, by default, Jetty will create a directory inside the
directory named by the java.io.tmpdir System property（/tmp by default）.
Comment thread
ArafatKhan2198 marked this conversation as resolved.
directory named by the ozone.metadata.dirs（/tmp/metadata)
Comment thread
sadanand48 marked this conversation as resolved.
Outdated
Comment thread
ArafatKhan2198 marked this conversation as resolved.
Outdated
While in production environment, it's strongly suggested to instruct Jetty
to use a different parent directory by setting this property to the name
of the desired parent directory. The value of the property will be used to
Expand Down
10 changes: 6 additions & 4 deletions hadoop-hdds/framework/src/main/java/org/apache/hadoop/hdds/server/http/BaseHttpServer.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -178,11 +178,13 @@ public BaseHttpServer(MutableConfigurationSource conf, String name)
}

String baseDir = conf.get(OzoneConfigKeys.OZONE_HTTP_BASEDIR);
if (!StringUtils.isEmpty(baseDir)) {
createDir(baseDir);
httpServer.getWebAppContext().setAttribute(JETTY_BASETMPDIR, baseDir);
LOG.info("HTTP server of {} uses base directory {}", name, baseDir);

if (StringUtils.isEmpty(baseDir)) {
baseDir = conf.get(OzoneConfigKeys.OZONE_METADATA_DIRS) + "/webserver";
Comment thread
sadanand48 marked this conversation as resolved.
Outdated
Comment thread
ArafatKhan2198 marked this conversation as resolved.
Outdated
}
createDir(baseDir);
httpServer.getWebAppContext().setAttribute(JETTY_BASETMPDIR, baseDir);
LOG.info("HTTP server of {} uses base directory {}", name, baseDir);
}
}

Expand Down