[SERVER-70103] Fix the accuracy of the test assertion in mirror_reads.js Created: 29/Sep/22  Updated: 29/Oct/23  Resolved: 31/Oct/22

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: 6.2.0-rc0

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

Issue Links:
Depends
Related
related to SERVER-65289 Improvements to Mirrored Reads Metric... Closed
Backwards Compatibility: Fully Compatible
Operating System: ALL
Sprint: Service Arch 2022-10-17, Service Arch 2022-10-31, Service Arch 2022-11-14
Participants:
Linked BF Score: 13

 Description   

The issue is outlined in the comment of the linked BF.

Possible fixes:

  1. Change naming and semantics of metrics. Example name changes could be from resolved to acknowledged/receivedResponse, and from received to recognized/mirrored. The metrics can be placed in more precise locations based on names, and the assertions can be reorganized or changed.
  2. Add a test only metric to measure how many responses were not successful. The current received metric is only incremented if the responses were successful, so assertions can be changed to consider how many responses were successfully completed.


 Comments   
Comment by Githook User [ 31/Oct/22 ]

Author:

{'name': 'Alex Li', 'email': '98560274+lia394126@users.noreply.github.com', 'username': 'lia394126'}

Message: SERVER-70103 Add mirroredReads succeeded and processed server status metrics
Branch: master
https://github.com/mongodb/mongo/commit/a17d06871696dcd5f7e9e35ecac54b205e29297b

Generated at Thu Feb 08 06:15:16 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.