imbajin commented on code in PR #517: URL: https://github.com/apache/hugegraph-doc/pull/517#discussion_r4263545513
########## content/en/docs/quickstart/hugegraph/hugegraph-server.md: ########## @@ -123,6 +123,48 @@ The standalone `docker-compose.yml` example uses `hugegraph/hugegraph:1.7.0`; it See [docker/README.md](https://github.com/apache/hugegraph/blob/master/docker/README.md) for the full setup guide. +#### Server service identity with PD + +For a PD-backed Server, the Helm chart exposes `server.serviceId` and `server.graphSpace`. +Both default to `DEFAULT`, which preserves the existing `server.service_id` and `server.graphspace` +values in `conf/rest-server.properties`. A non-default chart value injects `HG_SERVER_SERVICE_ID` +or `HG_SERVER_GRAPHSPACE` into the Server container. Use a Helm chart and Server image from the same +version, both including the service identity support in [Server PR #3322](https://github.com/apache/hugegraph/pull/3322). +Copying these values into an older chart or image does not add that support; do not use this example +with 1.7.0 or any chart/image version without these changes. +Non-empty service IDs supplied through `HG_SERVER_SERVICE_ID`, `SERVICE_ID`, or `server.serviceId` +must use printable ASCII; spaces and backslashes are supported, while deployment checks reject +non-ASCII and control characters. This entrypoint restriction does not change Core Service naming +or existing manually supplied properties. Helm preserves dollar signs in `server.serviceId` literally, +including `$(POD_IP)`; this field does not expand Kubernetes environment references. +Set the identity through these chart values; Server `extraEnv` reserves both the new and legacy identity variable names. +`HG_SERVER_GRAPHSPACE`, the effective `GRAPH_SPACE` fallback, +and `server.graphSpace` accept `DEFAULT` or an existing space name of up to 48 lowercase letters, digits, +and underscores, starting with a letter: + +```bash +helm upgrade --install hg ./helm/hugegraph \ + --set-string server.serviceId=analytics \ Review Comment: Addressed in 957409ff8: cleaned all unreleased PR references to #3322 and other PRs from documentation prose across Chinese and English guides, stating the service identity capability generically. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
