Skip to content

doc: add docker-compose guide and update deployment docs#455

Merged
imbajin merged 17 commits intoapache:masterfrom
bitflicker64:docs/hugegraph-doc-docker-bridge
Apr 7, 2026
Merged

doc: add docker-compose guide and update deployment docs#455
imbajin merged 17 commits intoapache:masterfrom
bitflicker64:docs/hugegraph-doc-docker-bridge

Conversation

@bitflicker64
Copy link
Copy Markdown
Contributor

@bitflicker64 bitflicker64 commented Mar 7, 2026

Purpose of the PR

Follow-up documentation update for apache/hugegraph#2952 which migrated Docker cluster setup from network_mode: host to Docker bridge networking.

Related PR: apache/hugegraph#2952

The doc repo had no wrong content — it simply never documented the Docker-based distributed cluster setup at all. This PR adds that missing documentation.

Main Changes

  • Create content/en/docs/guides/hugegraph-docker-cluster.md — new standalone Docker cluster guide covering single-node and 3-node setup, full HG_* env var reference tables, port reference, health checks, and troubleshooting
  • Create content/cn/docs/guides/hugegraph-docker-cluster.md — Chinese mirror
  • Update hugegraph-server.md (EN + CN) — fix Docker README URL, replace outdated inline compose yaml with references to new docker/ files, add 3-node cluster Docker quickstart inside section 5.1.1, add auth note
  • Update hugegraph-pd.md (EN + CN) — add section 3.3 Docker Deployment with HG_PD_* env var reference table
  • Update hugegraph-hstore.md (EN + CN) — add section 3.3 Docker Deployment with HG_STORE_* env var reference table, add section 6.3 Docker cluster quickstart

@imbajin imbajin changed the title Doc: add Docker bridge networking guide and update deployment docs doc: add docker-compose guide and update deployment docs Mar 8, 2026
@imbajin
Copy link
Copy Markdown
Member

imbajin commented Mar 8, 2026

BTW, we could only update the EN version during the review-period, then sync the CN version in the final stage

@bitflicker64
Copy link
Copy Markdown
Contributor Author

Updated all EN files pinned HUGEGRAPH_VERSION=1.7.0 on all compose commands, updated image references from :latest to 1.7.0, and removed the stale clone workaround notes. CN sync can be done in the final stage.

@bitflicker64 bitflicker64 requested a review from imbajin March 23, 2026 10:29

This comment was marked as outdated.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 9 comments.


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

#### 3.2 Compile from source

```bash
# 1. Clone the source code
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

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

In “3.2 Compile from source”, step 1 says “Clone the source code” but the git clone https://github.com/apache/hugegraph.git command is missing, so the following cd hugegraph step can’t be followed as written. Please add the clone command back or adjust the steps.

Suggested change
# 1. Clone the source code
# 1. Clone the source code
git clone https://github.com/apache/hugegraph.git

Copilot uses AI. Check for mistakes.
#### 3.2 Compile from source

```bash
# 1. Clone the source code
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

‼️ The source-build snippet says # 1. Clone the source code, but the actual git clone ... command was removed, so the later cd hugegraph and mvn ... steps are no longer runnable as written. Please restore the clone command or change the text to make the pre-cloned assumption explicit.

Suggested change
# 1. Clone the source code
git clone https://github.com/apache/hugegraph.git

@imbajin
Copy link
Copy Markdown
Member

imbajin commented Mar 26, 2026

BTW, we could only update the EN version during the review-period, then sync the CN version in the final stage

Almost LGTM now, after all EN docs addressed comments, sync the CN doc together

@bitflicker64
Copy link
Copy Markdown
Contributor Author

Synced all CN files to match EN pinned HUGEGRAPH_VERSION=1.7.0 on all compose commands, updated image references from :latest to 1.7.0, restored git clone in source build sections, and fixed docker-compose.dev.yml filename.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.


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

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 15 comments.


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

@imbajin imbajin requested a review from VGalaxies March 26, 2026 08:24
@imbajin
Copy link
Copy Markdown
Member

imbajin commented Apr 5, 2026

I'll check the CN version ASAP

@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Apr 5, 2026
@bitflicker64
Copy link
Copy Markdown
Contributor Author

bitflicker64 commented Apr 5, 2026

I'll check the CN version ASAP

EN files are all set, let me know if they look good and I'll sync the CN side.
edit - EN and CN are both synced now. Let me know if everything looks good

bitflicker64 and others added 2 commits April 5, 2026 13:23
Revise Chinese and English docs for HugeGraph Docker deployment to standardize on Docker Compose wording and usage. Add notes that users should clone/pull the HugeGraph repo (or at least the docker/ directory), include a reminder to keep the HUGEGRAPH_VERSION aligned with releases, and adjust memory recommendations for Mac (3-node cluster ~12 GB) with platform guidance. Clarify compose file descriptions, emphasize container hostnames for HG_* env vars, change some example log years to 2026, reformat deprecated-alias lines, and number/troubleshooting items for clarity. Minor phrasing and section-title tweaks across hugegraph-docker-cluster.md, hugegraph-hstore.md, hugegraph-pd.md, and hugegraph-server.md (EN & CN).

Signed-off-by: imbajin <jin@apache.org>
@imbajin imbajin force-pushed the docs/hugegraph-doc-docker-bridge branch from 5a2373b to f454c8c Compare April 7, 2026 11:45
@imbajin imbajin requested a review from Copilot April 7, 2026 11:49
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 13 comments.


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

Comment on lines +52 to +62
1. PD nodes start first and must pass the `/v1/health` check
2. Store nodes start only after all PD nodes are healthy
3. Server nodes start last, after all PD and Store nodes are healthy

Verify that the cluster is healthy:
```bash
curl http://localhost:8620/v1/health # PD health
curl http://localhost:8520/v1/health # Store health
curl http://localhost:8080/versions # Server
curl http://localhost:8620/v1/stores # Registered stores
curl http://localhost:8620/v1/partitions # Partition assignment
Copy link

Copilot AI Apr 7, 2026

Choose a reason for hiding this comment

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

This guide uses /v1/health for PD/Store health checks, but the existing quickstart docs for PD/Store verification use /actuator/health (e.g. content/en/docs/quickstart/hugegraph/hugegraph-pd.md:186 and hugegraph-hstore.md:270). Please clarify which endpoint is correct for Docker deployments (and/or document both) to avoid confusion.

Suggested change
1. PD nodes start first and must pass the `/v1/health` check
2. Store nodes start only after all PD nodes are healthy
3. Server nodes start last, after all PD and Store nodes are healthy
Verify that the cluster is healthy:
```bash
curl http://localhost:8620/v1/health # PD health
curl http://localhost:8520/v1/health # Store health
curl http://localhost:8080/versions # Server
curl http://localhost:8620/v1/stores # Registered stores
curl http://localhost:8620/v1/partitions # Partition assignment
1. PD nodes start first and must pass the Docker Compose health check (`/v1/health` in this guide)
2. Store nodes start only after all PD nodes are healthy
3. Server nodes start last, after all PD and Store nodes are healthy
> **Health endpoint note**: In the Docker Compose deployment documented here, PD and Store are checked via `/v1/health`. Some other HugeGraph quickstart documents use `/actuator/health` for PD/Store verification. If you are comparing guides or using a different image/version, verify which endpoint your deployment exposes; for the compose files in this guide, start with `/v1/health`.
Verify that the cluster is healthy:
```bash
curl http://localhost:8620/v1/health # PD health (Docker Compose in this guide)
curl http://localhost:8620/actuator/health # PD health (seen in some other docs/images)
curl http://localhost:8520/v1/health # Store health (Docker Compose in this guide)
curl http://localhost:8520/actuator/health # Store health (seen in some other docs/images)
curl http://localhost:8080/versions # Server
curl http://localhost:8620/v1/stores # Registered stores
curl http://localhost:8620/v1/partitions # Partition assignment

Copilot uses AI. Check for mistakes.
imbajin and others added 3 commits April 7, 2026 19:56
Cleanup and wording/formatting improvements across HugeGraph quickstart docs (CN/EN) for hugegraph-hstore, hugegraph-pd and hugegraph-server. Changes include: normalize “macOS” and Docker/Docker Hub terminology; clarify Java 11/JDK guidance and authentication requirements; rephrase HugeGraph-Server naming and startup/preload descriptions for clarity; update PD config comment for initial-store-count; add/standardize PD API curl example and expanded Store JSON example; improve Docker/Docker Compose examples (including mounting example.groovy and notes about Docker Desktop memory); fix punctuation, grammar and minor layout issues.

Signed-off-by: imbajin <jin@apache.org>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Demote the "Docker Distributed Cluster" section heading from H4 to H6 in both English and Chinese quickstart pages to correct the document hierarchy and formatting (files updated: content/en/docs/quickstart/hugegraph/hugegraph-server.md and content/cn/docs/quickstart/hugegraph/hugegraph-server.md).

Signed-off-by: imbajin <jin@apache.org>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

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


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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Member

@imbajin imbajin left a comment

Choose a reason for hiding this comment

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

Enhanced some desc & merge it now

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 7, 2026
@imbajin imbajin merged commit 1fda07c into apache:master Apr 7, 2026
1 check passed
github-actions bot pushed a commit that referenced this pull request Apr 7, 2026
Revise Chinese and English docs for HugeGraph Docker deployment to standardize on Docker Compose wording and usage. Add notes that users should clone/pull the HugeGraph repo (or at least the docker/ directory), include a reminder to keep the HUGEGRAPH_VERSION aligned with releases, and adjust memory recommendations for Mac (3-node cluster ~12 GB) with platform guidance. Clarify compose file descriptions, emphasize container hostnames for HG_* env vars, change some example log years to 2026, reformat deprecated-alias lines, and number/troubleshooting items for clarity. Minor phrasing and section-title tweaks across hugegraph-docker-cluster.md, hugegraph-hstore.md, hugegraph-pd.md, and hugegraph-server.md (EN & CN).

---------

Signed-off-by: imbajin <jin@apache.org>
Co-authored-by: imbajin <jin@apache.org>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> 1fda07c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants