Skip to content

HDDS-15358. Update environment variable docs to match shell script behavior#447

Merged
jojochuang merged 3 commits into
apache:masterfrom
chihsuan:HDDS-15358
May 26, 2026
Merged

HDDS-15358. Update environment variable docs to match shell script behavior#447
jojochuang merged 3 commits into
apache:masterfrom
chihsuan:HDDS-15358

Conversation

@chihsuan
Copy link
Copy Markdown
Contributor

@chihsuan chihsuan commented May 24, 2026

What changes were proposed in this pull request?

This is the website companion to HDDS-14443 (apache/ozone#10352), which fixed the in-distribution docs (ozone-env.sh) and the ozone launcher script in the apache/ozone repo. The website hosts a separate, hand-maintained copy of the same environment variable information that still carried the old inconsistencies. This PR updates docs/.../02-environment-variables.md so it matches the actual behavior of the shell scripts, brought to full parity with the variables defined in apache/ozone#10352.

Role-Specific Environment Variables table (added the missing rows):

  • OZONE_HTTPFS_OPTS, OZONE_CSI_OPTS (daemon JVM opts)
  • OZONE_SH_OPTS, OZONE_FS_OPTS, OZONE_ADMIN_OPTS, OZONE_DEBUG_OPTS, OZONE_FREON_OPTS, OZONE_VAPOR_OPTS (client/tool JVM opts)
  • OZONE_DAEMON_JSVC_EXTRA_OPTS (extra arguments passed to jsvc when launching secure/privileged daemons)

Common Environment Variables table (added for parity):

  • OZONE_LOGLEVEL (default INFO)
  • OZONE_SECURE_LOG_DIR (default $OZONE_LOG_DIR)
  • OZONE_WORKER_NAMES

HttpFS Gateway admonition: the :::note previously stated that HttpFS has no dedicated opts variable and that its JVM properties must be added to the global OZONE_OPTS. That is no longer true. The note now states that OZONE_HTTPFS_OPTS is supported and behaves like the other daemons (the generic ozone_subcommand_opts handler folds OZONE_<SUBCMD>_OPTS into OZONE_OPTS).

No reference to the non-functional OZONE_SECURE_LOG variable exists on this page (the scripts read OZONE_SECURE_LOG_DIR), so no change was needed there. The frozen released snapshot under versioned_docs/version-2.1.0/ was intentionally not modified.

⚠️ Merge order: This documentation change describes behavior introduced by apache/ozone#10352. It should be merged only after apache/ozone#10352 has merged, otherwise the website would document variables that are not yet in any released/shipped script.

What is the link to the Apache Jira?

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

Follow-up to HDDS-14443 (apache/ozone#10352).

How was this patch tested?

  • pnpm build completed successfully (Server, Client, and Service Worker all compiled without errors).
  • Verified the local preview renders the updated table rows and the rewritten HttpFS admonition at /docs/next/administrator-guide/configuration/basic/environment-variables.
Screenshot 2026-05-24 at 12 37 16 PM Screenshot 2026-05-24 at 12 37 22 PM

@chihsuan chihsuan marked this pull request as ready for review May 24, 2026 04:30
@adoroszlai adoroszlai requested a review from jojochuang May 24, 2026 07:25
Comment on lines 52 to 55
:::note HttpFS Gateway Configuration
The HttpFS Gateway does not use an `OZONE_HTTPFS_OPTS` variable. Its specific JVM properties must be added to the global `OZONE_OPTS` variable.
The HttpFS Gateway supports a dedicated `OZONE_HTTPFS_OPTS` variable, just like the other daemons. Any JVM properties set in `OZONE_HTTPFS_OPTS` are folded into `OZONE_OPTS` when the gateway starts, so you no longer need to add them to the global `OZONE_OPTS` variable.
:::

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.

let's remove this section.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed the section in f1dbf78

Copy link
Copy Markdown
Contributor

@jojochuang jojochuang left a comment

Choose a reason for hiding this comment

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

this is good. we can merge after removing the httpfs gw configuration section.

@chihsuan chihsuan requested a review from jojochuang May 25, 2026 13:12
@jojochuang jojochuang merged commit 44a4bb9 into apache:master May 26, 2026
12 checks passed
@jojochuang
Copy link
Copy Markdown
Contributor

Merged. Thanks @chihsuan

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