shuan1026 opened a new pull request, #51:
URL: https://github.com/apache/ozone-helm-charts/pull/51

   ## What changes were proposed in this pull request?
   
   Analyse Ratis configurations for handling network partitions / slow 
networks, covering Ratis timeouts, heartbeat settings, and leader election. 
This follows up on the HA Helm work in comment 
[r2632023653](https://github.com/apache/ozone-helm-charts/pull/20#discussion_r2632023653).
   
   * Adds `ratis.ozoneSite` to `values.yaml`: an empty map by default for 
optional `ozone-site` property overrides, with a curated, commented example 
list covering OM/SCM Ratis network-resilience timeouts and leader election.
   * Renders `ratis.ozoneSite` entries as `OZONE-SITE.XML_*` env vars from 
`ozone.configuration.env.common` in `_helpers.tpl`, reusing the convention 
already used throughout the chart.
   
   Curated keys documented as commented examples in `values.yaml` (any other 
`ozone-site`
   property can also be set via this map):
   * OM: `ozone.om.ratis.server.request.timeout`, 
`ozone.om.ratis.minimum.timeout`,
     `ozone.om.ratis.server.failure.timeout.duration`, 
`ozone.om.ratis.server.leaderelection.pre-vote`
   * SCM: `ozone.scm.ha.ratis.request.timeout`, 
`ozone.scm.ha.ratis.leader.election.timeout`,
     `ozone.scm.ha.raft.server.rpc.first-election.timeout`,
     `ozone.scm.ha.ratis.server.leaderelection.pre-vote`,
     `ozone.scm.ha.ratis.server.failure.timeout.duration`
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-14350
   
   ## How was this patch tested?
   
   * Green CI run
   * Manual verification:
   
   ```bash
   # default: no ratis override env vars rendered
   helm template test-release charts/ozone --show-only 
templates/om/om-statefulset.yaml | grep OZONE-SITE.XML_
   
   # with overrides
   cat > /tmp/ratis-override.yaml <<'EOF'
   ratis:
     ozoneSite:
       ozone.om.ratis.server.request.timeout: "10s"
   EOF
   # expect:
   #   - name: OZONE-SITE.XML_ozone.om.ratis.server.request.timeout
   #     value: "10s"
   helm template test-release charts/ozone -f /tmp/ratis-override.yaml \
     --show-only templates/om/om-statefulset.yaml \
     | grep -A1 'OZONE-SITE.XML_ozone.om.ratis.server.request.timeout'
     
   # real cluster smoke test
   kind create cluster --name ozone-helm-ratis
   helm upgrade --install ozone charts/ozone --wait --timeout 15m
   helm upgrade ozone charts/ozone -f /tmp/ratis-override.yaml --wait --timeout 
15m
   kubectl exec ozone-om-0 -- env | grep ratis.server.request.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]

Reply via email to