Skip to content

feat(hugegraph): add standalone and distributed addon - #3412

Open
weicao wants to merge 16 commits into
release-1.0from
dev/hugegraph-release-1.0
Open

feat(hugegraph): add standalone and distributed addon#3412
weicao wants to merge 16 commits into
release-1.0from
dev/hugegraph-release-1.0

Conversation

@weicao

@weicao weicao commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Summary

Add a HugeGraph 1.7.0 addon on release-1.0.

This is the HugeGraph development-branch main PR (dev/hugegraph-release-1.0 -> release-1.0). Later fixes go through child PRs into this branch, not directly onto release-1.0.

Scope

Supported:

  • Standalone: one HugeGraph Server 1.7.0 replica with embedded RocksDB
  • HTTP 8080 and Gremlin 8182
  • admin init system account
  • Persistent data on /hugegraph-data
  • Full-instance checkpoint backup of every persistent graph (standalone)
  • Restore to a new same-version standalone Cluster
  • Restart, Stop/Start, VerticalScaling, VolumeExpansion, Expose
  • Metrics sidecar hugegraph-exporter:0.1.1 (amd64+arm64) and MonitorDefinition
  • Distributed: PD + Store + Server topology, provision [pd,store] then server, Raft identity from pod FQDNs
  • PD HG_PD_INITIAL_STORE_COUNT derived from Store FQDNs (official 3-store bootstrap)
  • Server start waits for every Store /v1/health
  • Store start waits for every PD /v1/health

Not in this PR:

  • Distributed backup/restore, Store scale-in/rebalance, roleProbe
  • TLS, reconfigure, PITR, incremental/selective restore, RebuildInstance, cross-version or cross-topology restore

Candidate package

Item Pin
Target branch release-1.0 @ 678ef794
Development branch dev/hugegraph-release-1.0
Development head c986b81be7dcbcbf7801ba390c4fbc72bd9c3d1c
Addon path addons/hugegraph, addons-cluster/hugegraph, examples/hugegraph
Chart / app hugegraph-1.0.0 / 1.7.0
Exporter image docker.io/apecloud/hugegraph-exporter:0.1.1
Engine images hugegraph:1.7.0, hugegraph-pd:1.7.0, hugegraph-store:1.7.0, hugegraph-server:1.7.0 (CI rewrites official hugegraph/* to docker.io/apecloud/*)
KB line KubeBlocks release-1.0
Tests PR apecloud/kubeblocks-tests#949 @ 8ef64760

Child / companion PRs

Evidence

Offline on this head:

Prior runtime on equivalent standalone content (8220a018): final fresh all 33/0/0. Historical, not a new release-final count on ef535ae.

Acceptance gate

Reviewer cannot approve until both run on this exact head:

  • standalone checkpoint backup/restore
  • distributed Cluster create

Release-final still requires repeated full kubeblocks-tests HugeGraph suite rounds on this pinned head, on a dedicated HugeGraph vcluster. This PR is the upstreaming vehicle, not the release-standard closeout. Launcher: tests#949 hugegraph/launch-reviewer-gate.sh. No approved dedicated target yet.

Contract

Reviewed against addon API docs for ClusterDefinition topology (docs/addon-api/03-cluster-definition.md: orders, default topology, component name alignment), ComponentDefinition lifecycle/accounts, BackupPolicyTemplate/ActionSet checkpoint restore, and minimum acceptance (12a) for the declared standalone topology. Distributed topology is declared and statically contracted; distributed BR / scale-in / roleProbe / TLS remain unclaimed (12b). Exporter/MonitorDefinition is an observability enhancement, not an addon-api contract requirement.

@codecov-commenter

codecov-commenter commented Aug 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (678ef79) to head (d5e2d25).

Additional details and impacted files
@@             Coverage Diff             @@
##           release-1.0   #3412   +/-   ##
===========================================
  Coverage         0.00%   0.00%           
===========================================
  Files               69      69           
  Lines             7275    7275           
===========================================
  Misses            7275    7275           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@leon-ape leon-ape 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.

The title omits the required addon scope for this single-addon change. The PR cannot pass the approval title gate.

@weicao weicao changed the title feat: add HugeGraph standalone addon feat(hugegraph): add standalone addon Aug 18, 2026
Co-authored-by: Ava <ava@apecloud.com>
Co-authored-by: Amber <amber@slock.local>
@weicao weicao added the nopick Not auto cherry-pick when PR merged label Aug 18, 2026
@weicao weicao changed the title feat(hugegraph): add standalone addon feat(hugegraph): add standalone and distributed addon Aug 18, 2026
Co-authored-by: Amber <amber@slock.local>
@weicao

weicao commented Aug 18, 2026

Copy link
Copy Markdown
Contributor Author

Reviewer runtime gate on 83540814 is accepted. Title gate is already closed. No new source change.

Missing live evidence on this exact head:

  • standalone checkpoint backup/restore
  • distributed topology Cluster create

Those runs need a dedicated HugeGraph vcluster. None is handed off yet (local Kubernetes is out of scope). Routed Ops for that entrance. Offline contracts remain 25/0/0 on kubeblocks-tests#949 @ ac321e2d. Historical standalone 33/0/0 was on 8220a018, not this SHA.

@weicao

weicao commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Candidate SHA moved to f79df452 (child #3417). Store start now fails if PD /v1/health never succeeds. Exact-head live rows must restart on this SHA when the dedicated vcluster is handed off.

@weicao

weicao commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Candidate SHA moved to bb20a02c (child #3418). Server start now fails if Store /v1/health never succeeds. Exact-head live rows must restart on this SHA.

@weicao

weicao commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Head is now 9b1cc331 after merging #3419.

HG_PD_INITIAL_STORE_COUNT is derived from STORE_POD_FQDNS instead of hardcoded 1, matching official pd.initial-store-count for a 3-store bootstrap.

Exact-head live rows are still required and still blocked on a dedicated HugeGraph vcluster:

  • standalone checkpoint backup/restore
  • distributed Cluster create

Offline contract on this head: PASS.

Co-authored-by: Amber <amber@slock.local>
@weicao

weicao commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Head is now 2b088d88 after merging #3420.

Server start waits for every Store /v1/health in STORE_POD_FQDNS, not only the first FQDN. That matches official 3-store compose and the #3419 initial-store-count pin.

Exact-head live rows still required and still blocked on a dedicated HugeGraph vcluster:

  • standalone checkpoint backup/restore
  • distributed Cluster create

Co-authored-by: Amber <amber@slock.local>
@weicao

weicao commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Head is now ef535ae4 after merging #3421.

Store start waits for every PD /v1/health in PD_POD_FQDNS, not only the first FQDN. That matches official 3-PD compose.

Exact-head live rows still required and still blocked on a dedicated HugeGraph vcluster:

  • standalone checkpoint backup/restore
  • distributed Cluster create

@weicao

weicao commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Head is now c986b81b after merging #3422.

Store/Server health waits default to 90x2s. Store and Server startupProbe failureThreshold is 72 (360s), so official 3-node start_period cannot trip fail-closed.

Exact-head live rows still required and still blocked on a dedicated HugeGraph vcluster:

  • standalone checkpoint backup/restore
  • distributed Cluster create

@weicao

weicao commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Reviewer restated: no new source finding on c986b81b. Exact-head live rows still required and still blocked on a dedicated HugeGraph vcluster:

  • standalone checkpoint backup/restore
  • distributed Cluster create

Launcher: tests#949 @ 5cf1bc88 hugegraph/launch-reviewer-gate.sh.

Co-authored-by: Amber <amber@slock.local>

@leon-ape leon-ape 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.

Exact-head review checkpoint for d5e2d25f1f9a39053f235d2a9307bd9dd2df2cdd:

  • Source and contract review is clean; the test-only schema-pin successor introduces no new source finding.
  • Approval is pending because this exact head has no runtime result for standalone checkpoint backup/restore or distributed-topology Cluster creation.

Owner: HugeGraph vcluster/Test lane. Required artifact: terminal runtime results bound to this addon SHA and the exact KubeBlocks, test, vcluster, and image identities. Predecessor-head results do not transfer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

nopick Not auto cherry-pick when PR merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants