HDDS-15775. Support IPv6 literal authorities in OzoneFS URI parsing

[rich7420]

What changes were proposed in this pull request?

ofs://[::1]:9862/path fails because the authority is split on every colon.

BasicRootedOzoneFileSystem.java: the authority parsing splits on :, so a bracketed IPv6 literal yields more than two parts and throws IllegalArgumentException (URI_EXCEPTION_TEXT).

This change parses the authority with Guava HostAndPort, which understands bracketed IPv6 literals and rejects out-of-range ports. IPv6 hosts are kept bracketed when the OM host:port address is reassembled so the downstream parse stays unambiguous.

Why ofs only (o3fs is out of scope)

o3fs encodes the OM host inside the authority as o3fs://bucket.volume.host:port. For an IPv6 literal that would be o3fs://bucket.volume.[::1]:9862/, where the brackets sit in the middle of the authority. java.net.URI rejects that form (URISyntaxException: Illegal character in hostname), so an o3fs IPv6 authority can never reach the filesystem to begin with. ofs puts the authority at the start (ofs://[::1]:9862/), which java.net.URI accepts, so ofs is the only scheme where this fix is reachable. BasicOzoneFileSystem (o3fs) is left unchanged.

Parent epic: HDDS-15763.

What is the link to the Apache JIRA

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

How was this patch tested?

Added TestBasicOzoneFileSystems cases covering ofs authority parsing (hostname, service id, underscore host, [::1]:9862, [2001:db8::1]) and out-of-range port rejection.

https://github.com/rich7420/ozone/actions/runs/30821150742

[Copilot]

Pull request overview

Note

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Adds IPv6-literal-aware parsing for OzoneFS URI authorities (eg ofs://[::1]:9862/...) so bracketed IPv6 hosts are handled correctly and OM addresses remain unambiguous.

Changes:

• Switch authority parsing to an IPv6-aware parser (HostAndPort) in BasicRootedOzoneFileSystem.
• Ensure OM host resolution / OM address configuration correctly brackets IPv6 literals via getHostPortString.
• Add/extend unit tests validating IPv6 literal handling in filesystem init and OM host resolution.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
hadoop-ozone/ozonefs-common/src/main/java/org/apache/hadoop/fs/ozone/BasicRootedOzoneFileSystem.java	Parse authority with HostAndPort and preserve bracketed IPv6 literals.
hadoop-ozone/ozonefs-common/src/test/java/org/apache/hadoop/fs/ozone/TestBasicOzoneFileSystems.java	Add parameterized test cases covering bracketed IPv6 authorities for OFS.
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/OmUtils.java	Build host:port via getHostPortString before createSocketAddr to support IPv6 literals.
hadoop-ozone/common/src/test/java/org/apache/hadoop/ozone/TestOmUtils.java	Add test ensuring IPv6 literals don’t fail parsing in resolveOmHost.
hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/OzoneClientFactory.java	Configure OM address using getHostPortString to avoid ambiguous IPv6 host:port strings.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[smengcl]

looks good on the latest revision

[smengcl]

Thanks @rich7420 for the patch.