At lines 474-482, we have this code:
exists, err := dump.checkOplogTimestampExists(dump.oplogStart) if !exists { return fmt.Errorf( "oplog overflow: mongodump was unable to capture all new oplog entries during execution", ) } if err != nil { return fmt.Errorf("unable to check oplog for overflow: %v", err) }
Checking exists before err is backwards and produces confusing results.