errose28 commented on code in PR #380:
URL: https://github.com/apache/ozone-site/pull/380#discussion_r3113706207
##########
docs/05-administrator-guide/03-operations/03-node-decommissioning-and-maintenance/03-datanodes/02-datanode-maintenance.md:
##########
@@ -11,9 +11,21 @@ While in maintenance mode, a Datanode does not accept new
writes but may still s
The Datanode transitions through the following operational states during
maintenance:
1. **IN_SERVICE**: The Datanode is fully operational and participating in data
writes and reads.
-2. **ENTERING_MAINTENANCE**: The Datanode is transitioning into maintenance
mode. New writes will be avoided.
+2. **ENTERING_MAINTENANCE**: The Datanode is transitioning into maintenance
mode. New writes will be avoided. The SCM monitors the Datanode until it meets
all safety criteria before allowing it to fully enter maintenance.
3. **IN_MAINTENANCE**: The Datanode is in maintenance mode. Data will not be
written to it. If the Datanode remains in this state beyond the configured
maintenance window, its data will start to be replicated to other Datanodes to
ensure data durability.
+### Transition Criteria (ENTERING_MAINTENANCE to IN_MAINTENANCE)
+
+A Datanode will remain in the `ENTERING_MAINTENANCE` state until the SCM
(Storage Container Manager) verifies the following safety conditions:
+
+* **Pipeline Closure**: All open Ratis and EC pipelines on the Datanode must
be successfully closed. This ensures no active write operations are interrupted.
+* **Datanode Acknowledgment**: The Datanode must confirm it has received the
maintenance command and persisted the "ENTERING_MAINTENANCE" state to its local
disk. This prevents state confusion if the Datanode is rebooted.
+* **Container Health**: All containers on the DataNode must be in a stable
state (either CLOSED or QUASI_CLOSED). Nodes will not enter maintenance if they
+ contain OPEN or CLOSING containers
Review Comment:
If we are writing these in all caps as they appear in code and terminal
output we should wrap them in code blocks. This also means the spelling update
is not needed.
```suggestion
* **Container Health**: All containers on the DataNode must be in a stable
state (either `CLOSED` or `QUASI_CLOSED`). Nodes will not enter maintenance if
they
contain `OPEN` or `CLOSING` containers
```
--
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]
