Skip to content

HDDS-15912. HttpFS Gateway launcher should use runtime-managed log/temp dirs instead of OZONE_HOME paths.#10811

Merged
aryangupta1998 merged 1 commit into
apache:masterfrom
aryangupta1998:HDDS-15912
Jul 21, 2026
Merged

HDDS-15912. HttpFS Gateway launcher should use runtime-managed log/temp dirs instead of OZONE_HOME paths.#10811
aryangupta1998 merged 1 commit into
apache:masterfrom
aryangupta1998:HDDS-15912

Conversation

@aryangupta1998

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

HttpFS launcher currently sets httpfs.log.dir and httpfs.temp.dir under OZONE_HOME (${OZONE_HOME}/log, ${OZONE_HOME}/temp).
HttpFSServerWebApp initializes through ServerWebApp, and Server.init() validates these directories at startup. If ${OZONE_HOME}/temp is missing, startup fails with S01: Dir .../temp does not exist, and Jetty stops the connector.

Fix: switch launcher defaults to runtime-managed directories:

httpfs.log.dir=${OZONE_LOG_DIR}
httpfs.temp.dir=${OZONE_PID_DIR}

This removes dependency on install-tree temp/log paths and makes HttpFS startup more robust across deployment layouts.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-15912

How was this patch tested?

Tested Manually.

@aryangupta1998 aryangupta1998 changed the title HttpFS Gateway launcher should use runtime-managed log/temp dirs instead of OZONE_HOME paths. HDDS-15912. HttpFS Gateway launcher should use runtime-managed log/temp dirs instead of OZONE_HOME paths. Jul 19, 2026
@jojochuang
jojochuang requested a review from dombizita July 20, 2026 16:19

@dombizita dombizita left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this @aryangupta1998! I had one small question.

Comment thread hadoop-ozone/dist/src/shell/ozone/ozone

@dombizita dombizita left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing it @aryangupta1998!

@aryangupta1998
aryangupta1998 merged commit 024509e into apache:master Jul 21, 2026
30 of 31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants