nandakumar131 commented on code in PR #7669:
URL: https://github.com/apache/ozone/pull/7669#discussion_r1908843251


##########
hadoop-hdds/docs/content/feature/SCM-HA.md:
##########
@@ -33,14 +33,8 @@ This document explains the HA setup of Storage Container 
Manager (SCM), please c
 
 ## Configuration
 
-HA mode of Storage Container Manager can be enabled with the following 
settings in `ozone-site.xml`:
+As of Apache Ozone 2.0, SCM HA is always enabled. Test deployments can 
continue to use single node Ratis for SCM if needed.

Review Comment:
   The user might get confused with SCM HA and Ratis if they don't have 
understanding on how HA is implemented.
   We can remove this line completely as the end user doesn't have to know 
about Ratis to run single or multiple (HA) SCM.



##########
hadoop-hdds/docs/content/feature/SCM-HA.md:
##########
@@ -234,7 +228,7 @@ bin/ozone debug ldb --db=/tmp/metadata/scm.db scan 
--column-family=containers
 
 ## Migrating from existing SCM

Review Comment:
   ```suggestion
   ## Migrating from Non-HA to HA SCM
   ```



##########
hadoop-hdds/docs/content/feature/SCM-HA.md:
##########
@@ -234,7 +228,7 @@ bin/ozone debug ldb --db=/tmp/metadata/scm.db scan 
--column-family=containers
 
 ## Migrating from existing SCM
 
-SCM HA can be turned on on any Ozone cluster. First enable Ratis 
(`ozone.scm.ratis.enable`) and configure only one node for the Ratis ring 
(`ozone.scm.nodes.serviceId` should have one element).
+SCM HA is turned on for any Ozone cluster after upgrade to 2.0.0 or later. 
First configure only one node for the Ratis ring (`ozone.scm.nodes.serviceId` 
should have one element).
 
 Start the cluster and test if it works well.

Review Comment:
   ```suggestion
   Add additional SCM nodes and extend the cluster configuration to reflect the 
newly added nodes. 
   Bootstrap the newly added SCM nodes with `scm --bootstrap` command and start 
the SCM service.
   Note: Make sure that the `ozone.scm.primordial.node.id` property is pointed 
to the existing SCM before you run the `bootstrap` command on the newly added 
SCM nodes.
   ```



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