ArafatKhan2198 opened a new pull request, #6640:
URL: https://github.com/apache/ozone/pull/6640

   ## What changes were proposed in this pull request?
   **_Changes in this patch include the following :-_** 
   ### **Enhanced Logging for OM Snapshot Fetching**
   We enhanced the logging in the getOzoneManagerDBSnapshot method to provide 
detailed information when Recon fetches the OM snapshot. This includes checks 
for the presence of SST files, logging a warning if the snapshot is empty or 
lacks SST files, which could indicate issues such as OM leader instability or 
snapshot corruption.
   
   ### **Logging Last Sequence Number on Periodic Syncs**
   The synchronization process between Recon and the OM database now includes 
logging of the last sequence number before and after each sync. This 
enhancement helps in monitoring the progress of data synchronization and aids 
in debugging issues related to data consistency and timing.
   
   ### **Permission Checks for Recon DB Directories**
   We implemented a method to validate the permissions of the Recon DB 
directories against the expected configuration settings. The method logs a 
warning if the actual permissions do not match the expected permissions, 
enhancing operational awareness.
   
   ## What is the link to the Apache JIRA
   https://issues.apache.org/jira/browse/HDDS-10813
   ## How was this patch tested?
   
   Manually Tested out the logs. The required log messages get printed :- 
   
   1. Logging for SST Files Availability on Full DB Fetch: 
   ```
   2024-05-06 02:13:03 2024-05-05 20:43:03,285 [Recon-SyncOM-0] INFO 
impl.OzoneManagerServiceProviderImpl: Valid OM snapshot with SST files found 
at: /data/metadata/om.snapshot.db_1714941781590
   ```
   2. Printing the Sequence Number Before and After Sync: 
   ```
   2024-05-06 02:13:01 2024-05-05 20:43:01,587 [Recon-SyncOM-0] INFO 
impl.OzoneManagerServiceProviderImpl: Last known sequence number before sync: 0
   .....
   2024-05-06 02:13:03 2024-05-05 20:43:03,620 [Recon-SyncOM-0] INFO 
impl.OzoneManagerServiceProviderImpl: Sequence number after sync: 2
   ```
   3. Recon Data Directory Permissions:
   ```
   2024-05-06 02:13:01 2024-05-05 20:43:01,590 [Recon-SyncOM-0] INFO 
impl.OzoneManagerServiceProviderImpl: Permissions for Recon DB directory 
'/data/metadata' are correctly set to 'rwxr-x---'
   ```


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