[CDRIVER-4172] Migrate uses of json.send to perf.send in Evergreen project YAML Created: 01/Oct/21  Updated: 28/Oct/23  Resolved: 02/Nov/21

Status: Closed
Project: C Driver
Component/s: None
Affects Version/s: None
Fix Version/s: 1.20.0

Type: Task Priority: Major - P3
Reporter: Maria van Keulen Assignee: Roberto Sanchez
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Cloners
clones GODRIVER-2169 Migrate uses of json.send to perf.sen... Closed
Issue split
Related
Upstream Changes Summary:

DRIVERS-1892:
As of PM-2172, the TIPS team would like to remove its legacy performance gathering and analysis API in favor of a new, more robust option. In order to migrate to the new API, uses of json.send in Evergreen project YAML should be removed in favor of using perf.send (assuming it is desirable to continue to maintain / run the testing that uses json.send) . Please see the linked DRIVERS ticket for detailed information about how to use perf.send. We have also added docs for migrating away from json.send and onto perf.send.


 Description   

Summary

As of PM-2172, the TIPS team would like to remove its legacy performance gathering and analysis API in favor of a new, more robust option. In order to migrate to the new API, uses of json.send in Evergreen project YAML should be removed in favor of using perf.send (assuming it is desirable to continue to maintain / run the testing that uses json.send) . The following instructions describe how to use perf.send. We have also added docs for migrating away from json.send and onto perf.send.

Motivation

Who is the affected end user?

This ticket tracks the mongo-python-driver-perf change specifically.

How does this affect the end user?

The trend charts UI will continue to display any legacy performance data that had previously been gathered with json.send even after migration to perf.send. Additionally, it appears that change points are not presently being detected for these projects even in cases where performance data is being collected; migration to perf.send should avoid these sorts of errors.

How likely is it that this problem or use case will occur?

N/A

If the problem does occur, what are the consequences and how severe are they?

N/A

Is this issue urgent?

TIPS would like to remove the json.send tooling as part of PM-2172, which we are working on this quarter. If all the usages could be removed or migrated to the new tooling by October 25th, that would be great.

Is this ticket required by a downstream team?

Yes. It is required by TIPS for PM-2172.

Is this ticket only for tests?

Yes. This is specific to gathering and analyzing performance data.



 Comments   
Comment by Githook User [ 02/Nov/21 ]

Author:

{'name': 'Roberto C. Sánchez', 'email': 'roberto@connexer.com', 'username': 'rcsanchez97'}

Message: CDRIVER-4172 use perf.send for sending performance test results
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/3b9f2124a41c3d5874eb06b4d59a129d799c7749

Comment by Githook User [ 02/Nov/21 ]

Author:

{'name': 'Roberto C. Sánchez', 'email': 'roberto@connexer.com', 'username': 'rcsanchez97'}

Message: CDRIVER-4172 output performance results in new perf.send format
Branch: master
https://github.com/mongodb/mongo-c-driver-performance/commit/3959c333735149274cee08edf4df3a70a015c090

Comment by Roberto Sanchez [ 01/Nov/21 ]

maria.vankeulen I was able to access it. Thanks for your assistance. I have implemented the necessary changes to the C driver project and they are pending review.

Comment by Maria van Keulen [ 01/Nov/21 ]

roberto.sanchez were you able to receive access to the repo? IIRC you can request access via MANA.

Comment by Maria van Keulen [ 28/Oct/21 ]

Hello alexander.golin, do we have a sense when there will be time to work on this?

Generated at Wed Feb 07 21:20:09 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.