[SERVER-44313] Background validation hook checks wrong serverStatus parameter Created: 30/Oct/19  Updated: 29/Oct/23  Resolved: 05/Nov/19

Status: Closed
Project: Core Server
Component/s: Storage
Affects Version/s: None
Fix Version/s: 4.3.1

Type: Bug Priority: Major - P3
Reporter: Gregory Wlodarek Assignee: Gregory Wlodarek
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Backwards Compatibility: Fully Compatible
Operating System: ALL
Sprint: Execution Team 2019-11-18
Participants:

 Description   

We're currently checking the wrong parameter from "serverStatus" to see if checkpoint cursors are supported by a storage engine. This prevents the background validation hook from running at all.

 Here, we check for

!conn.adminCommand("serverStatus").storageEngine.supportsCheckpoints

when we should be checking for

!conn.adminCommand("serverStatus").storageEngine.supportsCheckpointCursors

 

Currently, running this with the WiredTiger we see this:

Skipping background validation against test node: localhost:20000 because its storage engine does not support background validation (checkpoints).

 

Logging the "serverStatus.storageEngine" we have these options:

{
 "name" : "wiredTiger",
 "supportsCommittedReads" : true,
 "oldestRequiredTimestampForCrashRecovery" : Timestamp(1572447953, 4),
 "supportsPendingDrops" : true,
 "dropPendingIdents" : NumberLong(0),
 "supportsSnapshotReadConcern" : true,
 "supportsCheckpointCursors" : true,
 "readOnly" : false,
 "persistent" : true,
 "backupCursorOpen" : false
 }



 Comments   
Comment by Githook User [ 05/Nov/19 ]

Author:

{'username': 'GWlodarek', 'email': 'gregory.wlodarek@mongodb.com', 'name': 'Gregory Wlodarek'}

Message: SERVER-44313 Background validation hook checks correct serverStatus parameter
Branch: master
https://github.com/mongodb/mongo/commit/eb9ba994bce0e44562e877530a29a01fc8b8da6c

Generated at Thu Feb 08 05:05:39 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.