imbajin commented on code in PR #517: URL: https://github.com/apache/hugegraph-doc/pull/517#discussion_r4286185240
########## content/en/docs/quickstart/hugegraph/hugegraph-server.md: ########## @@ -123,6 +123,59 @@ 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 supporting service identity. Review Comment: Updated the service identity section in both EN and CN to clearly state that `server.serviceId`, `server.graphSpace`, and `server.testGraphAlias` require Helm chart and Server image support introduced in apache/hugegraph#3322, and that no released chart or image currently supports these settings. ########## content/en/docs/guides/hugegraph-docker-cluster.md: ########## @@ -221,12 +221,28 @@ Unlike PD and Store, Server has no mandatory entrypoint variables: only set valu | `HG_SERVER_PD_REST_ENDPOINT` | Derived from `pd.peers` by replacing `:8686` with `:8620` | PD REST address polled by `wait-storage.sh` | | `PD_AUTH_USER` | `store` | Basic username for PD REST calls from `wait-storage.sh` | | `PD_AUTH_PASSWORD` | Empty | PD REST Basic password; must match `HG_PD_AUTH_SECRET_KEY` when PD REST authentication is enabled | -| `WAIT_PARTITION_TIMEOUT_S` | `120` | Partition allocation wait time in `wait-partition.sh` | +| `WAIT_PARTITION_TIMEOUT_S` | `120` | Partition polling timeout; also bounds the final diagnostic in builds with the fix described below | + +Images containing the authentication property fixes preserve spaces, backslashes and Unicode in explicit Review Comment: Explicitly named the exact boundaries across docker-cluster, Store, and PD pages in both languages (apache/hugegraph#3327, #3316, #3305, and #3310). ########## content/en/docs/guides/hugegraph-docker-cluster.md: ########## @@ -221,12 +221,28 @@ Unlike PD and Store, Server has no mandatory entrypoint variables: only set valu | `HG_SERVER_PD_REST_ENDPOINT` | Derived from `pd.peers` by replacing `:8686` with `:8620` | PD REST address polled by `wait-storage.sh` | | `PD_AUTH_USER` | `store` | Basic username for PD REST calls from `wait-storage.sh` | | `PD_AUTH_PASSWORD` | Empty | PD REST Basic password; must match `HG_PD_AUTH_SECRET_KEY` when PD REST authentication is enabled | -| `WAIT_PARTITION_TIMEOUT_S` | `120` | Partition allocation wait time in `wait-partition.sh` | +| `WAIT_PARTITION_TIMEOUT_S` | `120` | Partition polling timeout; also bounds the final diagnostic in builds with the fix described below | + +Images containing the authentication property fixes preserve spaces, backslashes and Unicode in explicit +signing keys. Use the same key on every Server replica. When upgrading from an image that misread such a key, explicitly supply the intended +`HG_SERVER_AUTH_TOKEN_SECRET` again; retaining the old configuration also retains its misread key. If the active key changes, sign in again to +obtain new tokens. The 1.7.0 image limitation above still applies. + +Older images could initialize a Unicode administrator password incorrectly when the initializer inherited the C locale. Upgrading or setting +`PASSWORD` again does not repair an existing account's stored password. A working administrator must change it through the +[user API](/docs/clients/restful-api/auth/). This fix provides no automatic recovery when no administrator identity is usable; preserve the +existing data and arrange account recovery before upgrading. > **Deprecated aliases**: `BACKEND` → `HG_SERVER_BACKEND`, `PD_PEERS` → > `HG_SERVER_PD_PEERS`. `wait-storage.sh` waits up to 300 seconds for an `Up` Store. This duration is hardcoded and cannot be changed through environment variables. +In repaired Server builds, HStore startup gives partition polling +and the final `/v1/partitions` diagnostic separate `WAIT_PARTITION_TIMEOUT_S` budgets. Earlier builds bound polling only; the final diagnostic +request has no timeout. With the default value, a stalled final response times out after 120 seconds; setting the value to `0` disables +GNU timeout's limit. JVM exit observation may wait until that diagnostic completes or reaches its timeout; signal-triggered shutdown remains Review Comment: Documented in both EN and CN that the entrypoint waits for `wait-partition.sh` before waiting for the Java launcher: if Java exits while partition polling is running, the poll can consume its full timeout before the exit is observed; if polling succeeds, the diagnostic step can consume a second full timeout. -- 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]
