imbajin opened a new pull request, #3322:
URL: https://github.com/apache/hugegraph/pull/3322

   ## Purpose of the PR
   
   Container and Helm deployments could not pass a Server service ID or graph 
space into `rest-server.properties`, even when callers supplied the legacy 
environment variables. Map the supported settings before startup and expose 
them in the chart.
   
   ## Main Changes
   
   - Accept `HG_SERVER_SERVICE_ID` / `HG_SERVER_GRAPHSPACE` and migrate their 
legacy aliases; an effective nonempty new value takes precedence.
   - Add `server.serviceId` and `server.graphSpace` chart values, preserving 
existing DEFAULT behavior and avoiding duplicate identity variables.
   - Reject unsupported identity input before modifying configuration. 
Deployment service IDs accept printable ASCII, including spaces and 
backslashes; graph spaces follow the existing Server naming rules.
   - Document graph-space creation, authentication and the deployment input 
boundary. Core service naming and mounted properties retain their current 
behavior.
   
   ## Verifying these changes
   
   Shell and process checks, real HugeConfig readback controls, Helm tests, 
packaging, lint/render and schema CI checks passed. Formatting and clean 
compilation passed with Java 17; three independent final reviews passed. Linux 
discovery, authorization and isolation still require topology validation.
   
   ## Documentation Status
   
   `Doc - Done`: Docker and Helm guides in this PR. Paired website 
documentation: [ASF #517](https://github.com/apache/hugegraph-doc/pull/517). 
Coordinate both merges.
   
   
   <!-- This is an auto-generated comment: release notes by coderabbit.ai -->
   
   ## Summary by CodeRabbit
   
   - **新功能**
     - Docker 镜像和 Helm Chart 支持配置 Server 的服务 ID 与图空间名称;默认值为 `DEFAULT`,可保留现有配置。
     - Docker 镜像兼容旧环境变量,非空的新变量优先。Helm 配置会将非默认值传递给 Server。
   - **错误修复与校验**
     - 无效字符或不符合命名规则的值会导致启动或 Helm 渲染失败。图空间名称最长为 48 个字符,且目标图空间须预先创建。
   - **文档**
     - 补充配置说明,包括服务身份设置、图空间要求及相关限制。
   
   <!-- end of auto-generated comment: release notes by coderabbit.ai -->


-- 
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]

Reply via email to